Сравнение строк со структурным элементом

Question

Сравнение строк со структурным элементом

0

У меня есть такой код в файле.c, который определяет, является ли tPerson.name равным одному из элементов const char * names [COUNT] или нет:

define COUNT 3
...
typedef struct {
int age;
char *name;
} tPerson;

const char* names[COUNT] = {
    "xxx", "yyy", "zzz"
};
....
char string[128];
strcpy(string, tPerson.name);//tPerson.name is already initizialed
int counter = 0;
while (counter != COUNT) {
    if (strcmp(names[counter], string) == 0) {
        counter++;
        return 0;
    }
}
...

Все необходимые библиотеки включены. Компилятор не обнаруживает никаких ошибок или предупреждений, но программа не работает так, как должна - ничего не делает после выполнения. Этот фрагмент кода является лишь частью огромной программы, поэтому я хотел бы знать, правильна ли эта конструкция, а где-то еще в программе ошибка или нет. благодаря

wrz90 16 нояб. 2014, в 18:30

Источник

0

Какой смысл counter++ в вашем цикле после оператора return?
Neil Kirk 16 нояб. 2014, в 17:28
0

это уже исправлено, во всяком случае это не было главной проблемой .. до сих пор не работает
wrz90 16 нояб. 2014, в 17:30
0

Затем обновите свой ответ, чтобы другие не запутались.
Neil Kirk 16 нояб. 2014, в 17:31
2

Ну и какой смысл counter++ внутри оператора if вообще? Он должен быть снаружи, потому что либо strcmp возвращает 0 а вы возвращаете 0 либо вы продолжаете сравнивать одни и те же две строки вечно, поскольку counter не изменяется.
Stefan Neubert 16 нояб. 2014, в 17:32
0

да, ты прав, просто глупая ошибка - счетчик внутри, если вместо того, чтобы .. спасибо
wrz90 16 нояб. 2014, в 17:39

Показать ещё 3 комментария

Теги:

c++

c

2 ответа

Ещё вопросы

Какой смысл counter++ в вашем цикле после оператора return?
это уже исправлено, во всяком случае это не было главной проблемой .. до сих пор не работает
Затем обновите свой ответ, чтобы другие не запутались.
Ну и какой смысл counter++ внутри оператора if вообще? Он должен быть снаружи, потому что либо strcmp возвращает 0 а вы возвращаете 0 либо вы продолжаете сравнивать одни и те же две строки вечно, поскольку counter не изменяется.
да, ты прав, просто глупая ошибка - счетчик внутри, если вместо того, чтобы .. спасибо

P.P. · Answer 1 · 2014-11-16T14-57-00.000Z

Вы хотите продолжить цикл, если нет совпадения. Поставьте выражение counter++; вне оператора if:

while (counter != COUNT) {
    if (strcmp(names[counter], string) == 0) {
        return 0;
    }
    counter++;
}

И используйте size_t для counter вместо int: size_t counter = 0;

Morad · Answer 2 · 2014-11-16T15-22-00.000Z

У вас есть return 0 перед увеличением счетчика

if (strcmp(names[counter], string) == 0) {
    return 0;
    counter++;
}

да спасибо, ИСПРАВЛЕНО - но все еще не работает