Почему мой список полностью не вычисляется правильно? C#

Question

Я должен получить значения, введенные в текстовое поле, а затем отобразить общее количество введенных значений, среднее значение и количество введенных значений.

До сих пор я закодированы:

List<int> intScoreList = new List<int>(20); 
decimal decScoreAverage = 0m; 
decimal decScoreTotal = 0m; 

private void btnAdd_Click(object sender, EventArgs e) 
{ 
    int intScore = Convert.ToInt32(txtScore.Text); 
    int intScoreCount = 0; 
    intScoreList.Add(intScore); 
    for (int i = 0; i < intScoreList.Count; i++) 
    { 
     intScoreList[0] = intScore; 
     decScoreTotal += intScoreList[i]; 
     intScoreCount++; //correct 
     decScoreAverage = decScoreTotal/intScoreCount; //correct 
    } 

Когда я вхожу тестовые значения 30, то 40, то общее дает мне 110 (30 + 40 * 2), а не 70 (30 + 40). Где я иду не так?

Answer 1

К комментаторскому (обоснованному) пункту ниже, вот еще несколько объяснений.

Вы не правильно вычисляете, потому что вы меняете первый элемент своих записанных значений: intScoreList[0] = intScore;. Изменяя это, вы мешаете сумме.

Очень чистая операция состоит из добавления новых данных в массив и пересчета суммы.

List<int> intScoreList = new List<int>(20);  
decimal decScoreAverage = 0m; 
decimal decScoreTotal = 0m; 

private void btnAdd_Click(object sender, EventArgs e) 
{ 
    int intScore = Convert.ToInt32(txtScore.Text); 
    int intScoreCount = 0; 
    intScoreList.Add(intScore); 

    decScoreTotal = 0; 
    foreach(int i in intScoreList) { 
     decScoreTotal += i; 
    } 

    decScoreAverage = decScoreTotal/intScoreList.Count; 
    intScoreCount = inScoreList.Count; 
} 

Обратите внимание, что это не обязательно является наиболее эффективной реализации, так как число значений увеличения, цикл foreach будет становиться все более и более дорогим. Лучший подход - отслеживать текущее общее и среднее значение и корректировать их с новым значением (которое вы можете добавить в список для других целей, если вам нужно).

Среднее значение вычисляется следующим образом:

New_average = old_average * (count-1)/count + new_value /count 

И вот новый код:

List<int> intScoreList = new List<int>(20);  
decimal decScoreAverage = 0m; 
decimal decScoreTotal = 0m; 

private void btnAdd_Click(object sender, EventArgs e) 
{ 
    int intScore = Convert.ToInt32(txtScore.Text); 
    int intScoreCount = 0; 
    intScoreList.Add(intScore); 

    intScoreCount = inScoreList.Count; 
    decScoreTotal += intScore;   
    decScoreAverage = decScoreAverage * (intScoreCount- 1)/intScoreCount + intScore/intScoreCount; 

} 

Answer 2

Поскольку цикл добавляет все элементы, «не требуется

decScoreTotal = 0 

перед вашей петлей?

Answer 3

Использование Linq. На событии щелчка кнопки добавления просто добавьте анализируемое значение. Затем отобразите Avg и Count.

List<decimal> scoreList = new List<decimal>(); 
    private void btnAdd_Click(object sender, EventArgs e) 
    { 
     decimalnum; 
     if (decimal.TryParse(txtScore.Text, out num)) 
     { 
      scoreList.Add(num); 
     } 

     // Assign to count label Text property = scoreList.Count; 
     // Assign to average label Text property = scoreList.Average(); 
    } 

Теперь представьте себе возможность сбросить все пользователя ввода:

private void btnReset_Click(object sender, EventArgs e) 
    { 
     scoreList.Clear(); 
    } 

Используя экземпляр списка вы можете легко добавить номер в качестве введен правильно и проанализирован от пользователя. Метод расширения AverageLinq сделает всю математику для вас, не нужно делать это самостоятельно.