Change <= to < and make variable instead of using same function 3 times
authorKevin Zhang <icestormf1@gmail.com>
Sun, 24 Jan 2016 00:10:12 +0000 (16:10 -0800)
committerKevin Zhang <icestormf1@gmail.com>
Sun, 24 Jan 2016 00:10:12 +0000 (16:10 -0800)
src/org/usfirst/frc/team3501/robot/subsystems/Shooter.java

index 675b4b8cbf52353aa2a70418ee73c5b3ae3f150e..ef27bf92157dc341fef9f188634a94be34d9ffb3 100755 (executable)
@@ -19,9 +19,9 @@ public class Shooter extends Subsystem {
   }
 
   public void setSpeed(double speed) {
-    if (speed >= 1.0)
+    if (speed > 1.0)
       shooter.set(1.0);
-    else if (speed <= -1.0)
+    else if (speed < -1.0)
       shooter.set(-1.0);
     else
       shooter.set(speed);
@@ -37,12 +37,12 @@ public class Shooter extends Subsystem {
 
   // Use negative # for decrement. Positive for increment.
   public void changeSpeed(double change) {
-    if (getCurrentSetPoint() + change >= 1.0)
+    double newSpeed = getCurrentSetPoint() + change;
+    if (newSpeed > 1.0)
       shooter.set(1.0);
-    else if (getCurrentSetPoint() + change <= -1.0)
+    else if (newSpeed <= -1.0)
       shooter.set(-1.0);
     else {
-      double newSpeed = getCurrentSetPoint() + change;
       setSpeed(newSpeed);
     }
   }