Коды, приводящие к неожиданным результатам

@Виктор Гор · Регистрация: 25.03.2014

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

Код помещения кортежа в стрим.
Компилируется. По счастливому случаю обнаружил автоматическую ошибку до тестирования.
Кто скажет, почему следующий блок работал бы не так как ожидалось?

C++

for(uint8_t idx = 0; idx < inputDataSet->Fields->Count; idx++)
    switch (inputDataSet->Fields->Fields[idx]->DataType){
    case    ftBoolean:  { writer->WriteBoolean(inputDataSet->Fields->Fields[idx]->AsBoolean); break; }
    case    ftSmallint,
            ftInteger,
            ftWord,
            ftAutoInc:  { writer->WriteInteger(inputDataSet->Fields->Fields[idx]->AsInteger); break; }
    case    ftFloat,
            ftDouble:   { writer->WriteFloat(inputDataSet->Fields->Fields[idx]->AsFloat); break; }
    case    ftCurrency: { writer->WriteCurrency(inputDataSet->Fields->Fields[idx]->AsCurrency); break; }
    case    ftDate,
            ftTime,
            ftDateTime: { writer->WriteDate(inputDataSet->Fields->Fields[idx]->AsDateTime); break; }
    case    ftMemo:     { writer->WriteWideString(inputDataSet->Fields->Fields[idx]->AsWideString); break; }
    default:            { writer->WriteString(inputDataSet->Fields->Fields[idx]->AsString); break; }
    }

@DrOffset · 21.01.2015, 21:45

Сообщение от Виктор Гор

Кто скажет, почему следующий блок работал бы не так как ожидалось?

Оператор запятая.

@Виктор Гор · 22.01.2015, 00:32 **[ТС]**

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

C++

 for(uint8_t idx = 0; idx < inputDataSet->Fields->Count; idx++)
    switch (inputDataSet->Fields->Fields[idx]->DataType){
    case    ftBoolean:  { writer->WriteBoolean(inputDataSet->Fields->Fields[idx]->AsBoolean); break; }
    case    ftSmallint: { writer->WriteInteger(inputDataSet->Fields->Fields[idx]->AsInteger); break; }
    case    ftInteger:  { writer->WriteInteger(inputDataSet->Fields->Fields[idx]->AsInteger); break; }
    case    ftWord:     { writer->WriteInteger(inputDataSet->Fields->Fields[idx]->AsInteger); break; }
    case    ftAutoInc:  { writer->WriteInteger(inputDataSet->Fields->Fields[idx]->AsInteger); break; }
    case    ftFloat:    { writer->WriteFloat(inputDataSet->Fields->Fields[idx]->AsFloat); break; }
    case    ftDouble:   { writer->WriteFloat(inputDataSet->Fields->Fields[idx]->AsFloat); break; }
    case    ftCurrency: { writer->WriteCurrency(inputDataSet->Fields->Fields[idx]->AsCurrency); break; }
    case    ftDate:     { writer->WriteDate(inputDataSet->Fields->Fields[idx]->AsDateTime); break; }
    case    ftTime:     { writer->WriteDate(inputDataSet->Fields->Fields[idx]->AsDateTime); break; }
    case    ftDateTime: { writer->WriteDate(inputDataSet->Fields->Fields[idx]->AsDateTime); break; }
    case    ftMemo:     { writer->WriteWideString(inputDataSet->Fields->Fields[idx]->AsWideString); break; }
    default:            { writer->WriteString(inputDataSet->Fields->Fields[idx]->AsString); break; }
    }

@DrOffset · 22.01.2015, 18:08

Сообщение от Виктор Гор

Но есть ещё один момент который не виден с первого взгляда и он остался.

Предлагаешь угадывать?
Рискну предположить, что если элементов больше чем 255, то будет переполнение в счетчике.

Сообщение от Виктор Гор

Перепишу с учётом исправления.

Избыточность. Лучше так:

C++

 for(size_t idx = 0; idx < inputDataSet->Fields->Count; idx++)
    switch (inputDataSet->Fields->Fields[idx]->DataType){
    case    ftBoolean:  { writer->WriteBoolean(inputDataSet->Fields->Fields[idx]->AsBoolean); break; }
    case    ftSmallint: 
    case    ftInteger:  
    case    ftWord:     
    case    ftAutoInc:  { writer->WriteInteger(inputDataSet->Fields->Fields[idx]->AsInteger); break; }
    case    ftFloat:    
    case    ftDouble:   { writer->WriteFloat(inputDataSet->Fields->Fields[idx]->AsFloat); break; }
    case    ftCurrency: { writer->WriteCurrency(inputDataSet->Fields->Fields[idx]->AsCurrency); break; }
    case    ftDate:     
    case    ftTime:     
    case    ftDateTime: { writer->WriteDate(inputDataSet->Fields->Fields[idx]->AsDateTime); break; }
    case    ftMemo:     { writer->WriteWideString(inputDataSet->Fields->Fields[idx]->AsWideString); break; }
    default:            { writer->WriteString(inputDataSet->Fields->Fields[idx]->AsString); break; }
    }

Ну и вот это вот писать "inputDataSet->Fields->Fields[idx]" каждый раз неужели не напрягает? Не лучше ли получить указатель на текущий элемент в начале цикла, и потом везде работать с ним. Будет и короче и понятней.

@Виктор Гор · 22.01.2015, 19:11 **[ТС]**

255 полей это много, даже не буду предполагать для какой сущности такое может понадобится.

TField * field = inputDataSet->Fields->Fields[idx]; переписал ещё утром)

Угадывать не надо. ftDouble - это константа не с TFieldType. Её значение равно ftString - но так как стринг ловится в default то дублирования нет и компилятор пропускает. С первого же теста б вылетело.

@DrOffset · 23.01.2015, 00:06

Сообщение от Виктор Гор

Угадывать не надо. ftDouble - это константа не с TFieldType. Её значение равно ftString - но так как стринг ловится в default то дублирования нет и компилятор пропускает.

Вот именно что надо было. Ибо из приведенного кода это никак не очевидно, а телепаты все в отпусках как назло

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

Сообщение от Виктор Гор

255 полей это много, даже не буду предполагать для какой сущности такое может понадобится

Опять же, из приведенного кода не видно всего этого. Что это за поля и т.п.
Код, который ты приводишь, должен отражать проблему. Вот проблема с оператором запятой видна была сразу и ты получил ссылку. А тут - нет. Тут только телепатия.
Хотя бы можно было написать в шапке темы, что это Builder

@DrOffset 18840 / 9839 / 2408 Регистрация: 30.01.2014 Сообщений: 17,280
	21.01.2015, 21:45	2
	Сообщение от Виктор Гор Кто скажет, почему следующий блок работал бы не так как ожидалось? Оператор запятая. 1

@Виктор Гор 1 / 1 / 0 Регистрация: 25.03.2014 Сообщений: 29
	22.01.2015, 19:11 [ТС]	5
	255 полей это много, даже не буду предполагать для какой сущности такое может понадобится. TField * field = inputDataSet->Fields->Fields[idx]; переписал ещё утром) Угадывать не надо. ftDouble - это константа не с TFieldType. Её значение равно ftString - но так как стринг ловится в default то дублирования нет и компилятор пропускает. С первого же теста б вылетело. 0

@DrOffset 18840 / 9839 / 2408 Регистрация: 30.01.2014 Сообщений: 17,280
	23.01.2015, 00:06	6
	Сообщение от Виктор Гор Угадывать не надо. ftDouble - это константа не с TFieldType. Её значение равно ftString - но так как стринг ловится в default то дублирования нет и компилятор пропускает. Вот именно что надо было. Ибо из приведенного кода это никак не очевидно, а телепаты все в отпусках как назло Добавлено через 4 минуты Сообщение от Виктор Гор 255 полей это много, даже не буду предполагать для какой сущности такое может понадобится Опять же, из приведенного кода не видно всего этого. Что это за поля и т.п. Код, который ты приводишь, должен отражать проблему. Вот проблема с оператором запятой видна была сразу и ты получил ссылку. А тут - нет. Тут только телепатия. Хотя бы можно было написать в шапке темы, что это Builder 0