# Loop Statment.

The evaluation of SixSix will be to a thousand. How can the expression be writen with less code.
Can live with (If TePCom1Lenght > 0 And TePCom1Lenght < 6 Then
SixSix = 6)

Function SixSix(TePCom1Lenght As Currency)
If TePCom1Lenght < 0 Then
Exit Function
Else
If TePCom1Lenght > 0 And TePCom1Lenght < 5.5 Then
SixSix = 6
Else
If TePCom1Lenght > 5.5 And TePCom1Lenght <= 11.5 Then
SixSix = 12
Else
If TePCom1Lenght > 11.5 And TePCom1Lenght <=17.5  Then
SixSix = 18
Else
If TePCom1Lenght > 17.5 And TePCom1Lenght <= 23.5 Then
SixSix = 24
Else
If TePCom1Lenght > 23.5 And TePCom1Lenght <= 29.5 Then
SixSix = 30
Exit Function
End If
End If
End If
End If
End If
End If

Thanks Lynn
###### Who is Participating?

Commented:
This does exactly what your function does:
JR

Function SixSix(TePCom1Lenght As Currency) As Currency

If TePCom1Lenght > 0 Then
SixSix = Fix(((TePCom1Lenght + 0.49) / 6) + 1) * 6
End If

End Function

0

Commented:
Hi newt25,

This is not exact, as if it was 11.5, it would also go to 12, but it's close. Do you need it exact?

TePCom1Lenght = Ceil(Round(TePCom1Lenght)%6)

Regards,
Zyloch
0

Commented:
Hi!
Thinking that a select case structure might work best in here:

Function SixSix6(TePCom1Lenght As Currency)
If TePCom1Lenght < 0 Then Exit Function
Select Case TePCom1Lenght
Case 0 To 5.5
SixSix = 6
Case 5.5 To 11.5
SixSix = 12
Case 11.5 To 17.5
SixSix = 18
Case 17.5 To 23.5
SixSix = 24
Case 23.5 To 29.5
SixSix = 25
End Select
End Function

Matti
0

Commented:
SixSix = (int((TePCom1Lenght +0.4)/6) * 6) + 6
0

Commented:
Basically you want to round up to the nearest interger divisable by six if the input is over zero.
This function will do the trick.
JR

Function SixSix(TePCom1Lenght As Currency) As Currency

If TePCom1Lenght > 0 Then
SixSix = (TePCom1Lenght \ 6 + 1) * 6
End If
End Function
0
Question has a verified solution.

Are you are experiencing a similar issue? Get a personalized answer when you ask a related question.

Have a better answer? Share it in a comment.