Läsa textfiler och starta program

Så här brukar jag läsa filer:

FILE * pFile = NULL;

pFile = fopen("fil.txt", "rb");

if(pFile == NULL)
  // Kunde inte öppnas

char str[255];
fread(str, sizeof(char) * 255, pFile);

if (ferror( pFile ))
// Fel i läsningen av filen

fclose(pFile);

Nu var ju det där mest för läsning av binära filer. Har för mig att det ska finnas en funktion som heter fget() också som läser rader.

Ok, får väl testa det. Fast jag är ju hos min kompis com lånat min c++ bok nu så jag får väl ta och läsa igen den också.

Har du över huvud taget läst felmeddelandet? :)
"initialization of 'fin' is skipped by 'case' label"

Det antyder att det har något med switch/case att göra. Prova att lägga koden i ett eget block {}.

Ja, jag har läst felmeddelandet men förståd inte så mcyket, men jag ska testa det du sa.

För det mesta lämnar kompilatorer tips om vad som är fel. Hittar man inte felet ändå är debuggern ens bästa vän :)

Har tyvärr aldrig använt degubern särskilt mycket, har svårt att förstå mig på allting som visas. :(

Pröva att sätta "ifstream fin;" före switch-satsen. Fast jag tycker att varningen/felet låter misstänkt likt en kompilatorbugg. Vilken kompilator, och vilka felrättningar har du installerat?

Har tyvärr aldrig använt degubern särskilt mycket, har svårt att förstå mig på allting som visas.

Nej jag säger inte att det är enkelt, men ett tips är att åtminstone försöka sätta sig in i vad som sker och varför det sker. Då blir debuggern din bästa vän :)

Visual c++ 6.0 använder jag, felrättningar vet jag inte vad det är riktigt, skulle du kunna förklara mer. Det börjar i alla fall fungera lite nu, i alla fall så länge filen är tom om den inte är det så får jag ett minnesfel!

felet:
instruktionen på "0x77d38c2c" referade till "0x00000073".
The memory could not be read

Ser ut som att du petar i minne som du inte ska peta i, men att felet ligger i hur funktioner ska användas inte i hur du själv pekar i minnet. Vid avlusning av minnesfel är debuggern verkligen ens bästa vän då den steg för steg avslöjar vad varje processorinstruktion gör, men i ditt fall är kanske koden såpass liten att man ändå kan klura ut vad som går snett. Du skriver att om filen är tom fungerar det... kan det tyda på att det du läst in och vill skicka vidare inte är det du tror det är? Om filen är tom körs ju inte whileloopen... ;)

PeW skrev:

Har tyvärr aldrig använt degubern särskilt mycket, har svårt att förstå mig på allting som visas.

Nej jag säger inte att det är enkelt, men ett tips är att åtminstone försöka sätta sig in i vad som sker och varför det sker. Då blir debuggern din bästa vän :)

Jag följde ditt råd pew och satte ut lite breakpoints och fick ett nytt meddelandet. Och allting i fin assembler(?).

Bifogar en bild också.

bild

Edit: ser precis nu att min c++ bok ligger precis ijämte mig, får nog ta och öppna den.

Edit 2: Nu vet jag på vilken rad felet är, men jag vet inte hur jag ska lösa det.

ifstream fin("server.cfg");
char ch;
while (fin.get(ch))
SendDlgItemMessage(hwnd, IDC_SERVER, LB_ADDSTRING, 0, (LPARAM)ch);(felet är här)
fin.close();

Jag följde ditt råd pew och satte ut lite breakpoints och fick ett nytt meddelandet. Och allting i fin assembler(?).

Bifogar en bild också.

bild

Edit: ser precis nu att min c++ bok ligger precis ijämte mig, får nog ta och öppna den.

Jo, det är i x86 assembler. Men bara att titta på ett stycke leder sällan någon vart då det är ett antal rader... ;) Som sagt så tror jag att felet är att du läser in ett tecken, men av funktionen du anropar tolkar det tecknets värde (ascii för 73?) som en minnesadress och då blir det knas. Inte säker på detta, men om det är så, blir det naturligtvis en krasch.

Lite god litteratur är aldrig fel :)

Hur gör jag att för att läsa in varje rad istället ?
Knaske har nogåt med det att göra men jag är inte säker.
För om jag tar bort

SendDlgItemMessage(hwnd, IDC_SERVER, LB_ADDSTRING, 0, (LPARAM)ch);(felet är här)

så funkar ju det, är det någon som vet om det finns ngot annat sätt att lägga till nya rader i en listbox?

Chainsaw skrev:

Pröva att sätta "ifstream fin;" före switch-satsen.

Ja, det är också en lösning. Antingen kan man deklarera variabeln utanför switch-satsen eller innanför om man lägger den innanför ett klammerblock {}.