[Omar]

I agree, your code will not work as intended as pointed out above.
Also with the embedded while loop it should not be located in Teleop

Your Distance is a float but you do not have a continuous range from 0 to 60. What will happen if Distance is 15.5 or 24.3 or 38.04 or 44.99999? Does your “distance sense by the vision camera” return? Does it return whole numbers (Blue) or floating point numbers (orange)? see attached code

Also, are there any “dead zones” in between the positions of your touch sensors when your shoots is moving up or down that no sensors will be “TRUE”? That will make it hard to program for.

A Potentiometer would work better for reading absolute position and you could have a variable numbers of shooting position instead of just 5.

But, can you set up your hardware, (the limit switches and your shooter) so that as it move up the next higher limit switch is engaged before the lower one is disengaged (or they could just all stay on as it moves up with you testing for only the highest one)

Give us a better idea on how your hardware works.