|
|
|
![]() |
|
|||||||
|
||||||||
![]() |
| Thread Tools | Rate Thread | Display Modes |
|
#1
|
||||
|
||||
|
Extremely Broken Code Please Help!
I have no idea why the issue with this program escapes me. I have been looking at it for days with no headway. Maybe a new set of eyes can see the problem. The program is simple. It uses my XBox class to control a four motor arcade drive system using one of the joysticks on the controller. Some buttons then control a couple accessory motors.
Here is the code: https://github.com/FranklinMarquette/Robot2016 Here is what the driver station says when I try to enable the robot: Quote:
Thanks so much, Franklin |
|
#2
|
||||
|
||||
|
Re: Extremely Broken Code Please Help!
So I notice code breaks here and the motor1Stop class has
Code:
public static Motor1 motor = new Motor1(); |
|
#3
|
|||
|
|||
|
Re: Extremely Broken Code Please Help!
Not sure what is causing your error yet, but it sounds like an undefined pointer.
But you should move a few lines around in your Motor1 and Motor2 classes. Right now you have the instantiation of the SpeedController above your constructor. SpeedController motor = new Victor(4); public Motor1() { } change it to. Private SpeedController motor; public Motor1(){ motor = new Victor(4); } |
|
#4
|
||||
|
||||
|
Re: Extremely Broken Code Please Help!
Okay so I applied those changes and have updated the code here:
https://github.com/FranklinMarquette/Robot2016 And it returns this error: Quote:
|
|
#5
|
||||
|
||||
|
Re: Extremely Broken Code Please Help!
Just a quick question. Are you reading the errors or just posting them here? It's ok if you are because they aren't the most lovely things to read (though you should try. It'll be much faster) What it's giving you is what's called a stack trace. It basically tells you the path the code took before it got an error. I've added a comment on what the error tells you.
Code:
ERROR Unhandled exception: java.lang.IllegalArgumentException: Subsystem must not be null. at [ edu.wpi.first.wpilibj.command.Command.requires(Com mand.java:194), #6 org.usfirst.frc.team4623.robot.commands.Extend.<in it>(Extend.java:16), #5 ^------------This is the file-------------------------^---------------Line #- ^ org.usfirst.frc.team4623.robot.OI.<init>(OI.java:5 6), #4 org.usfirst.frc.team4623.robot.Robot.robotInit(Rob ot.java:44), #3 du.wpi.first.wpilibj.IterativeRobot.startCompetit ion(IterativeRobot.java:72), #2 edu.wpi.first.wpilibj.RobotBase.main(RobotBase.jav a:241)] #1 Code:
requires(Robot.pnue); Last edited by TimTheGreat : 01-25-2016 at 08:25 PM. |
|
#6
|
|||
|
|||
|
Re: Extremely Broken Code Please Help!
I'm having the same problem. Compiles just fine but red light on driver station robot code.
When I comment out everything in public OI{}, I get green light on driver station robot code. But why?? TimTheGreat: What do you mean by "Try putting OI after everything"? Thanks for your reply! |
|
#7
|
||||
|
||||
|
Re: Extremely Broken Code Please Help!
Quote:
Code:
oi = new OI();
pnue = new Pnuematics();
|
|
#8
|
||||
|
||||
|
Re: Extremely Broken Code Please Help!
Okay that makes perfect sense. I did trace it back and could deduct that it had something to do with the extend class but never thought of movie the onus constructor
Thanks |
|
#9
|
|||
|
|||
|
Re: Extremely Broken Code Please Help!
Yes! That worked! Thanks so much, TimTheGreat!!!
|
|
#10
|
||||
|
||||
|
Re: Extremely Broken Code Please Help!
Glad I could help
![]() |
|
#11
|
||||
|
||||
|
Re: Extremely Broken Code Please Help!
Okay so moving that fixed it. Now the pnuematics are not working.
What does this error mean? Quote:
Thanks again, Franklin |
|
#12
|
||||
|
||||
|
Re: Extremely Broken Code Please Help!
Is your code up to date in the github repo?
PS: A good way of reaching me is on the Freenode IRC channel #robotpy. If you're on then you can have quick discussion which is a lot faster than CD Last edited by TimTheGreat : 01-27-2016 at 05:24 PM. |
|
#13
|
||||
|
||||
|
That's not what I changed it to today but that code threw the same error - the ports for the double solenoid are 0 and 1
|
|
#14
|
||||
|
||||
|
Re: Extremely Broken Code Please Help!
Quote:
Ok I think the problem is when you call the retract command you create a new solenoid on ports 0 and 1. Don't do this. If you call the retract command more than once (or call the extend command once) then you can't call it again because it tries to create a new solenoid. Create the solenoid somewhere and pass it to the command. |
![]() |
| Thread Tools | |
| Display Modes | Rate This Thread |
|
|