vlladimir
1.
2.
3.
struct node *n, ...;
...
n = (node*)malloc(sizeof(node*));
Вы запрашиваете блок памяти размером с указатель, а надо запрашивать блок памяти размером с node:
1.
n = (node*)malloc(sizeof(node));
vlladimir
1.
2.
3.
4.
5.
6.
7.
8.
9.
10.
int poscinstr(char *str, char c)
{
unsigned int i;
for (i = 0; i < strlen(str); i++){
if (str == c){
return i;
}
}
return -1;
}
Так лучше не писать, потому что функция strlen вызывается на [i]каждой итерации цикла. Вообще, для поиска char-а в ноль-завершаемой строке есть стандартная функция
strchr :
1.
2.
3.
4.
5.
6.
7.
#include <string.h>
int find_char(const char s[], char c) {
const char* p;
p = strchr(s, c);
return (p != NULL ? (int)(p - s) : -1);
}
vlladimir
1.
2.
3.
4.
5.
6.
7.
8.
node *a[256];
...
unsigned int j;
...
for (j = 0; j < 256; j++){
a[j] = (node*)malloc(sizeof(node*));
a[j] = NULL;
}
Отмеченная строка не нужна.
vlladimir
1.
estr = fgets(str, 2000, fi);
Все современные компьютеры оснащены гигабайтами оперативной памяти, поэтому при работе с текстовыми файлами в большинстве случаев нет смысла заморачиваться на чтение файла по кусочкам. Проще загрузить весь файл в один буфер, и работать с ним как с большой строкой, например так:
1.
2.
3.
4.
5.
6.
7.
8.
9.
10.
11.
12.
13.
14.
15.
16.
17.
18.
19.
20.
21.
22.
23.
24.
25.
26.
27.
28.
29.
30.
31.
32.
33.
34.
35.
36.
37.
38.
39.
40.
41.
42.
43.
44.
45.
46.
47.
48.
49.
50.
51.
52.
53.
54.
55.
56.
57.
58.
59.
60.
61.
62.
63.
64.
65.
66.
67.
#include <stdio.h>
#include <stdlib.h>
#include <stdbool.h>
char* load_text_file(const char path[]) {
FILE* file;
size_t size;
char* text;
/* 'b' чтобы не тратить время на замену "\r\n" (CR LF) -> "\n" (LF) */
file = fopen(path, "rb");
/* узнаём размер файла */
fseek(file, /*offset:*/0, SEEK_END);
size = ftell(file);
fseek(file, /*offset:*/0, SEEK_SET);
text = (char*)malloc(size + 1);
fread(text, 1, size, file);
text[size] = '\0';
fclose(file);
return text;
}
bool is_letter(char c);
void handle_word(const char word[], size_t word_len);
void load_words(const char file_path[]) {
char* text;
size_t pos, word_pos;
text = load_text_file(file_path);
pos = 0;
for (;;) {
/* пропускаем не-буквы */
for (;;) {
if (is_letter(text[pos]) || (text[pos] == '\0'))
break;
pos++;
}
if (text[pos] == '\0')
break;
/* сканируем слово */
word_pos = pos;
pos++; /* пропускаем первую букву */
while (is_letter(text[pos]))
pos++;
handle_word(text + word_pos, /*word_len:*/ pos - word_pos);
}
free(text);
}
bool is_letter(char c) {
return (('a' <= c) && (c <= 'z')) || (('A' <= c) && (c <= 'Z'));
}
void handle_word(const char word[], size_t word_len) {
/* строка word не завершается нулевым char-ом ('\0'), поэтому явно указываем длину */
printf("%.*s\n", (int)word_len, word);
}
Обработку ошибок не писал, чтобы код был проще. Но по-хорошему, ошибки надо обрабатывать. Есть два способа: коды возврата и исключения (
setjmp /
longjmp , или тупо
exit ). Какой способ лучше -- holy war.
egorychvlladimirНет просто вылетает и всене удивительно
1.
2.
3.
4.
5.
6.
7.
8.
9.
10.
11.
void prntable(struct node *a[255]){
int i;
node *p;
for (i = 0; i < 256; i++){
p = a[i];
while (p){
printf("%s %d\n", p->word, p->count);
p = p->next;
}
}
}
выход за пределы массива всегда заканчиваются не приятноА там как раз массив длины 256:
vlladimir
1.
2.
3.
void main(int argc, char *argv[]) {
node *a[256];
поэтому с индексом всё нормально. А то, что в заголовках функций написано «struct node *a[255]», значения не имеет, потому что длина массива реально не передаётся. Но лучше конечно поправить, а то сбивает с толку.
