C++ Débutant: Passer un fstream à une fonction pour lire les données

J'ai un fichier texte nommé num.txt qui n'contenu est la ligne 123. Puis-je avoir le suivant:

void alt_reader(ifstream &file, char* line){
    file.read(line, 3);
    cout << "First Time: " << line << endl;
}

int main() {
    ifstream inFile;
    int num;
    inFile.open("num.txt");
    alt_reader(inFile, (char*)&num);
    cout << "Second Time: " << num << endl;
}

La sortie est:

First Time: 123
Second Time: 3355185

Pouvez-vous m'aider à comprendre comment obtenir un fstream qui est lu dans une fonction toujours attribuer à la variable principale? Je fais cela parce que alt_reader a vraiment beaucoup plus à lui, mais c'est la partie que je suis coincé sur. Merci beaucoup pour l'aide.

Mise à JOUR:
En utilisant le projet de Loi Oneal les commentaires, j'ai écrit

void alt_reader(ifstream &file, stringstream &str, int n){
    char buffer[n+1];
    file.read(buffer, n);
    buffer[n] = 0;
    str << buffer;
    cout << "First Time: " << buffer << endl; //First Time: 123
}

int main() {
    ifstream inFile;
    stringstream strm;
    int num;
    inFile.open("num.txt");
    alt_reader(inFile, strm, 3);
    cout << "Second Time: " << num << endl; //Second Time: 123
}

Grâce. Toutes les critiques avec ce qui est là maintenant?

InformationsquelleAutor physicsmichael | 2010-04-05
  1. 7

    La première fois que vous avez imprimé la variable, de l'imprimer comme un char *, l'impression de traiter le fichier en tant que fichier texte (Et vous avez de la chance vous n'avez pas de crash). Le deuxième temps de l'imprimer, vous réinterprété comme un int, faisant de la représentation de complètement différent.

    Chaque fois que vous lancez des pointeurs d'un type vers un autre type que vous êtes généralement en invoquant un comportement indéfini. Depuis char n'a pas de norme concernant int, vous l'avez ici.

    EDIT: pour répondre À votre question:

    #include <sstream>

void foo(std::stream &str) {
 str << "42\n";
};

int main() {
 int aNumber;
 std::stringstream aStringStream;
 foo(aStringStream); //Pass our stream to the function. It contains
    //"42\n" when the function returns.
 aStringStream >> aNumber; //aNumber == 42
 aNumber += 10; //aNumber == 52;
 std::cout << aNumber; //Print "52"
}
    • Merci. Des conseils sur la façon de passer un fstream et l'obtention de la lecture de la valeur de la fonction (sans une instruction de retour?)?
    • ONeal: j'ai juste ajouté une tentative d'élargir mon code avec votre technique. Elle fonctionne. Que pensez-vous de cela?
    • Bien, vous pouvez le faire de cette façon, mais en général c'est mieux pour le type de retour de données sûre. Je suis en supposant que vous avez une très bonne raison pour avoir la fonction d'écriture de la chaîne directement ici. Oh .. c'est un dépassement de la mémoire tampon: char buffer[n]; buffer[n] = 0;. L'indice maximum d'un tableau de longueur n est n-1. Aussi: je suis surpris de voir que même compile parce que la matrice de déclarations doivent être constante.
    • ONeal: Merci, fixe l'index. Je n'ai jamais vraiment compris lorsque array déclarations doivent être constante. Depuis que j'ai déménagé à la C++ ça a été comme ils utilisent tous une certaine forme de malloc pour faire la déclaration et ils n'ont pas besoin d'être constant. Mais je suis terrible à C++!
    • ONeal: afin De clarifier la nécessité pour cela, je suis en train de lire un fichier binaire qui est quelque chose comme: int header_length; infile.read( (char*)&header_length, sizeof(header_length)); if(byteswap_bool) EndianByteSwap(header_length); et je suis juste essayer de condenser le lire et si les appels vers quelque chose de plus propre.
    • Sonne comme un cas de l'Abstrait modèle de Fabrique, où l'usine est initialisé au démarrage après un endian détection. Ensuite, vous n'avez pas la vérification constante de l'endianness tout le temps, vous pouvez utiliser la plus évidente des opérations pour chaque action et d'obtenir le comportement que vous souhaitez.
    • ONeal: quelque Chose comme ça serait l'idéal, mais je crains que ça va un peu au-dessus de ma tête dans la façon de mettre en œuvre "de l'initialisation de l'usine" après-boutiste de détection. Pourriez-vous un lien de quelque chose d'enseignement afin de me permettre d'en savoir plus sans vous déranger plus loin? =)

    InformationsquelleAutor Billy ONeal
  2. 2

    Vous avez au moins deux problèmes ici.

    Dans main():

    1. Vous devriez être en train de passer un tampon de chaîne en alt_reader, vous êtes plutôt du passage d'un int.
    2. Vous semblez vouloir lire la chaîne '123' mais qui veulent un int pour avoir la valeur de 123.

    Que vous pouvez faire:

    void alt_reader(ifstream &file, char* line){
    file.read(line, 3);
    line[3]=0;
    cout << "First Time: " << line << endl;
}

int main() {
    ifstream inFile;
    inFile.open("num.txt");

    char buffer[128];
    alt_reader(inFile, buffer);

    int num=atoi(buffer);
    cout << "Second Time: " << num << endl;

    return 0;
}

    Notez que j'ai ajouté line[3]=0; à alt_reader et atoi la conversion de chaîne de caractères (un scii) int.

    • Qui fonctionne parfaitement pour la situation que j'ai décrite, mais le nombre je vais lire n'est pas nécessairement un int, et la chaîne de flux ci-dessous seront heureux de vous aller dans un int ou double. Même alors, merci beaucoup.
    InformationsquelleAutor quamrana