Solved setItem() syntax error in Eclipse?

Hello Bukkit,

I keep getting this syntax error in eclipse from setItem. I set the two strings in set item (location and item) to variables, but replacing the variables with said content doesn't get a better result.

Any help is appreciated!

Code:java

World world = Bukkit.getWorld("world");
Location loc = new Location(world, -79.559, 78, -36.67);
ItemStack item = new ItemStack(Material.WOOL, 1, (byte)11);
    world.setItem(loc, item);
    Location loc2 = new Location(world, -79.87, 78, -72.54);
ItemStack item2 = new ItemStack(Material.WOOL, 1, (byte)14);
    world.setItem(loc2, item2);

The error is on the () of world.setItem()

 

What are you trying to do? I m new in this plugin things but i think that if u are trying to replace or add a block to the world u might be doing it wrong.

 

Yes, I am trying to add a fallen block to the world (Wool)

 

Why dont u just do like that:

Code:java

event.getPlayer().getWorld().spawnFallingBlock(Block.getLocation(), new ItemStack(Material.WOOL), (byte) 0);

 

Because it is not tied to a player. It will be an arena handler, when the game begins, wool blocks will spawn at the two team targets. But I think
world.spawnFallingBlock(loc, item); would work, unless my variables are messing it up...

 

Are you trying to spawn a Falling Block? Or an ItemStack laying on the ground?

 

Item stack lying on the ground RealDope

 

ccrama

PHP:

loc.getWorld().dropItem(loc, item)

 

Thanks xTrollxDudex


(and Haha)


EDIT: I get a new syntax error when I use xTrollxDudex 's method. I am thinking it's a variable problem. This is what I now have:

Code:java

Location loc = new Location(Bukkit.getWorld("world"), -79.559, 78, -36.67);
    ItemStack item = new ItemStack(Material.WOOL, 1, (byte)11);
    loc.getWorld().dropItem(loc, item);
    Location loc2 = new Location(Bukkit.getWorld("world"), -79.87, 78, -72.54);
    ItemStack item2 = new ItemStack(Material.WOOL, 1, (byte)14);
    loc2.getWorld().dropItem(loc2, item2);

 

ccrama
What's the syntax error?

 

xTrollxDudex
Identifier expected after this token on getWorld(). . Also says { expected after the period... I can't figure this one out!

 

ccrama
Can I see everything in the class?

 

This is the class so far. I haven't done much to it since I can't get items to spawn

Code:java

package me.ccrama.MineFortress2;
 
import org.bukkit.Bukkit;
import org.bukkit.Location;
import org.bukkit.Material;
import org.bukkit.World;
import org.bukkit.event.EventHandler;
import org.bukkit.event.world.WorldEvent;
import org.bukkit.inventory.ItemStack;
 
public class Arena {
    Location loc = new Location(Bukkit.getWorld("world"), -79.559, 78, -36.67);
    ItemStack item = new ItemStack(Material.WOOL, 1, (byte)11);
    loc.getWorld().dropItem(loc, item);
    Location loc2 = new Location(Bukkit.getWorld("world"), -79.87, 78, -72.54);
    ItemStack item2 = new ItemStack(Material.WOOL, 1, (byte)14);
    loc2.getWorld().dropItem(loc2, item2);
 
 
}
}

 

ccrama
Well you do have too many ending braces...

 

xTrollxDudex Eclipse told me to... IDK what's up with it.

xTrollxDudex It says there is a { expected after the period after getWorld(), and there is still an error with the getWorld method...

I hate to do this, but bump. I can't really continue my plugin until I get this item drop problem figured out. I edited the code to create a variable for world, because the new error was on getWorld, but now I get an error on the () saying Syntax error on "(", delete this token.

Code:java

package me.ccrama.MineFortress2;
 
import org.bukkit.Bukkit;
import org.bukkit.Location;
import org.bukkit.Material;
import org.bukkit.World;
import org.bukkit.event.EventHandler;
import org.bukkit.event.world.WorldEvent;
import org.bukkit.inventory.ItemStack;
 
public class Arena {
    Location loc = new Location( Bukkit.getWorld("world"), -79.559, 78, -36.67);
    ItemStack item = new ItemStack(Material.WOOL, 1, (byte)11);
    World w = Bukkit.getWorld("world");
    w.dropItem(loc, item);
    Location loc2 = new Location(Bukkit.getWorld("world"), -79.87, 78, -72.54);
    ItemStack item2 = new ItemStack(Material.WOOL, 1, (byte)14);
    w.dropItem(loc2, item2);
 
 
}

EDIT by Moderator: merged posts, please use the edit button instead of double posting.

 

ccrama
You need to put that into a method... Like

Code:java

public void doSomething() {
    // your code
}

Then just call the method if you need it.

 

I'm sorry, this has nothing to do with the topic, but your post says it was edited 6,969 times, and I'm dying of laughter. Thanks for inadvertently making my day

Your code should be in a method! You're simply placing it within the class, with no method wrapper around it. Try this:

Code:java

 
package me.ccrama.MineFortress2;
 
import org.bukkit.Bukkit;
import org.bukkit.Location;
import org.bukkit.Material;
import org.bukkit.World;
import org.bukkit.inventory.ItemStack;
 
public class Arena {
 
public Arena()
{
Location loc = new Location( Bukkit.getWorld("world"), -79.559, 78, -36.67);
ItemStack item = new ItemStack(Material.WOOL, 1, (byte)11);
World w = Bukkit.getWorld("world");
w.dropItem(loc, item);
Location loc2 = new Location(Bukkit.getWorld("world"), -79.87, 78, -72.54);
ItemStack item2 = new ItemStack(Material.WOOL, 1, (byte)14);
w.dropItem(loc2, item2);
}
 
}

This type of method is called a constructor, by the way. It would be called when you say something like:

Arena newArena = new Arena();

If you wanted it to be an actual method you can call, like Arena.doStuff(); You would do something like this:

Code:java

 
package me.ccrama.MineFortress2;
 
import org.bukkit.Bukkit;
import org.bukkit.Location;
import org.bukkit.Material;
import org.bukkit.World;
import org.bukkit.inventory.ItemStack;
 
public class Arena {
 
public void doStuff()
{
Location loc = new Location( Bukkit.getWorld("world"), -79.559, 78, -36.67);
ItemStack item = new ItemStack(Material.WOOL, 1, (byte)11);
World w = Bukkit.getWorld("world");
w.dropItem(loc, item);
Location loc2 = new Location(Bukkit.getWorld("world"), -79.87, 78, -72.54);
ItemStack item2 = new ItemStack(Material.WOOL, 1, (byte)14);
w.dropItem(loc2, item2);
}
 
}

and you would use it like this:

Arena newArena = new Arena();
newArena.doStuff();

If you were to add "static" before "void" you could also do:

Arena.doStuff();

Good Luck!

 

Minecrell and Pew446 thanks so much! I totally overlooked that :3. Works great now!

 

He didn't edit his post..

 

Trolldudex did haha

 

Oh god it's his signature.. That is the absolute best signature on earth. Guess I got trolled Made my day tho!

 

Lol, and I have signatures turned off, so I didn't see it!