a simple boxe timer (code included)

[mackrackit]

COOL.
Do you have the Pic Basic code so folks around here can work with it?

[skimask]

COOL.
Do you have the Pic Basic code so folks around here can work with it?

It's in the zip file...and it looks a bit goofy to me (But that's just me. It doesn't have near enough colons in it )

[mackrackit]

It's in the zip file...and it looks a bit goofy to me (But that's just me. It doesn't have near enough colons in it )

Gotta get new glasses. Did not see it.

Colons-- Definitely needs a re-write

hozone, thanks for sharing.

[T.Jackson]

The project looks good

Avoid this coding style though ...

    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

Very inefficient. Suggest rethinking your schema using a "Select Case" structure. No need for the logical AND operator at all. Put your thinking cap on and you should be able to reduce the size of the code by half!

Regards,

Trent Jackson

[T.Jackson]

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

[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?