Vfrdtyky

namespace Files_and_Arrays_II

{

    public partial class Form1 : Form

    {

        public Form1()

        {

            InitializeComponent();

        }



        private void button3_Click(object sender, EventArgs e)

        {

            this.Close();

        }



        private void button1_Click(object sender, EventArgs e)

        {

            StreamReader inputFile;

            int doctor = 0;

            double total = 0, average_sys = 0;

            string name, DocN;



            string doctors = new string[3] { "D. ABRAMS, MD", "D. JARVIC, MD", "T. PANOS, MD" };

            int systolic = new int[5];

            int diastolic = new int[5];



            OpenFileDialog openFile = new OpenFileDialog();



            if (openFile.ShowDialog() == DialogResult.OK)

            {

                inputFile = File.OpenText(openFile.FileName);

                while (!inputFile.EndOfStream)

                {

                    name = inputFile.ReadLine();

                    for (int i = 0; i < 5; i++)

                    {

                        systolic[i] = int.Parse(inputFile.ReadLine());

                        diastolic[i] = int.Parse(inputFile.ReadLine());

                    }



                    //Calculates average for systolic

                    for (int count = 0; count < systolic.Length; count++)

                    {

                        total += systolic[count];

                    }

                    average_sys = total / 5;



                    doctor = int.Parse(inputFile.ReadLine());

                    DocN = doctors[doctor];

                    listBox1.Items.Add(name + "t" + average_sys + "t" + DocN);



                }               

            }

        }

    }

}

This is the file it is getting it from

When running the program I get the following averages for systolic: 184.6 (correct), 312 (wrong).

I've tried resetting the array at the end of the loop but that solves nothing

Others have pointed out the problem in this case, but it's a symptom of declaring variables at the top of the function. If you'd declared them close to where they are used, it would be obvious which variables apply to the whole function and which have a scope that only applies inside the loop.

Like this:

                string name = inputFile.ReadLine();



                //Calculates average for systolic

                double total = 0;

                for (int count = 0; count < systolic.Length; count++)

                {

                    total += systolic[count];

                }

                double average_sys = total / 5;



                int doctor = int.Parse(inputFile.ReadLine());

                string DocN = doctors[doctor];

                listBox1.Items.Add(name + "t" + average_sys + "t" + DocN);

Even better, use var instead of setting the variable type in two places and risking getting it wrong.

You're not resetting your variables for total and average for the second set of measurements.