Help me fix looting.

[Vilius]

Fixed

Edited September 15, 2015 by Viliuks

[Precise]

public void process(main script) {
			
			GroundItem salmon = script.getGroundItems().closest("Raw salmon");
			
			script.log(salmon.isVisible());
			
			if(salmon != null && salmon.isVisible()) {
				
				salmon.interact("Take");
				
			} else {
				
				GroundItem trout = script.getGroundItems().closest("Raw trout");
				
				script.log(trout.isVisible());
				
			    if(trout != null && trout.isVisible()) {
			    	
			    	trout.interact("Take");
			    }
				
			}


		}

	},

What am I doing wrong?

 

 

you're logging if it is visible before you null check it. you must null check before using it at all

[Chris]

script.log(salmon.isVisible());

Edited September 15, 2015 by Sinatra

[Vilius]

you're logging if it is visible before you null check it. you must null check before using it at all

tried that, didn't work tho.

[Joseph]

you're logging if it is visible before you null check it. you must null check before using it at all

[Vilius]

_{Nevermind, canProcess is returning false. ;_;}