C4700 неинициализированная локальная переменная «number2» и «number3»

Question

У меня есть эта проблема с моей визуальной студией 2015 года. Ошибка возникает в 15-й строке. Внезапно говорится, что я не инициализировал переменную, я не знаю, как ее инициализировать.

// Average.cpp. : Defines the entry point for the console application. 

#include "stdafx.h" 
#include <iostream> 
#include <math.h> 
#include <stdio.h> 
using namespace std; 

int main() 
{ 
    int number1, number2, number3; 
    double average; 
    cout << "Enter three integers and I will display the average" << endl; 
    cin >> number1, number2, number3; 
    average = (number1 + number2 + number3)/3.0; 
    cout << "The average is" << average << endl; 

    return 0; 
} 

Answer 1

Эта линия не делать то, что вы ожидаете, что делать:

cin >> number1, number2, number3; 

Вы, вероятно, хотел прочитать 3 номера из cin, это делается следующим образом:

cin >> number1 >> number2 >> number3; 

В текущей форме оно оценивается как

(cin >> number1), number2, number3; 

Результат этого выражения: number3, который не инициализирован в вашей программе. number2 является частью выражения, также не инициализированным, поэтому компилятор также предупреждает об этом, хотя значение отбрасывается.

Answer 2

Оператор запятая разделяет выражения в C++ и присоединяется к ним в большую выражении, так что оператор cin >> number1, number2, number3 прочитал бы выражение number1, number2, number3 от cin эквивалентно (cin >> number1), number2, number3, который не имеет особого смысла, и это, конечно, не то, что вы хотели делать.

Попробуйте cin >> number1 >> number2 >> number3 вместо этого.

Это одна из тех странных «особенностей» C++, с которой язык был бы намного лучше.

Answer 3

Как уже упоминалось, это должно работать:

#include <iostream> 

using namespace std; 

int main() 
{ 
    // ... 
    cin >> number1 >> number2 >> number3; // here is the fix 

    // ... 
    cout << "The average is" << average << endl; // same as you did here 

    return 0; 
}