C# get average of a set of numericalupdowns, excclude 0's

Question

http://gyazo.com/246cafd03301ba69d4e4e74251737a3e
~so my form will be filled with numeric updowns
~now the number (of numericupdowns) used by the user of the program can vary (a lot) some may use 5 some 15
~I need to take those numbers in the numeric updown and get the average of each column
but
~how to I code it where it will discard the number if the value = 0
~I want the code to be clean there was a VERY MESSY alternative (adding switches for every updown) but i would like to learn from people smarter than me !
Eat Mold. Subway

Answer 1

All right i see that the code you use is more complicated.
Firstly i would advise you to collect all numerical controls to collection. It will prevent you from empty result collection:
List<NumericUpDown> numList = new List<NumericUpDown>();
foreach (Control c in this.Controls)
if (c.GetType() == typeof(NumericUpDown))
NumericUpDown n = (NumericUpDown)c;
numList.Add(n);
Then remove numbers higher then 0 in the time of groupping results ( default average is zero here ):
var result = numList.GroupBy(num => num.Location.X, n => n.Value).Select(r => new
ColumnNo = r.Key,
ColumnAverage = r.Where(x => x > 0).DefaultIfEmpty(0).Average()
}).ToList();
Finnally sorting result collection will make the same order of results as columns are (from left to right)
result.Sort((a, b) => a.ColumnNo.CompareTo(b.ColumnNo));
Now result collection should contain correct results and have the size of the number of columns, so for example:
if(result.Count > 0)
label1.Text = "AVG: " + result[0].ColumnAverage;
Shows the average of the first column. It is also good practise to check whether result collection is not empty.
Thank you so much! how do i put this on a loop without a timer? because a timer is way to glitchy
Eat Mold. Subway