Среднее и стандартное отклонение

Question

Я пытаюсь сделать калькулятор среднего и стандартного отклонения в визуальной студии, используя форму Windows. Это для класса, и я должен использовать форму окна, и вы должны иметь возможность вводить данные вручную или выбрать файл.txt с данными в нем.

Трудно объяснить, что мне нужно делать, потому что я новичок в этом. В основном мне нужно, что делает button1_Click, но сделать то же самое для button2_Click. Дело в том, что он читает файл, а не принимает данные от пользователя. Он хранит (не уверен, что это правильное слово) значения в списке, а не в массиве, я думаю. Я не знаю, смогу ли я преобразовать его в массив или добавить значения в список в виде массива или как. Я действительно потерян!

Вот точная формулировка моего профессора для этого задания:

"Напишите программу, используя C#, которая найдет среднее и стандартное отклонение числа точек данных. Программа должна позволять пользователю вводить данные вручную или через текстовый файл".

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

using System;
using System.Collections.Generic;
using System.ComponentModel;
using System.Data;
using System.Drawing;
using System.Linq;
using System.Text;
using System.Threading.Tasks;
using System.Windows.Forms;

namespace BsweeneyCsharp3_1_
{
    public partial class Form1 : Form
    {
        List<double> values;

        public Form1()
        {
            InitializeComponent();
        }

        private void Form1_Load(object sender, EventArgs e)
        {
            values = new List<double>();
        }

        void ShowValues()
        {
            listBox1.Items.Clear();

            for (int i = 0; i < values.Count; i++)
                listBox1.Items.Add(values[i]);
        }

        private void button1_Click(object sender, EventArgs e)
        {

            double value = 0.00;
            double sum = 0.00, sumSquares = 0.00, squareSums;
            double stdDev = 0.00, mean = 0.00;

            if (textBox1.Text.Length == 0)
            {
                MessageBox.Show("You must enter a value.", "Standard Deviation");
                return;
            }

            try
            {               
                value = double.Parse(textBox1.Text);              
                values.Add(value);
                ShowValues();
                textBox1.Text = "";
                textBox1.Focus();
            }
            catch (FormatException)
            {
                MessageBox.Show("The value you entered is invalid.",
                                "");
            }

            for (int i = 0; i < values.Count; i++)
            {
                sum += values[i];
            }
            mean = sum / values.Count;
            squareSums = sum * sum;

            for (int i = 0; i < values.Count; i++)
                sumSquares += (values[i] * values[i]);

            double numerator = values.Count * sumSquares - squareSums;
            double denominator = values.Count * (values.Count - 1);
            stdDev = Math.Sqrt(numerator / denominator);

            textBox2.Text = mean.ToString();

            textBox3.Text = stdDev.ToString("F");

        }

        private void button2_Click(object sender, EventArgs e)
        {
            OpenFileDialog ofd = new OpenFileDialog();
            ofd.Filter = "TXT|*.txt";
            double mean = 0.00, stdDev = 0.00, sum = 0.00;
            double sumSquares = 0.00, squareSums = 0.00;
            int counter = 0;

            if (ofd.ShowDialog() == DialogResult.OK)
            {
                textBox4.Text = ofd.FileName;
            }

            string line;
            System.IO.StreamReader file = new System.IO.StreamReader(textBox4.Text);
            List<string> list = new List<string>();           

            while ((line = file.ReadLine()) != null)
            {                
                listBox2.Items.Add(line);
                var dbl = Convert.ToDouble(line);

                sum += dbl;                
                counter++;
            }
            if (counter > 0)
            {
                mean = sum / counter;
                squareSums += sum * sum;
            }
            if(counter > 0)
            {

                sumSquares += Math.Pow((sum - mean), 2);

                double numerator = counter * sumSquares - squareSums;
                double denominator = counter * (counter - 1);
                stdDev = Math.Sqrt(numerator / denominator);

                textBox2.Text = mean.ToString();

                textBox3.Text = numerator.ToString();
            }
        }
    }

}

Answer 1

Я сделал все возможное, чтобы интерпретировать то, что вы написали. Я учел, что вы новый программист. Я избежал серьезного рефакторинга, включая добавление столь необходимого метода. Начните изучать методы/ функции, которые дают вам повторное использование кода. Метод предотвращает проблему у вас здесь. Проблема в том, что вы рассчитываете одно и то же двумя способами.

Вы заявили, что "ручной" метод (не чтение файла) работает правильно, поэтому я принял это как источник правды.

соответствующие изменения кода в button2_Click

    if (counter > 0)//you don't need the same check 2x
    {
        mean = sum / counter;
        for (int i = 0; i < listBox2.Items.Count; i++)
            sumSquares += (Convert.ToDouble(listBox2.Items[i]) * Convert.ToDouble(listBox2.Items[i]));
        squareSums += sum * sum;

        //sumSquares += Math.Pow((sum - mean), 2);//You changed the formula again here

        double numerator = counter * sumSquares - squareSums;
        double denominator = counter * (counter - 1);
        stdDev = Math.Sqrt(numerator / denominator);

        textBox2.Text = mean.ToString();

        //textBox3.Text = numerator.ToString();//You changed how you did things again
        textBox3.Text = stdDev.ToString("F");
    }

К вашему сведению, я не проверял вашу математику.