Novice Coder Help

The robot has the code on the roboRio and is connected to the driver station. The joystick is also connected but none of the motors are firing when the joystick is moved.

Here is the code:

package org.usfirst.frc.team4404.robot;

import edu.wpi.first.wpilibj.IterativeRobot;
import edu.wpi.first.wpilibj.Joystick;
import edu.wpi.first.wpilibj.RobotDrive;
import edu.wpi.first.wpilibj.Solenoid;
import edu.wpi.first.wpilibj.Spark;
import edu.wpi.first.wpilibj.Timer;


/**
 * The VM is configured to automatically run this class, and to call the
 * functions corresponding to each mode, as described in the IterativeRobot
 * documentation. If you change the name of this class or the package after
 * creating this project, you must also update the manifest file in the resource
 * directory.
 */
public class Robot extends IterativeRobot {
	Joystick mainStick;
	Timer timer = new Timer();
	RobotDrive drive;
	Solenoid gearDropper = new Solenoid(1);
	Spark rightMotor1;
	Spark rightMotor2;
	Spark leftMotor1;
	Spark leftMotor2;
	



	/**
	 * This function is run when the robot is first started up and should be
	 * used for any initialization code.
	 */
	@Override
	public void robotInit() {
		rightMotor1 = new Spark(0);
		rightMotor2 = new Spark(1);
		leftMotor1 = new Spark(2);
		leftMotor2 = new Spark(3);
		
		drive = new RobotDrive(rightMotor2, rightMotor1, leftMotor2, leftMotor1);
		mainStick = new Joystick(0);
		
		
		
	}

	/**
	 * This autonomous (along with the chooser code above) shows how to select
	 * between different autonomous modes using the dashboard. The sendable
	 * chooser code works with the Java SmartDashboard. If you prefer the
	 * LabVIEW Dashboard, remove all of the chooser code and uncomment the
	 * getString line to get the auto name from the text box below the Gyro
	 *
	 * You can add additional auto modes by adding additional comparisons to the
	 * switch structure below with additional strings. If using the
	 * SendableChooser make sure to add them to the chooser code above as well.
	 */
	@Override
	public void autonomousInit() {
		timer.reset();
		timer.start();
	}

	/**
	 * This function is called periodically during autonomous
	 */
	@Override
	public void autonomousPeriodic() {
		
	}

	/**
	 * This function is called periodically during operator control
	 */
	@Override
	public void teleopPeriodic() 
	{
		drive.arcadeDrive(mainStick);
		boolean buttonValue, isOpen;
		isOpen = false;
		buttonValue = mainStick.getTrigger();
		if(buttonValue)
		{
			if(!isOpen)
			{
				gearDropper.set(true);
			}
			else
			{
				gearDropper.set(false);
			}
			
			isOpen = !isOpen;
			
		}
		
	}

	/**
	 * This function is called periodically during test mode
	 */
	@Override
	public void testPeriodic() {
	}
}

What is the solution to this problem?

Do the lights on the motor controllers change colors when you move the joystick?
Have you confirmed that your PWMs are plugged in correctly?

The PWM’s leds flash on and off

The PWM is the black, white, and red cord that goes between the roborio and the motor controller. Is it plugged in properly? The light on the motor controller should change to red or green depending on which way your joystick is.

I would recommend printing out the joystick values or going to the USB Devices tab in the driver station to ensure that it is giving actual values. It is possible that your joystick or USB connection is the issue.

isOpen = false;
		buttonValue = mainStick.getTrigger();
		if(buttonValue)
		{
			if(!isOpen)
			{
				gearDropper.set(true);
			}
			else
			{
				gearDropper.set(false);
			}
			
			isOpen = !isOpen;
			
		}

isOpen will be always false, you have to put the variable global.

Check if your joystick is really in port 0 in driverStation.

If the LED’s are blinking, you might have the limit switch settings configured wrong or the PWM connection is not hooked up correctly. You should check the cable orientation and read the manual to match the blink code to the operating state of your Spark. It will also tell you how to make sure the limit switch configuration is correct in the manual. Other than the issue with the isOpen variable, your code looks correct. (Keeping in mind I’m not familiar with Java wpilib stuff)
https://images-na.ssl-images-amazon.com/images/I/51GXnDV605L.jpg