Особенности while (pch != NULL)

@russian_pepper · Регистрация: 14.06.2014

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

Здравствуйте!
Подскажите, почему пример ниже завершается ошибкой? Ведь если pch == NULL, то запись в вектор не должна производиться?
В массиве char str[] есть непечатные символы, которые тут не отображаются, но как я понимаю, проблема не в этом.
Мне нужно записать элементы массива в структуру, на свои места, но у меня пока не получатся даже запись в вектор.

C++

#define _CRT_SECURE_NO_WARNINGS
#include <iostream>
#include <string>
#include <cstdlib>
#include <stdio.h>
#include <vector>
 
using namespace std;
 
struct glass//структура под данные из стаканов
{
    vector <float> BID_PRICE;
    vector <unsigned int> BID_QUANTITY;//спрос цена увеличивается от первого значения
    vector <float> OFFER_PRICE;
    vector <unsigned int> OFFER_QUANTITY;//предложение цена увеличивается от первого значения
};
 
struct glass STAKAN;
//vector<glass> all_STAKANS;
vector<string> temp_vector;
 
void main()
{
    const char separator[] = "=\01\02\03\04"; //Символы-разделители строки
    char str[] = "BID_COUNT=20.000000OFFER_COUNT=20.000000BID=PRICE=139.820000QUANTITY=1062.000000PRICE=139.830000QUANTITY=2.000000PRICE=139.850000QUANTITY=9.000000PRICE=139.870000QUANTITY=1.000000PRICE=139.880000QUANTITY=1.000000PRICE=139.900000QUANTITY=54.000000PRICE=139.910000QUANTITY=1.000000PRICE=139.920000QUANTITY=1.000000PRICE=139.930000QUANTITY=4.000000PRICE=139.940000QUANTITY=1.000000PRICE=139.970000QUANTITY=1.000000PRICE=139.980000QUANTITY=1.000000PRICE=140.000000QUANTITY=17.000000PRICE=140.010000QUANTITY=301.000000PRICE=140.020000QUANTITY=400.000000PRICE=140.040000QUANTITY=2.000000PRICE=140.050000QUANTITY=1.000000PRICE=140.060000QUANTITY=320.000000PRICE=140.070000QUANTITY=1.000000PRICE=140.100000QUANTITY=2787.000000OFFER=PRICE=140.120000QUANTITY=23.000000PRICE=140.150000QUANTITY=7.000000PRICE=140.200000QUANTITY=19.000000PRICE=140.260000QUANTITY=106.000000PRICE=140.290000QUANTITY=20.000000PRICE=140.300000QUANTITY=100.000000PRICE=140.310000QUANTITY=400.000000PRICE=140.320000QUANTITY=20.000000PRICE=140.350000QUANTITY=50.000000PRICE=140.400000QUANTITY=20.000000PRICE=140.420000QUANTITY=400.000000PRICE=140.480000QUANTITY=150.000000PRICE=140.500000QUANTITY=40.000000PRICE=140.520000QUANTITY=440.000000PRICE=140.550000QUANTITY=270.000000PRICE=140.580000QUANTITY=3.000000PRICE=140.670000QUANTITY=10.000000PRICE=140.680000QUANTITY=10.000000PRICE=140.700000QUANTITY=10.000000PRICE=140.720000QUANTITY=80.000000";
    char* pch;
    printf("Splitting string \"%s\" into tokens:\n", str);
    pch = strtok(str, separator);
    while (pch != NULL)
    {
        printf("%s\n", pch);
        pch = strtok(NULL, separator);
        temp_vector.push_back(pch);
        //if (pch != "BID_COUNT")// || pch == "OFFER_COUNT")
        //{
        //}
    }
}

@russian_pepper · 23.09.2015, 01:21 **[ТС]**

Не правильно выразился выше, запись в вектор получается, но завершается ошибкой почему-то.

Croessmah · 23.09.2015, 01:23

C++

1
2
3

pch = strtok(NULL, separator);
if ( pch != 0 )
    temp_vector.push_back(pch);

@russian_pepper · 23.09.2015, 02:05 **[ТС]**

Croessmah, Спасибо!
Да действительно, так всё работает, без ошибок. Но я не могу понять, почему выполнение программы доходит до temp_vector.push_back(pch);
через этот оператор
while (pch != NULL)
почему не прекращается выполнение цикла по условию pch != NULL?

Ведь оператор
if ( pch != 0 )
повторяет условие которое записано в операторе цикла...

Croessmah · 23.09.2015, 02:12

Сообщение от russian_pepper

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

C++

    while (pch != NULL)//Не ноль, тогда выполняем цикл
    {
        pch = strtok(NULL, separator);//Если здесь strtok вернул ноль, то
        temp_vector.push_back(pch);//здесь будет ошибка
        //а уж если дошли до сюда, то возвращаемся к началу цикла и проверяем условие.
    }

Evg · 23.09.2015, 09:58

Подозреваю, что всего лишь перепутаны местами строки 32 и 33 в программе из первого поста

@russian_pepper · 23.09.2015, 15:32 **[ТС]**

Evg, Да, действительно, так красивее и логичнее и правильнее, но теперь
не могу понять, почему в таком варианте программы, в консоль, надпись "pch = PRICE" не выводится ни разу, хотя значение "PRICE" присваивается pch и даже записывается в вектор temp_vector. Не подскажете, почему так?

C++

#define _CRT_SECURE_NO_WARNINGS
#include <iostream>
#include <string>
#include <cstdlib>
#include <stdio.h>
#include <vector>
 
using namespace std;
 
struct glass//структура под данные из стаканов
{
    vector <string> BID_PRICE;
    vector <unsigned int> BID_QUANTITY;//спрос цена увеличивается от первого значения
    vector <float> OFFER_PRICE;
    vector <unsigned int> OFFER_QUANTITY;//предложение цена увеличивается от первого значения
};
 
struct glass STAKAN;
//vector<glass> all_STAKANS;
vector<string> temp_vector;
 
void main()
{
    const char separator[] = "=\01\02\03\04"; //Символы-разделители строки
    char str[] = "BID_COUNT=20.000000OFFER_COUNT=20.000000BID=PRICE=139.820000QUANTITY=1062.000000PRICE=139.830000QUANTITY=2.000000PRICE=139.850000QUANTITY=9.000000PRICE=139.870000QUANTITY=1.000000PRICE=139.880000QUANTITY=1.000000PRICE=139.900000QUANTITY=54.000000PRICE=139.910000QUANTITY=1.000000PRICE=139.920000QUANTITY=1.000000PRICE=139.930000QUANTITY=4.000000PRICE=139.940000QUANTITY=1.000000PRICE=139.970000QUANTITY=1.000000PRICE=139.980000QUANTITY=1.000000PRICE=140.000000QUANTITY=17.000000PRICE=140.010000QUANTITY=301.000000PRICE=140.020000QUANTITY=400.000000PRICE=140.040000QUANTITY=2.000000PRICE=140.050000QUANTITY=1.000000PRICE=140.060000QUANTITY=320.000000PRICE=140.070000QUANTITY=1.000000PRICE=140.100000QUANTITY=2787.000000OFFER=PRICE=140.120000QUANTITY=23.000000PRICE=140.150000QUANTITY=7.000000PRICE=140.200000QUANTITY=19.000000PRICE=140.260000QUANTITY=106.000000PRICE=140.290000QUANTITY=20.000000PRICE=140.300000QUANTITY=100.000000PRICE=140.310000QUANTITY=400.000000PRICE=140.320000QUANTITY=20.000000PRICE=140.350000QUANTITY=50.000000PRICE=140.400000QUANTITY=20.000000PRICE=140.420000QUANTITY=400.000000PRICE=140.480000QUANTITY=150.000000PRICE=140.500000QUANTITY=40.000000PRICE=140.520000QUANTITY=440.000000PRICE=140.550000QUANTITY=270.000000PRICE=140.580000QUANTITY=3.000000PRICE=140.670000QUANTITY=10.000000PRICE=140.680000QUANTITY=10.000000PRICE=140.700000QUANTITY=10.000000PRICE=140.720000QUANTITY=80.000000";
    char* pch;
    printf("Splitting string \"%s\" into tokens:\n", str);
    pch = strtok(str, separator);
    while (pch != NULL)
    {
        printf("%s\n", pch);
        temp_vector.push_back(pch);
        pch = strtok(NULL, separator);
        if (pch == "PRICE")
        {
            cout << "pch = PRICE" << endl;
        }
    }
}

Здесь есть управляющие символы, которые невидны, по ссылке ниже исходный cpp файл

https://drive.google.com/open?... 1lfblZNOFk

Evg · 23.09.2015, 15:43

Сообщение от russian_pepper

не могу понять, почему в таком варианте программы, в консоль, надпись "pch = PRICE" не выводится ни разу, хотя значение "PRICE" присваивается pch и даже записывается в вектор temp_vector.

Ну ответ же очевиден и точно такой же, как и в прошлый раз. Между "хотя значение PRICE присваивается pch и даже записывается в вектор temp_vector" и "if (pch == PRICE)" стоит модификация переменной pch. Т.е. твой код эквивалентен:

C

int x, y;
x = 1;
printf ("%d\n", x); /* увидели печать, что x=1 */
x = 10;
y = x; /* почему-то основываясь на печати делаем вывод, что y=1 */

Добавлено через 2 минуты
Ну и к слову. Почти все интерфейсы поиска спроектированы так, чтобы пользоваться ими в цикле for:

C

for (pch = strtok(str, separator); pch != NULL; pch = strtok(NULL, separator))
{
  ...
}

При таком раскладе ты хотя бы не будешь наступать на эти же самые грабли (модификация pch должна быть в самом конце тела цикла, а у тебя опять посередине)

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

C

if (pch == "PRICE")

В языке Си строки надо сравнивать через strcmp. Хоть ты и используешь Си++, но строки у тебя в виде char*, а потому нужно использовать strcmp

@russian_pepper · 23.09.2015, 15:47 **[ТС]**

Evg, Да, действительно, я поторопился, но так тоже не работает, как ожидал:

C++

        printf("%s\n", pch);
        temp_vector.push_back(pch);
        
        if (pch == "PRICE")
        {
            cout << "pch = PRICE" << endl;
        }
        pch = strtok(NULL, separator);
    }

Croessmah · 23.09.2015, 16:28

У Вас сравниваются указатели. Используйте strcmp, например

C++

if ( !strcmp(pch,"PRICE") )
{
//...
}

@russian_pepper 44 / 1 / 0 Регистрация: 14.06.2014 Сообщений: 150
	23.09.2015, 01:21 [ТС]	2
	Не правильно выразился выше, запись в вектор получается, но завершается ошибкой почему-то. 0

@russian_pepper 44 / 1 / 0 Регистрация: 14.06.2014 Сообщений: 150
	23.09.2015, 02:05 [ТС]	4
	Croessmah, Спасибо! Да действительно, так всё работает, без ошибок. Но я не могу понять, почему выполнение программы доходит до temp_vector.push_back(pch); через этот оператор while (pch != NULL) почему не прекращается выполнение цикла по условию pch != NULL? Ведь оператор if ( pch != 0 ) повторяет условие которое записано в операторе цикла... 0

Evg 21279 / 8301 / 637 Регистрация: 30.03.2009 Сообщений: 22,659 Записей в блоге: 30
	23.09.2015, 09:58	6
	Подозреваю, что всего лишь перепутаны местами строки 32 и 33 в программе из первого поста 1

Evg 21279 / 8301 / 637 Регистрация: 30.03.2009 Сообщений: 22,659 Записей в блоге: 30
	23.09.2015, 15:43	8
	Сообщение от russian_pepper не могу понять, почему в таком варианте программы, в консоль, надпись "pch = PRICE" не выводится ни разу, хотя значение "PRICE" присваивается pch и даже записывается в вектор temp_vector. Ну ответ же очевиден и точно такой же, как и в прошлый раз. Между "хотя значение PRICE присваивается pch и даже записывается в вектор temp_vector" и "if (pch == PRICE)" стоит модификация переменной pch. Т.е. твой код эквивалентен: C int x, y; x = 1; printf ("%d\n", x); /* увидели печать, что x=1 / x = 10; y = x; / почему-то основываясь на печати делаем вывод, что y=1 / Добавлено через 2 минуты* Ну и к слову. Почти все интерфейсы поиска спроектированы так, чтобы пользоваться ими в цикле for: C for (pch = strtok(str, separator); pch != NULL; pch = strtok(NULL, separator)) { ... } При таком раскладе ты хотя бы не будешь наступать на эти же самые грабли (модификация pch должна быть в самом конце тела цикла, а у тебя опять посередине) Добавлено через 1 минуту C if (pch == "PRICE") В языке Си строки надо сравнивать через strcmp. Хоть ты и используешь Си++, но строки у тебя в виде char*, а потому нужно использовать strcmp 1

	23.09.2015, 16:28

Особенности while (pch != NULL)

Решение