[C++] probleem met inlezen bestand *

[GAEvakYD]

Ik moet een opdracht maken die vanaf 1 regel cijfers inleest. Hier moet ik dan later een driehoek van maken.De cijfers kunnen varieren van 1 tot 1000 en zijn gescheiden door een tab de 3 cijfers worden beeindigd door END aan het einde van de file te zetten.


Dit character voor character inlezen gaat goed, ook detecteerd hij de tabs enzo. Alleen als ik ze dan weer ga geven krijg echt van die wazige tekens te zien. Meer als er een fout zit in de pointers ofzo.

Please help me.

code:

#include <vcl.h>
#pragma hdrstop
#include <iostream.h>
#include <fstream.h>
#include <fstream>
#include <string.h>
#define TAB "\t"




//---------------------------------------------------------------------------

#pragma argsused

class CTriangle {
    int a, b, c;
  public:
    CTriangle(){};
    ~CTriangle(){};
    void set_valueA (int);
    void set_valueB (int);
    void set_valueC (int);
    int get_from_file(CTriangle);
    int get_valueA();
    int get_valueB();
    int get_valueC();


  //  int a, b, c;
};

const char * filename = "triangle.txt";


void CTriangle::set_valueA (int aa) {
  a = aa;
}

void CTriangle::set_valueB (int bb) {
  b = bb;
}

void CTriangle::set_valueC (int cc) {
  c = cc;
}


int CTriangle::get_valueA(){
return a;
}

int CTriangle::get_valueB(){
return b;
}

int CTriangle::get_valueC(){
return c;
}

int get_from_file(CTriangle& test){

  char * buffer;
  char a[50],b[50],c[100],END[50];

  long size,size2;
  int iTeller=0;
  int iTeller2=0;
//  size2 = 2;

  ifstream file (filename, ios::in|ios::binary|ios::ate);
  size = file.tellg();
  file.seekg (0, ios::beg);
  while (iTeller != size) {
    buffer = new char [size];
    file.read (buffer, 1) ;

    if (!strcmp(buffer,TAB))
    {
     cout << "tab found" << endl;
     iTeller2++;
    }
    else
    {
    if (iTeller2 == 0)
    {
      cout << buffer << endl;
      strcat(a,buffer);
    }
    if (iTeller2 == 1)
    {
      cout << buffer << endl;
      strcat(b,buffer);
    }
    if (iTeller2 == 2)
    {
      cout << buffer << endl;
      strcat(c,buffer);
    }
    if (iTeller == 3)
    {
     strcat(END,buffer);
       if (END== "END")
       {
         break;
       }
    }
    }
    iTeller++;
  }
  file.close();
  cout << "the complete file is in a buffer" << endl;
  cout << a << endl;
  cout << b << endl;
  cout << c << endl;
//  delete[] buffer;
};


int main () {
  CTriangle rect;
  get_from_file(rect);
//  cout << "Zijde A: " <<  rect.get_valueA() << endl;
//  cout << "Zijde B: " <<  rect.get_valueB() << endl;
//  cout << "Zijde C: " <<  rect.get_valueC() << endl;

  int iTemp;  cin >> iTemp;
}

[whoami]

Doe eens

code:

cout << *buffer;

ipv

code:

cout << buffer;

In het eerste geval gaat de variable waarnaar buffer verwijst getoond worden, in het 2de geval het adres waar buffer naar verwijst.
De naam van uw tabel is nl een pointer naar de geheugenlocatie van het eerste element van uw tabel.

_{Tip: zet in het vervolg code - tags rond uw code, dat leest makkelijker}

[GAEvakYD]

Dat gaat wel goed. de fout krijg ik bij dit stuk

cout << a << endl;
cout << b << endl;
cout << c << endl;

en dat zijn geen pointers

[whoami]

Op vrijdag 08 februari 2002 14:07 schreef GAEvakYD het volgende:
Dat gaat wel goed. de fout krijg ik bij dit stuk

cout << a << endl;
cout << b << endl;
cout << c << endl;

en dat zijn geen pointers

Het zijn wel pointers. Kijk maar eens je declaratie na:

code:

char a[50]

Een array is , zoals ik eerder al zei een pointer.
Dus, als je doet

code:

cout << a;

Dan ga je eigenlijk het adres van het eerste element van de array gaan tonen. Je kunt dit op die manier oplossen:

code:

cout << *a

of

code:

cout << a[0]; // voor het eerste element te tonen

(Edit eens je post, en zet code-tags rond je code, dan is het al wat makkelijker leesbaar en is het makkelijker om fouten op te sporen).

[GAEvakYD]

Dammit heb beide manieren geprobeerd maar lukt nog steeds niet./

cout << "a is:" << a[0] endl;
cout << "b is:" <<*b << endl;
cout << "c is:" <<*c << endl;


Ligt het niet aan de reservering van het geheugen ofzo??, ik heb al lopen pielen met a[4] enzo maar niets helpt

[whoami]

Je leest nummers (int) in in een character array. Die integers zullen dan, doordat ze in een char-veld komen omgezet worden naar de ASCII waarde van dat getal en daardoor krijg je dus die rare tekens. Je krijgt dus de ASCII representatie van ieder nummer.

Zoek eens naar functies die int naar char converteren enzo.

[GAEvakYD]

Ja maar dat gaat niet werken. Ik lees ze in van een fiel dus zijn ze auto als char *. Als ik probeer te conveteren zegt die dat ook namelijk

[GAEvakYD]

Dat gaat niet werken, de ingelezen charaters zijn al char *.
Dus van integer naar char omzetten hoeft niet meer.

Heb ik een pointer ofzo verkeerd staan ofzo, bij het vullen van de variabelen a.b.c????

Ik worddd gek

[eek]

Hmm.. laat ik het zo stellen hier
klopt dus echt totaal niks van, ga eerst maar
weer eens goed nadenken. Je wil meerdere punten
inlezen maar in die klasse heb je geen array van
getallen. Je geeft test mee aan "get_from_file"
maar die zien we nergens meer terug in je functie.
Of je code klopt niet of ik snap er geen f*ck van
wat je wilt.

[GAEvakYD]

Heb je gelijk in dat ik test niet gebruikte. Was me nog helemaal niet opgevallen. Maar de functies GetValueA kan ik ook niet pakken want dat zijn INTergers. En dan werkt de het optellen van de getallen niet meer want 1+0 = 1 en geen 10.



Iemand een oplossing ??

[GAEvakYD]

Ik heb het geprobeert op te lossen, maar er zit ergens een fout in. hij telt niet helemaal goed

de Uitvoer is nu

Zijde a = 20
Zijde b = 2580
Zijde C = 40

in de file staat dit 10 20 30 END

code:

//---------------------------------------------------------------------------

#include <vcl.h>
#pragma hdrstop
#include <iostream.h>
#include <fstream.h>
#include <fstream>
#include <string.h>
#define TAB "\t"




//---------------------------------------------------------------------------

#pragma argsused

class CTriangle {
    int a, b, c;
  public:
    CTriangle(){};
    ~CTriangle(){};
    void set_valueA (int);
    void set_valueB (int);
    void set_valueC (int);
    int get_from_file(CTriangle);
    int get_valueA();
    int get_valueB();
    int get_valueC();


  //  int a, b, c;
};

const char * filename = "triangle.txt";


void CTriangle::set_valueA (int aa) {
  a = aa;
}

void CTriangle::set_valueB (int bb) {
  b = bb;
}

void CTriangle::set_valueC (int cc) {
  c = cc;
}


int CTriangle::get_valueA(){
return a;
}

int CTriangle::get_valueB(){
return b;
}

int CTriangle::get_valueC(){
return c;
}

int get_from_file(CTriangle& test){

  char * buffer;
  char a[50],b[50],c[100],END[50];

  long size,size2;
  int iTeller=0;
  int iTeller2=0;
//  size2 = 2;

  ifstream file (filename, ios::in|ios::binary|ios::ate);
  size = file.tellg();
  file.seekg (0, ios::beg);
  while (iTeller != size) {
    buffer = new char [size];
    file.read (buffer, 1) ;

    if (!strcmp(buffer,TAB))
    {
     cout << "tab found" << endl;
     iTeller2++;
    }
    else
    {
    if (iTeller2 == 0)
    {
      cout << buffer << endl;
      if (test.get_valueA() == 0)
        {
        test.set_valueA(atoi(buffer));
        }
      else
        {
        if (atoi(buffer) == 0)
          {
            test.set_valueA(test.get_valueA()*10);
          }
        else
          {
            test.set_valueA(test.get_valueA()+ atoi(buffer));
          }
         }
    }

    if (iTeller2 == 1)
    {
      cout << buffer << endl;
      if (test.get_valueB() == 0)
        {
        test.set_valueB(atoi(buffer));
        }
      else
        {
        if (atoi(buffer) == 0)
          {
            test.set_valueB(test.get_valueB()*10);
          }
        else
          {
            test.set_valueB(test.get_valueB()+ atoi(buffer));
          }
         }
    }
    if (iTeller2 == 2)
    {
      cout << buffer << endl;
      if (test.get_valueC() == 0)
        {
        test.set_valueC(atoi(buffer));
        }
      else
        {
        if (atoi(buffer) == 0)
          {
            test.set_valueC(test.get_valueC()*10);
          }
        else
          {
            test.set_valueC(test.get_valueC()+ atoi(buffer));
          }
         }
    }
    if (iTeller == 3)
    {
     strcat(END,buffer);
       if (END== "END")
       {
         break;
       }
    }
    }
    iTeller++;
  }
  file.close();
  cout << "the complete file is in a buffer" << endl;
 // cout << test.get_valueA() << endl;
 // cout << test.get_valueB() << endl;
 // cout << test.get_valueC() << endl;
//  delete[] buffer;
};


int main () {
  CTriangle rect;
  get_from_file(rect);
  cout << "Zijde A: " <<  rect.get_valueA() << endl;
  cout << "Zijde B: " <<  rect.get_valueB() << endl;
  cout << "Zijde C: " <<  rect.get_valueC() << endl;

  int iTemp;  cin >> iTemp;
}

[marcusk]

ik heb de code niet helemaal doorgelezen, maar denk dat je iig Triangle::a, b en c moet initializeren op 0 in je constructor.

[Tomatrix]

Kan je niet gewoon met fscanf rechtstreeks een int inlezen?

[GAEvakYD]

WOWWWWWWWWWW

Kerel jij bent mijn held.


Zo'n klein foutje, heb ze op nul gezet in de constructor en gelijk draaien.

Pffff wat dom van me

Thanks man

[Verwijderd]

volgens mij kan het veeeeeeeeel makelijker, aangezien je toch een istream gebruikt :-)

Je moet eventuele fouten maar negeren wat ik type dit uit m'n hoofd in

class CTriangle
{
friend istream& operator>>( istream& is, CTriangle& triangle);

private:
int a;
int b;
int c;
}

friend istream& operator>>( istream& is, CTriangle& triangle)
{
is >> triangle.a;
is >> triangle.b;
is >> triangle.c;

return is;
}


void main()
{
ifstream if( "triangle.txt")

CTriangle triangle;

if >> triangle;
}


Eventueel zou je nog een try/catch block kunnen gebruiken om een eventuele fout af te vangen (end of file etc..).

In je orginele code zit nog een memory leak trouwens

Succes met je opdracht verder.

[GAEvakYD]

Eens kijken of ik het op jou manier kan krijgen