void Tf::mo(char m[30])
{
char fname[30] = "text.txt\0";
FILE *in;
if((in = fopen(fname, "rt")) == NULL)
{
return;
}
char name[30];
while(!feof(in)){
fscanf(in, "%s %d %d", &name, &yd, &la);
if(m == name){ fclose(in); return; };
}
fclose(in);
}

Нужно чтоб он сравнил m c name и вышел из функции
а он не сравнивает!
так и продолжает цыкл до конца файла.
Как сделать чтоб он смог найти нужное и выйти,?

m и name у тебя объявлены как массивы char, поэтому поэтому выражение вида m означат ссылку на первый элемент массива. Поэтому сравниваются не строки, а значения ссылок.

Дополнение к предидушему
if(m == name)
нужно заменить на
if(strcmp(m, name) == 0 )

Кроме того, так для общего развития, строка в кавычках уже подрузумевает ноль в конце и если содержимое массива fname - константа, то можно и даже нужно писать fname[] = "text.txt", а если не константа, то сначала описывают массив, а потом копируют в него значения (char fname[30]; strcpy(fname, "text.txt");)
return не надо заключать в {}.

Цитата

константа, то можно и даже нужно писать fname[] = "text.txt",

Цитата

return не надо заключать в {}.

Это стиль программирования, а не обязательное требование. К вопросу не относится

char fname[30] = "text.txt\0";
Это не стиль программирования а излишний код ноль действительно не нужен при использовании кавычек
char fname[30] = "text.txt";
и почему 30? от балды?
если размер не меняется то как сказал RAN лучше
char fname[] = "text.txt";
или потом возможно содержимое fname поменяется? тогда лучше
char fname[MAX_PATH] = "text.txt";
(лишние символы автоматом инициилизируются нулями)

а стиль
не очень то красиво...
где вы тут стиль увидели...
программа то после исправления будет работать но стиля оформления тут и близко нет...

одна команда пишется в три строки
if((in = fopen(fname, "rt")) == NULL)
{
return;
}
а две команды в одну
if(m == name){ fclose(in); return; };
да еще и ; после фигурных скобок
лучше бы эти два куска оформить наоборот

Я все это говорю не к тому что хочу человека обидеть...
сам и учу и учусь а к тому что стиль оформления становится привычкой и если на него не обращать внимания то со временем себя будет трудно переламывать...
(к примеру у меня почти полное отсутствие запятых)

Цитата

char fname[MAX_PATH] = "text.txt"; (лишние символы автоматом инициилизируются нулями)

Это в начале, а в процессе перезаписи в этот буфер более короткой строки уже не все

Цитата

char fname[30] = "text.txt"; и почему 30? от балды?

Потому что, возможно в дальнейшем требуется буфер именно такого размера Гипотетически предполагая, мы же не знаем что дальше в коде происжодит?

Цитата

или потом возможно содержимое fname поменяется? тогда лучше char fname[MAX_PATH] = "text.txt";

Ай-ай а ведь только что товарища попрекал:

Цитата

Это не стиль программирования а излишний код

Это сообщение отредактировал(а) DENNN - 19.5.2003, 20:54

Другими словами, рассуждение о стиле - это больше во флейм, а если уж попрекать другого, то хотя бы себе при этом не противоречить.
Без обид. Ничего лишнего. Просто человек конкретный вопрос задал, а вы ему уже все кости перемыли

Поддержу HexoGenus. Все взаимосвязанно: если у человека в коде бардак, то это показывает небрежный подход к написанию программы, а это уже весьма существенное качество.

Ни чего страшного, пару раз на грабли наступит типа:

Цитата

if(m == name){ fclose(in); return; };

... в этой строчке нельзя поставить BreakPoint на выполнение условия.

И все встанет на свои места.