🚀 go-pugleaf

epsilon$ while read LINE; do echo \>"$LINE"; done < "michiuiu"
>	printf("%s\n",record[i].text);
[...]
>Ma ktoś jakiś pomysł co jest nie tak?

To.

-- 
d'`'`'`'`'`'`'`'`'`'`'`'`'Yb 
`b  Kruk@epsilon.eu.org   d' 
d' http://epsilon.eu.org/ Yb 
`b,-,.,-,.,-,.,-,.,-,.,-,.d' 

michiuiu wrote:
> 	while (fgets(line, 13, f)) {

Skoro masz 13-znakowe słowa to tutaj musisz mieć 14, a nie 13.

> 	    item = strtok(line, "\n");
> 	    strcpy(record[reccount].text, item);

Tutaj (zresztą nie tylko tutaj) wypadałoby sprawdzić co zwróci strtok().

>     }
> }

Jak funkcja main() ma typ int to przydałoby się zwrócić jakąś wartość, nie?

> Ma ktoś jakiś pomysł co jest nie tak?

Przede wszystkich doczytaj _do_końca_ dokumentację do wszystkich funkcji,
których używasz. Często diabeł tkwi w szczegółach. Poza tym, jak już
wspomniałem, sprawdzaj wartość zwracaną przez każdą funkcję, która coś
zwraca, jeśli z tego dalej korzystasz.

Dnia Wed, 09 Mar 2011 09:33:42 +0100, michiuiu napisał(a):

> No i gdy go odpalam, to się wywala:
> $ ./w
> kae8koh1Eisikae8koh1Eisi

> 	while (fgets(line, 13, f)) {

wczytalo sie: kae8koh1Eisi (plus zero na koncu)

> 	    printf("%s",line);
> 	    item = strtok(line, "\n");

\n nie został znaleziony i strtok zwraca null

> 	    strcpy(record[reccount].text, item);

strcpy uzywa tego null i wywala program

Poprawiony kod: http://pastebin.com/UkzJCe40

-- 
Tomek
http://www.ttmath.org

michiuiu wrote:

> Cześć
> 
> Próbuję napisać taki prosty program, który będzie wczytywał z pliku
> (powiedzmy 100k do 1mln linii) listę słów.
> Ma je pamiętać w jakieś tablicy 1d albo w strukturze, obojętnie.
> 
> Program wygląda narazie tak:
> 
> #include <stdio.h>
> #include <string.h>
> #include <stdlib.h>
> 
> typedef struct {
>     char text[14];
> } word;
> word record[100000];
> 
> int main() {
> 
>     FILE *f;
>     char line[14];
>     char *item;
>     int reccount = 0;
>     int i;
> 
>     if (f = fopen("plik1", "r")) {
> while (fgets(line, 13, f)) {

lepiej stosowac fgets(line,sizeof(line),f)

> printf("%s",line);

-> wywal to

> item = strtok(line, "\n");

nie wiem po co ci to?
-> wywal to

> strcpy(record[reccount].text, item);

tu chyba chcesz dolaczyc '\n'?
zalatwilo to fgets. poczytaj (man fgets)
a jesli chcesz cos sklejac to poczytaj o strcat.

-> wywal to

> printf("%s\n", record[reccount].text);
> reccount++;
> }
> fclose(f);
>     }
> 
>     for (i=0; i<reccount; i++) {
> printf("%s\n",record[i].text);
>     }
> }
> 
> A plik "plik1" tak:
> kae8koh1Eisi
> EkaiLa4ao8ci
> GaeRup9vevoo
> ojohwu7ooKai
> na9TohTaixei
> eiqu5VulioCh
> caechi0Aevoo
> ku5iu8ahYo6s
> nut1aa2Iawie
> qui8eeFaeth4
> iXooropi5rah
> eeHef1TooSei
> [...]
> 
> Tyle że ma np. 100 tysięcy linii.
> 
> No i gdy go odpalam, to się wywala:
> $ ./w
> kae8koh1Eisikae8koh1Eisi
> 
> Segmentation fault
> $
> 
> Ma ktoś jakiś pomysł co jest nie tak?
> I nie bijcie proszę, bo dopiero zaczynam naukę programowania :(
> 
> michiuiu.

pozdr.

On 9 Mar, 10:33, michiuiu <michi...@asdfasfda.asdf13er.pl> wrote:
> Cześć
>
> Próbuję napisać taki prosty program, który będzie wczytywał z pliku
> (powiedzmy 100k do 1mln linii) listę słów.
> Ma je pamiętać w jakieś tablicy 1d albo w strukturze, obojętnie.
>
> Program wygląda narazie tak:
>
> #include <stdio.h>
> #include <string.h>
> #include <stdlib.h>
>
> typedef struct {
>     char text[14];} word;
>
> word record[100000];
>
> int main() {
>
>     FILE *f;
>     char line[14];
>     char *item;
>     int reccount = 0;
>     int i;
>
>     if (f = fopen("plik1", "r")) {
>         while (fgets(line, 13, f)) {
>             printf("%s",line);
>             item = strtok(line, "\n");
>             strcpy(record[reccount].text, item);
>             printf("%s\n", record[reccount].text);
>             reccount++;
>         }
>         fclose(f);
>     }
>
>     for (i=0; i<reccount; i++) {
>         printf("%s\n",record[i].text);
>     }
>
> }
>
> A plik "plik1" tak:
> kae8koh1Eisi
> EkaiLa4ao8ci
> GaeRup9vevoo
> ojohwu7ooKai
> na9TohTaixei
> eiqu5VulioCh
> caechi0Aevoo
> ku5iu8ahYo6s
> nut1aa2Iawie
> qui8eeFaeth4
> iXooropi5rah
> eeHef1TooSei
> [...]
>
> Tyle że ma np. 100 tysięcy linii.
>
> No i gdy go odpalam, to się wywala:
> $ ./w
> kae8koh1Eisikae8koh1Eisi
>
> Segmentation fault
> $
>
> Ma ktoś jakiś pomysł co jest nie tak?
> I nie bijcie proszę, bo dopiero zaczynam naukę programowania :(
>
> michiuiu.

przekroczyłeś rozmiar stosu

> word record[100000];

zapisz to w postaci wskaźnika i powinno być ok

Pozdrawiam
Dariusz