Error value computed is not used

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

double * array;
double result;
double myMin;
double myMax;

// Assume array is initialized properly...
for (int i = 0; i < sizeOfArray; ++i) {
    result = transmogrify(array[i]);
    if (i == 0) {
        myMin = result;
        myMax = result;
    }
    else if (result < myMin) {
        myMin = result;
    }
    else if (result > myMax) {
        myMax = result;
    }
}

Я получаю предупреждение, что значение, вычисленное для result никогда не используется, и поскольку мы рассматриваем все предупреждения как ошибки, он не компилируется. Как я могу исправить этот код, чтобы избежать предупреждения? Я использую g ++ для своего компилятора.

Вот текст предупреждения:

cc1plus: warnings being treated as errors
foo.cc:<lineno of transmogrify call>: error: value computed is not used

Изменить: я не понимаю голосов против, но теперь у меня все работает. Спасибо всем, что нашли время мне помочь.

Предполагая, что вам не нужно result вне цикла вы можете объявить result внутри цикла таким образом:

for( int i=0; i < sizeOfArray; ++i ) {
    double result = transmogrify( array[i] );
    ...
}

ответ дан 27 апр.

Я получаю предупреждение, что значение, вычисленное для result никогда не используется, потому что (теоретически) возможно, что ни одна из веток if / else не будет выбрана

Это не может быть причиной предупреждения, потому что result также используется в условиях if. Даже если ни одна из веток не будет взята, result До сих пор принято решение, что их принимать нельзя.

ответ дан 27 апр.

ответ дан 27 апр.

result = [...]
if (i == 0) {
    [... do something with result ...]
}
else if (result < myMin) {

В обеих ветвях if (), result используется. В первом случае он присваивается переменной, во втором — используется для сравнения. Так что компилятор не должен предупреждать.

Я подозреваю, что вы неправильно поставили диагноз. Скажите, пожалуйста, в чем именно заключается сообщение об ошибке (скопируйте и вставьте его). Кроме того, попробуйте опубликовать как можно меньший фрагмент кода, который действительно компилируется и выдает предупреждение? (Простая попытка сделать это, вероятно, позволит вам найти проблему)

РЕДАКТИРОВАТЬ: Возможно ли, что transmogrify () — это макрос, который использует result внутренне?

ответ дан 27 апр.

Перед оператором if:

result = 0.0;

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

ответ дан 27 апр.

Я получаю предупреждение о том, что значение, вычисленное для результата, никогда не используется, потому что (теоретически) возможно, что ни одна из ветвей if / else не будет выбрана, и поскольку мы обрабатываем все предупреждения как ошибки, это не компилируется. Как я могу исправить этот код, чтобы избежать предупреждения? Я использую g ++ для своего компилятора

Всегда используется значение, используемое для результата. Если не назначен, он используется в компараторе. Следовательно, компилятор неисправен.

ответ дан 27 апр.

Быстрое решение — развернуть первую итерацию, например:

double * array;
double result = transmogrify(array[0]);
double myMin = result;
double myMax = result;
int i;

for (i = 1; i < sizeOfArray; ++i) {
    result = transmogrify(array[i]);

    if (result < myMin) {
        myMin = result;
    }

    if (result > myMax) {
        myMax = result;
    }
}

РЕДАКТИРОВАТЬ: Я немного расскажу об этом. Вы не предоставили никакой подробной информации о sizeOfArray, но я предполагаю, что это целочисленный тип со знаком.

Я полагаю, вы неправильно поняли причину предупреждения, результат может не использоваться, потому что sizeOfArray может быть меньше или равным нулю, а не из-за предложений if..else внутри цикла. В приведенном выше коде может быть немного яснее, почему вам нужно осторожно обращаться с кейсом (sizeOfArray <= 0), но это не менее важно и в исходном фрагменте кода.

ответ дан 27 апр.

Удаление первого else влияет только на время выполнения первого прогона цикла. Я бы лучше это устранил.

if (i == 0) {
    myMin = result;
    myMax = result;
}
if (result < myMin) 
    myMin = result;
else if (result > myMax) 
    myMax = result;

ответ дан 27 апр.