a simple boxe timer (code included)

[skimask]

Jeremy will correct it for you. If he does his IQ is 120+

Already did...down to 38 lines. But since nobody else can read it, decided not to post it...and didn't use select case. Those numbers aren't linear, but they're close enough to do some math on to get the desired results...thereby resulting in a simple single if/then/endif 3-line statement to handle all those if/then's.

[T.Jackson]

Can you post a readable example using either IF's or Select Case?

[skimask]

Can you post a readable example using either IF's or Select Case?

Nope...
It's a state secret now, top notch double-A rating, return postage included...
And it's doesn't have IF's, it has a single IF statement

[T.Jackson]

That's a shame Jeremy, I was going to expose you to the world as border line genius.

[T.Jackson]

I realize that I'm a bit drunk at the moment, now I'm saying 105+

[T.Jackson]

The usual suspects.

[skimask]

I realize that I'm a bit drunk at the moment, now I'm saying 105+

Then step away from the keyboard and go chase parked cars or something...

[skimask]

That's a shame Jeremy, I was going to expose you to the world as border line genius.

I've already exposed myself. I don't need any help.

[T.Jackson]

One possible "improved" solution is as follows ...

   Select Case b1
      Case Is < 219                                        
         timerN = 3
         timerAc = 120
         timerBc = 60     
      
      Case Is < 243                                         
         timerN = 2
         timerAc = 180
         timerBc = 60

      Case Is <= 255                                         
         timerN = 1
         timerAc = 600
         timerBc = 180 
   End Select

That code will do exactly the same job as ...

    If (b1 <= 255 And b1 >= 243) Then
        timerN = 1
        timerAc = 600
        timerBc = 180
    End If
    If (b1 <= 242 And b1 >= 219) Then
        timerN = 2
        timerAc = 180
        timerBc = 60
    End If
    If (b1 <= 218 And b1 >= 193) Then
        timerN = 3
        timerAc = 120
        timerBc = 60
    End If

BUT, it'll probably compile to half the size, run twice as fast, be more readable, easier to comprehend and definitely far easier to maintain.

Regards,

Trent Jackson

[T.Jackson]

I might add also that the same thing is possible using IF's, select case is always more effcient though however.

      for (int i = 0; i < 12; i++)
      {
          sample = tempArray[i];
          
          if (sample < 0)
             rangeA++;
          else if (sample < 10)
             rangeB++;
          else if (sample < 20)
             rangeC++;
          else if (sample < 30)
             rangeD++;
          else
             rangeE++;
      }

Notice how we start with the smallest number first? Always start with the smallest value first and work downwards with the next biggest value.

Hope it helps

Trent Jackson

[mackrackit]

What
is
so
hard
to
read
with
colons
?

I
think
they
help
make
the
code
MORE
readable
if
used
in
the
right
place.

It
makes
it
more
like
reading
a
book.