Function issue

[pieterdehoogd]

Hello,

I wrote a code in C++ that can encode and decode files. After that the programm have to check or numbers from the file are lychrel numbers. I used functions for every action.
But when I put all those functions in a row, the last one doesn't work. See my code:

#include <iostream>
#include <fstream>
#include <climits>
using namespace std;

    long omgekeerd(long);
    bool isPalindroom(long);

void bestanden(ifstream & invoer, ofstream & uitvoer, char & keuze) {
    string origineel; //Bestandsnaam invoer
    string doelfile; //Bestandsnaam uitvoer
    cout << "Goedendag, u kunt kiezen of u met dit programma een bestand wilt "
         << "coderen of decoderen." << endl;
    cout << "Voor coderen voer een 'c' in. Voor decoderen voer een 'd'"
         << "in." << endl;
    cin >> keuze;
    cout << "Voer de naam van het originele bestand in: " << endl;
    cin >> origineel;
    invoer.open(origineel, ios::in);
    cout << "Voer de gewenste naam van het doelbestand in: " << endl;
    cin >> doelfile;
    uitvoer.open(doelfile, ios::out);
}

bool isBackslash(char c) { //Bepaalt of karakter een backslash is
    if(c == 92) { //c is nummer van karakter in ascii
        return true;
    }
    else {
        return false;
    }
}//isBackslash

bool isCijfer(char c) { //Bepaalt of karakter een cijfer is
    if (c > 48 && c < 58) { //c is nummer van karakter in ascii
        return true;
    }
    else {
        return false;
    }
}//isCijfer

void coderen(ifstream & invoer, ofstream & uitvoer) { //Codeert invoer
    char kar1, kar2; //kar 1 is eerste karakter in invoer, kar2 de volgende
    int i = 1; //aantal zelfde karakters
    bool dubbel = false; //staan er 2 dezelfde karakters achter elkaar?

    kar1 = invoer.get();
    kar2 = invoer.get();
//coderen
    while(!invoer.eof()) {
    if(kar1 == '\n') {
        uitvoer.put('\n');
    }
    else {
        if(kar1!=kar2) {
            if(isBackslash(kar1) || isCijfer(kar1)) {
                uitvoer.put('\\');
            }
            uitvoer.put(kar1);
            if(i>1) {
                uitvoer << i;
            }
            i=1;
        }
        else {
            i++;
        }
    }
//Cijfers uit het invoerbestand naar het uitvoerbestand
    if(isCijfer(kar1)){
            dubbel = true;
        }
        kar1 = kar2;

        if (!isCijfer(kar2)&&dubbel==true){
            dubbel= false;
        }
        kar2 = invoer.get();
        }
        uitvoer.put(kar1);
        if (i>1){
            uitvoer << i;
    }
    cout << "Uw bestand is gecodeerd." << endl;
}//coderen

void decoderen(ifstream & invoer, ofstream & uitvoer){ //decodeert de invoer
    char kar1, kar2; //kar 1 is een volgende regel, kar 2 is volgende karakter
    int getal = 0; //Uitrekenen werkelijke getal
    int i = 1; //Teller voor aantal dezelfde karakters

    kar1 = '\n';
    kar2 = invoer.get();
//decoderen
    while (!invoer.eof()){
        if (isCijfer(kar2)) {
            getal = getal * 10 + (kar2 - '0');
        }
        else {
            if(getal > 0) {
                for(i=1; i < getal; i++)
                    uitvoer.put(kar1);
                    getal=0;
            }
            if(isBackslash(kar2)) {
                kar2 = invoer.get();
            }
            uitvoer.put(kar2);
            kar1 = kar2;
        }
        kar2 = invoer.get();
        }
    cout << "Uw bestand is gedecodeerd." << endl;
}//decoderen

bool isPalindroom(long getal) { //check of getal een palindroom is
    return getal == omgekeerd(getal);
}//isPalindroom

long omgekeerd(long num) { //is een getal een spiegelgetal?
    long omgekeerd = 0; //check of getal een spiegelgetal is
    while(num > 0) {
        long rest = num%10; //getal wat bij berekening er nog bij komt
        omgekeerd = (omgekeerd*10)+rest;
        num=num/10;
    }
    return omgekeerd;
}//omgekeerd

bool checkLychrel(long num, int & p) { //checkt of getal een lychrelgetal is
    if(isPalindroom(num)) {
        if(p>0) {
            p=p-1;
        }
        return false;
    }
    else {
    for(p=1; p>0; p++) { //p is teller voor iteraties
        num = num + omgekeerd(num);
        if(isPalindroom(num)) {
            return false;
        }
    }
     return true;
}
}//checkLychrel

void isLychrel(ifstream & invoer) {//haalt getallen uit de invoer en haalt deze
    int p=0; //teller iteraties    //door checkLychrel
    char kar; //karakter uit de invoer
    while(!invoer.get(kar)) {
        if(isCijfer(kar)) {
            invoer.unget();
            int n; //geheel getal uit invoer
            invoer >> n;
            if(n < INT_MAX) {
                if(checkLychrel(n, p)) {
                    cout << n << " is een lychrelgetal. " << endl;
            }
                else {
                cout << n << " is geen lychrelgetal." << " Het aantal iteraties"
                     << " om bij een palindroom te komen is " << p << endl;
            }
            }
            else {
                cout << n << " Dit getal is te groot." << endl;
            }
    }
}
}//isLychrel

int main() {
    ifstream invoer;
    ofstream uitvoer;
    char keuze;
    bestanden(invoer, uitvoer, keuze);
    if(keuze == 'c') {
        coderen(invoer, uitvoer);
    }
    if(keuze == 'd') {
        decoderen(invoer, uitvoer);
    }
    else {
        cout << "Wrong letter." << endl;
    }
    cout << "Dit programma controleert nu alle getallen uit de invoerfile op"
         << " lychrelgetallen." << endl;
    isLychrel(invoer);
return 0;
}//main 

So, everything works, except the isLychrel function.
When I write the code withoud the encode (void coderen()) part or the decode (void decoderen()) part, the isLychrel function works. What am I doing wrong?

[Handy Andy]

Hello pieterdehoogd,

So what is not working in the "main" function? (You said the last function).

A few blank through out your code would make it much easier to read and avoid any delays in a response.

Andy

[pieterdehoogd]

in the main function:
void bestanden(), user can choose whether he wants to encode or decode and user have to give the filenames.

if he choose 'c' the programm must go to the function coderen() (coderen is dutch for encode) and if he choose 'd' the programm must go to the function decoderen() (decoderen is dutch for decode).
If he choose an other letter than c or d the programm says: Wrong letter.
When the programm is done with encode or decode, all of the numbers from the imputfile have to be checked whether they are lychrel numbers or not. I wrote a function void isLychrel().

So when I only put void isLychrel in the main function, the programm checks al of the numbers from the imput file whether they are lychrel number or not. But when I put the if(keuze == 'c') then coderen en if(keuze == 'd') then decoderen part in the main function, the isLychrel function doesn't work anymore...

[Handy Andy]

Hello pieterdehoogd,

Thank you. I believe that you are the first to mention what language you use in your program. It really helps.

Now that I understand what to look for I will check into it. Translating some of the code will take a bit longer, but not a problem.

Andy

[pieterdehoogd]

Hello Andy,

It's written in C++, dutch language.
Thank you.

[jlb]

What exactly do you mean by "doesn't work"?

Look at this snippet:

void isLychrel(ifstream & invoer)  //haalt getallen uit de invoer en haalt deze
{
    int p = 0; //teller iteraties    //door checkLychrel
    char kar; //karakter uit de invoer
    while(!invoer.get(kar))
    {
        if(isCijfer(kar))

First since you are using this stream in several places I would recommend you always check the state of the stream before you try to use it and you may also want to rewind the stream in this function as well.

Also why are you only executing the code within that while() statement when you fail to get a character?

Also here: http://www.cplusplus.com/forum/general/273355/