Почему условный оператор даже, если условие неверно, выдает истину?

@Outmee · Регистрация: 26.01.2014

Студворк — интернет-сервис помощи студентам

C++
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
    void change()
    {
        int tmp = 0;
        do
        {
            tmp = 0;
            cout << "Napishite Nazvanie, Regisera filma" << endl;
            char chName[20];
            char chRegiser[20];
            gets(chName);
            cout << ' ';
            gets(chRegiser);
 
            for (int i = 0; i < cntfilms; i++)
            {
                if (strcmp(warehouse[i].sNazvanie, chName) && strcmp(warehouse[i].sReziser, chRegiser))
                {
                    cout << "Zapishite Novie dannie fil'ma" << endl;
                    cout << "\nВведите название фильма: ";
                    cin.ignore();
                    gets(warehouse[i].sNazvanie);
                    cout << "\nВведите режисера фильма: ";
                    gets(warehouse[i].sReziser);
                    cout << "\nВведите жанр фильма: ";
                    gets(warehouse[i].sZanr);
                    cout << "\nВведите рейтинг фильма: ";
                    cin >> warehouse[i].sReiting;
                    cout << "\nВведите цену за один ЦД фильма: ";
                    cin >> warehouse[i].sChena;
                    cout << "\nВведите количество ЦД фильма: ";
                    cin >> warehouse[i].count;
                    system("cls");
                    cout << "УСПЕХ" << endl;
                    cout << "Возможно вы хотите изменить еще что-то?"
                        << endl << "Да я хочу изменить еще что-то - 1; Нет, спасибо, все что нужно я сделал - 0." << endl;
                }
                else
                {
                    cout << "Нет записи с таким НАЗВАНИЕМ и РЕЖИСЕРОМ, может быть стоит повторить поиск?"
                        << endl << "Повторить - 1, Отмена - 0." << endl;
                }
            }
            cout <<"Заметка : Функция void change() работает ОТЛИЧНО" << endl;
            cin >> tmp;
        } while (tmp != 0);
    }

Что бы я не вводил, строки сравниваются и условие всегда верное. Ах и да сравниваю я с данными из структуры

@Kuzia domovenok · 20.05.2014, 02:32

strcmp возвращает 0 если строки равны. Ноль в условии if означает false.
И наоборот, если строки не равны, strcmp возвращает ненулевое значение и условие if выполняется (true).
более того, непонятно какие строки ты сравниваешь в if, если ты их ещё не вводил с клавиатуры, а только собираешься сделать это после if

@Outmee · 20.05.2014, 20:12 **[ТС]**

C++
1
2
3
gets(chName);
            cout << ' ';
            gets(chRegiser);

Вот же вроде как ввод из клавиатуры или я вообще крабокодер.

@Kuzia domovenok · 21.05.2014, 01:19

Outmee, ты сравниваешь 2 строки!
chRegiser вводится с клавиатуры,
А откуда вводится то, с чем она сравнивается: warehouse[i].sReziser?
Ну и главное, как я уже сказал, strcmp надо сравнивать с нулём, а не просто так использовать в условии иф.

@Outmee · 21.05.2014, 02:12 **[ТС]**

Kuzia domovenok, это структура, с ранее введенной информацией и как я понимаю она должна заимствоваться от туда. Если вам удобнее, могу скинуть весь код программы.

Добавлено через 5 минут

C++
1
if (strcmp(warehouse[i].sNazvanie, chName) == 0 && strcmp(warehouse[i].sReziser, chRegiser) == 0)

Верю что так правильно, ибо работает, благодарю вас Kuzia domovenok

@moomot · 21.05.2014, 02:13

C
1
strcmp(warehouse[i].sNazvanie, chName)

Ты не указываешь условие, первая строка больше второй, или вторая больше первой, или они равны вообще.

C
1
strcmp(warehouse[i].sNazvanie, chName) == 0

C
1
strcmp(warehouse[i].sNazvanie, chName) == -1

C
1
strcmp(warehouse[i].sNazvanie, chName) == 1

И еще, после gets желательно использовать

C++
1
fflush(stdin);

@Outmee · 21.05.2014, 12:45 **[ТС]**

C++
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
void change()
    {
        int tmp;
        do
        {
            tmp = 0;
            cout << "Napishite Nazvanie, Regisera filma" << endl;
            char chName[20];
            char chRegiser[20];
            gets(chName);
            fflush(stdin);
            gets(chRegiser);
            fflush(stdin);
 
 
            for (int i = 0; i < cntfilms; i++)
            {
                
                if (strstr(warehouse[i].sNazvanie, chName) && strstr(warehouse[i].sReziser, chRegiser))
                    {
                        cout << "Zapishite Novie dannie fil'ma" << endl;
                        cout << "\nВведите название фильма: ";
                        
                        gets(warehouse[i].sNazvanie);
                        cout << "\nВведите режисера фильма: ";
                        gets(warehouse[i].sReziser);
                        cout << "\nВведите жанр фильма: ";
                        gets(warehouse[i].sZanr);
                        cout << "\nВведите рейтинг фильма: ";
                        cin >> warehouse[i].sReiting;
                        cout << "\nВведите цену за один ЦД фильма: ";
                        cin >> warehouse[i].sChena;
                        cout << "\nВведите количество ЦД фильма: ";
                        cin >> warehouse[i].count;
                        system("cls");
                        cout << "УСПЕХ" << endl;
                        cout << "Возможно вы хотите изменить еще что-то?"
                            << endl << "Да я хочу изменить еще что-то - 1; Нет, спасибо, все что нужно я сделал - 0." << endl;
                        ;
                    }
                } 
 
            cin >> tmp;
 
        } while (tmp == 1);
    }

Сделал с горем пополам всем спасибо кто подсказал, буду рад советам по улучшению кода и да как видите я использовал не strcmp a strstr в первом случае ничего не работало во втором все нормально, объясните че это черевато?

@moomot · 21.05.2014, 13:49

strstr ищет первое вхождение подстроки в строку. У вас может быть к примеру первая строка "abcdef", вторая строка "abc" и результат будет получен true.

@Outmee · 21.05.2014, 13:54 **[ТС]**

moomot, в принципе это меня более чем устраивает на сдаче скажу что у меня, интеллектуальный подбор вариантов))

Новые блоги и статьи Все статьи Все блоги /
Символьное дифференцирование igorrr37 13.02.2026 / * Логарифм записывается как: (x-2)log(x^2+2) - означает логарифм (x^2+2) по основанию (x-2). Унарный минус обозначается как ! */ #include <iostream> #include <stack> #include <cctype>. . .	Камера Toupcam IUA500KMA Eddy_Em 12.02.2026 Т. к. у всяких "хикроботов" слишком уж мелкий пиксель, для подсмотра в ESPriF они вообще плохо годятся: уже 14 величину можно рассмотреть еле-еле лишь на экспозициях под 3 секунды (а то и больше),. . .	И ясному Солнцу zbw 12.02.2026 И ясному Солнцу, и светлой Луне. В мире покоя нет и люди не могут жить в тишине. А жить им немного лет.	«Знание-Сила» zbw 12.02.2026 «Знание-Сила» «Время-Деньги» «Деньги -Пуля»
SDL3 для Web (WebAssembly): Подключение Box2D v3, физика и отрисовка коллайдеров 8Observer8 12.02.2026 Содержание блога Box2D - это библиотека для 2D физики для анимаций и игр. С её помощью можно определять были ли коллизии между конкретными объектами и вызывать обработчики событий столкновения. . . .	SDL3 для Web (WebAssembly): Загрузка PNG с прозрачным фоном с помощью SDL_LoadPNG (без SDL3_image) 8Observer8 11.02.2026 Содержание блога Библиотека SDL3 содержит встроенные инструменты для базовой работы с изображениями - без использования библиотеки SDL3_image. Пошагово создадим проект для загрузки изображения. . .	SDL3 для Web (WebAssembly): Загрузка PNG с прозрачным фоном с помощью SDL3_image 8Observer8 10.02.2026 Содержание блога Библиотека SDL3_image содержит инструменты для расширенной работы с изображениями. Пошагово создадим проект для загрузки изображения формата PNG с альфа-каналом (с прозрачным. . .	Установка Qt-версии Lazarus IDE в Debian Trixie Xfce volvo 10.02.2026 В общем, достали меня глюки IDE Лазаруса, собранной с использованием набора виджетов Gtk2 (конкретно: если набирать текст в редакторе и вызвать подсказку через Ctrl+Space, то после закрытия окошка. . .

@Kuzia domovenok 4268 / 3327 / 926 Регистрация: 25.03.2012 Сообщений: 12,532 Записей в блоге: 1
	20.05.2014, 02:32
	strcmp возвращает 0 если строки равны. Ноль в условии if означает false. И наоборот, если строки не равны, strcmp возвращает ненулевое значение и условие if выполняется (true). более того, непонятно какие строки ты сравниваешь в if, если ты их ещё не вводил с клавиатуры, а только собираешься сделать это после if 1

@Kuzia domovenok 4268 / 3327 / 926 Регистрация: 25.03.2012 Сообщений: 12,532 Записей в блоге: 1
	21.05.2014, 01:19
	Outmee, ты сравниваешь 2 строки! chRegiser вводится с клавиатуры, А откуда вводится то, с чем она сравнивается: warehouse[i].sReziser? Ну и главное, как я уже сказал, strcmp надо сравнивать с нулём, а не просто так использовать в условии иф. 1

@moomot 9 / 9 / 6 Регистрация: 12.07.2013 Сообщений: 57
	21.05.2014, 13:49
	strstr ищет первое вхождение подстроки в строку. У вас может быть к примеру первая строка "abcdef", вторая строка "abc" и результат будет получен true. 1

@Outmee 2 / 2 / 1 Регистрация: 26.01.2014 Сообщений: 59
	21.05.2014, 13:54 [ТС]
	moomot, в принципе это меня более чем устраивает на сдаче скажу что у меня, интеллектуальный подбор вариантов)) 0