Question

I have a 10-field average lap calculator. However, in testing, someone said they normally only run X laps in practice, vs. 10 (let’s say 7).

I think I could use an if statement, but there’d be at least 10 of them and a bunch of clumsy code, and I’m not sure on arrays/switch statements exactly. I think all of those might be possible, but my low level of experience has yet to fully comprehend these useful tools.

CURRENT CODE:

double tenLapAvgVar = ((lap1Var + lap2Var + lap3Var + lap4Var + lap5Var + lap6Var + lap7Var + lap8Var + lap9Var + lap10Var) / 10);

So essentially, if someone leaves a field or fields blank, I want to calculate the average based on the populated fields, not 10 (if they leave 3 fields blank, calculate based on 7, for instance). Any help you guys could provide would be much appreciated, thanks!

Answer 1

You could have an ArrayList<EditText> object and a method which iterates over it and adds up the values. Something like:

    public double getLapAverage()
    {
    int noOfCompletedLaps = 0;

    double lapAve = 0;
    double lapsTotal = 0;    

    for(EditText text : textBoxes)
    {
       if(text.getText().toString().length() > 0)
       {
          //psuedo code, and assuming text is numerical
          lapsTotal += Double.parse(text.getText().toString());
          noOfCompletedLaps++;
       }
    }

    if( noOfCompletedLaps > 0)
    {
       lapAve = lapsTotal / noOfCompletedLaps;
    }

 return lapAve;
 }