Lots of errors

So I'm not so used to RSBuddy scripting, more for Powerbot. When I run my script with -dev, I get these errors:



My code is available here.

Usually the top error will contain a line of your script which has a problem.
One thing I noticed looking at your onStart method was that you call your GUI twice.

b0xb0x wrote:

Usually the top error will contain a line of your script which has a problem.
One thing I noticed looking at your onStart method was that you call your GUI twice.

Yes, that worked. Thanks!!!

b0xb0x wrote:

Usually the top error will contain a line of your script which has a problem.
One thing I noticed looking at your onStart method was that you call your GUI twice.

That worked, but when I run it, it shows my GUI. Then, I hit "Begin", and the GUI shows up again. I hit "Begin" once more and RSBuddy closes.

Remo wrote:

b0xb0x wrote:

Usually the top error will contain a line of your script which has a problem.
One thing I noticed looking at your onStart method was that you call your GUI twice.

That worked, but when I run it, it shows my GUI. Then, I hit "Begin", and the GUI shows up again. I hit "Begin" once more and RSBuddy closes.

Hmm. Post your code.

Remo wrote:

So I'm not so used to RSBuddy scripting, more for Powerbot. When I run my script with -dev, I get these errors:



My code is available here.

There's my code. I've tried changing the startButton actionPerformed to this.dispose();, System.exit(0);, dispose();, none work.

You are initializing the GUI twice. Remove your second GUI call. Once you do that, try it again.

b0xb0x wrote:

You are initializing the GUI twice. Remove your second GUI call. Once you do that, try it again.

The GUI appears once, though when I click begin it just closes.

Remo wrote:

b0xb0x wrote:

You are initializing the GUI twice. Remove your second GUI call. Once you do that, try it again.

The GUI appears once, though when I click begin it just closes.

Oh. Change this.dispose() to dispose(). By adding "this", you are disposing the whole class, therefore disposing the client GUI as well.

b0xb0x wrote:

Remo wrote:

b0xb0x wrote:

You are initializing the GUI twice. Remove your second GUI call. Once you do that, try it again.

The GUI appears once, though when I click begin it just closes.

Oh. Change this.dispose() to dispose(). By adding "this", you are disposing the whole class, therefore disposing the client GUI as well.

Did that, the same happens.

Do some debugging; add a log("t"); to the start of your loop so we know if it gets to the loop or not. If it gets to your loop, my guess is that you are terminating the JVM with System.exit(0). An alternative to this would be stop(), as that stops the script rather than terminating the JVM.

b0xb0x wrote:

Do some debugging; add a log("t"); to the start of your loop so we know if it gets to the loop or not. If it gets to your loop, my guess is that you are terminating the JVM with System.exit(0). An alternative to this would be stop(), as that stops the script rather than terminating the JVM.

I think the only thing I need help with is programming the start button. I just need the method to use and maybe then I wouldn't screw up so much.

Add a setVisible(false) before this.dispose(). Maybe that will work if the above didn't.

b0xb0x wrote:

Add a setVisible(false) before this.dispose(). Maybe that will work if the above didn't.

Alright, let me try that...

Remo wrote:

Here's my new code, same problem persists.

When you run it, does it log t?

b0xb0x wrote:

Remo wrote:

Here's my new code, same problem persists.

When you run it, does it log t?

No, so maybe it's a problem in the loop.

Remo wrote:

No, so maybe it's a problem in the loop.

Since it never gets to log("t");, the onStart() does not complete.

You can remove the main from your GUI class.

I don't know if this will fix it or not. But I remove this from my GUIs

Code:

 public void main(String args[]) {
           java.awt.EventQueue.invokeLater(new Runnable() {
               public void run() {
                   new AirRC().setVisible(true);
               }
           });
       }

TaffTech wrote:

I don't know if this will fix it or not. But I remove this from my GUIs

Code:

 public void main(String args[]) {
           java.awt.EventQueue.invokeLater(new Runnable() {
               public void run() {
                   new AirRC().setVisible(true);
               }
           });
       }

Yeah, b0xb0x told me that. Do you think this code will work?

Hmm, idk, try it
By the way you have no mouse being drawn, just a new instance of Graphics being rotated constantly :P

b0xb0x wrote:

Hmm, idk, try it
By the way you have no mouse being drawn, just a new instance of Graphics being rotated constantly :P

I fixed that, yet the script still won't work.

How am I supposed to get to scripting if I can't?