Thread: Whats wrong with my java code? I really need help!

So i was working on my code for a game called Minecraft and so here is my first code, titled mod_MOO.java.

package net.minecraft.src;
 
public class mod_MOO extends BaseMod
{
        public static final Block MOOBlock = new MOO(123, ModLoader.addOverride("/terrain.png", "/MOO/MOOBlock.png").setHardness(2F).setResistance(1F).setStepSound(Block.soundStoneFootstep).setBlockName("MOO");
 
        public mod_MOO()
        {
 
        }
 
        public void load()
        {
                ModLoader.RegisterBlock(MOOBlock);
                ModLoader.AddName(MOOBlock, "MOO");
                ModLoader.AddShapelessRecipe(new ItemStack(MOOBlock, 123), new Object[] {Item.leather, Item.leather});
        }
 
        public String getVersion()
        {
                return "1.0.0";
        }
}

And now here is MOO.java:

package net.minecraft.src;
 
public class MOOBlock extends Block
{
        public MOOBlock(int i, int j)
        {
                super(i, j, Material.rock);
        }
}

The error is on mod_MOO.java, it says on eclipse that the line with all of the properties and ids that it is missing two )'s.

Please help!!!

If your editor does not have the pair finding tool, you can use your eyeballs.
Edit the line in question and put each section up to an ending ) on a line by itself and scan the lines for the correct pairing. Put the crusor after a ) and press Enter to move to a new line
An example line:
.setHardness(2F)

Then scan the lines and pair the ( with their ending )s

I've found that the parenthesis after "MOO" is underlined red, and the one matching with it (before "MOO") is not underlined, so now I am just confused.
Please don't get too complicated, Im very new to java

This is not complicated. You need to find where the missing ) is. Starting on the left find a ( and then move to the right end and find its pairing ).

Break your long statement up into a bunch of smaller simpler statements instead of trying to do it all in one statement.
For example:
<type1> var1 = <value for var1>
<type2> var2 = <value for var2>
public static final Block MOOBlock = new MOO(var1, var2) ;

If this is too complicated, print the program on a piece of paper and use a pencil to circle the ( and draw a line to the ending ) and circle it and connect the two. Continue until all (s are paired with a ).

for the "new MOO(123," there is no closing parenthesis, but everywhere I try on that line will give me another error.

Otherwise, everything matches up!

for the "new MOO(123," there is no closing parenthesis

Add the ending ) at the just before the ;
Break the statement up into several short simple statements like I suggested before.

Add the ending ) at the just before the ;

What it already is: setBlockName("MOO"); and when i do this: setBlockName("MOO")); basically the whole line goes crazy
I separated each property thing into its own separate line, and it still says (on .setBlockName) theres an error
heres my whole thing now:

package net.minecraft.src;
 
public class mod_MOO extends BaseMod
{
        public static final Block MOOBlock = new MOO(120,
        ModLoader.addOverride("/terrain.png","/MOO/MOOBlock.png"),
        setHardness(2F),
        setResistance(1F),
        setStepSound(Block.soundStoneFootstep),
        setBlockName("MOO");
 
        public mod_MOO()
        {
 
        }
 
        public void load()
        {
                ModLoader.RegisterBlock(MOOBlock);
                ModLoader.AddName(MOOBlock, "MOO");
                ModLoader.AddShapelessRecipe(new ItemStack(MOOBlock, 123), new Object[] {Item.leather, Item.leather});
        }
 
        public String getVersion()
        {
                return "1.0.0";
        }
}

NOTE: it says this in the line with the setBlockName thing:

Multiple markers at this line
	- Syntax error, insert ")" to complete 
	 VariableInitializer

And again, the ) next to "MOO" is underlined red

The chained code that you wrote used .s to connect the value returned by one method to connect to the next method. You have left them out when you put the method calls on separate lines. Put the .s back where they go.

Where did you copy this statement from? It is a classical example of how to confuse beginners.

Go back to where you copied the statement from and copy it again.

this is what i was supposed to type(replacing some things with MOO and MOOBlock):

        public static final Block oreTitanium = new CamelOreBlockOre(123, ModLoader.addOverride("/terrain.png", "/CamelMod/CamelOre/terrain/titaniumore.png").setHardness(3F).setResistance(5F).setStepSound(Block.soundStoneFootstep).setBlockName("oreTitanium");

and now this is what mine says:

package net.minecraft.src;
 
public class mod_MOO extends BaseMod
{
    public static final Block MOOBlock= new MOO(123, ModLoader.addOverride("/terrain.png", "/MOO/MOOBlock.png").setHardness(3F).setResistance(5F).setStepSound(Block.soundStoneFootstep).setBlockName("MOO");
 
        public mod_MOO()
        {
 
        }
 
        public void load()
        {
                ModLoader.RegisterBlock(MOOBlock);
                ModLoader.AddName(MOOBlock, "MOO");
                ModLoader.AddShapelessRecipe(new ItemStack(MOOBlock, 123), new Object[] {Item.leather, Item.leather});
        }
 
        public String getVersion()
        {
                return "1.0.0";
        }
}

yet it still gives an error on that same line on that same closing parenthesis

Multiple markers at this line
	- Syntax error, insert ")" to complete 
	 VariableInitializer
	- Syntax error, insert ")" to complete 
	 VariableInitializer

Simplify your code removing the following from the constructor call.

.setHardness(3F).setResistance(5F).setStepSound(Bl ock.soundStoneFootstep).setBlockName("MOO")

Do those method calls one by one.

Originally Posted by Norm

Simplify your code removing the following from the constructor call.

Do those method calls one by one.

like this?

package net.minecraft.src;
 
public class mod_MOO extends BaseMod
{
    public static final Block MOOBlock = new MOO(123, ModLoader.addOverride("/terrain.png", "/MOO/MOOBlock.png")
    		.setHardness(3F)
    		.setResistance(5F)
    		.setStepSound(Block.soundStoneFootstep)
    		.setBlockName("MOO");
 
        public mod_MOO()
        {
 
        }
 
        public void load()
        {
                ModLoader.RegisterBlock(MOOBlock);
                ModLoader.AddName(MOOBlock, "MOO");
                ModLoader.AddShapelessRecipe(new ItemStack(MOOBlock, 123), new Object[] {Item.leather, Item.leather});
        }
 
        public String getVersion()
        {
                return "1.0.0";
        }
}

because theres still errors

can you try this with eclipse and see if it gives you an error? Or will that not work?

What parameters does the MOO class's constructor take?
Code the call to that constructor with single variables for each parameter. Remove all the rest of the method calls from the constructor.
For example:

 public static final Block MOOBlock = new MOO(123, <a variable here>);

Then before the constructor, define a variable for <a variable here> and use it in the constructor call above.

Originally Posted by Norm

What parameters does the MOO class's constructor take?
Code the call to that constructor with single variables for each parameter. Remove all the rest of the method calls from the constructor.
For example:

 public static final Block MOOBlock = new MOO(123, <a variable here>);

Then before the constructor, define a variable for <a variable here> and use it in the constructor call above.

So heres my new mod_MOO:

package net.minecraft.src;
 
public class mod_MOO extends BaseMod
{
    public static final Block MOOBlock = new MOO(123, ModLoader.addOverride("/terrain.png","/MOO/MOOBlock.png").setHardness(3F).setResistance(5F).setStepSound(Block.soundStoneFootstep).setBlockName("MOO"));
 
        public mod_MOO()
        {
 
        }
 
        public void load()
        {
                ModLoader.RegisterBlock(MOOBlock);
                ModLoader.AddName(MOOBlock, "MOO");
                ModLoader.AddShapelessRecipe(new ItemStack(MOOBlock, 123), new Object[] {Item.leather, Item.leather});
        }
 
        public String getVersion()
        {
                return "1.0.0";
        }
}

Now it has

ModLoader.addOverride("/terrain.png","/MOO/MOOBlock.png").setHardness(3F)

underlined red and the reason:
"
Cannot invoke setHardness(float) on the primitive type
int
"
And heres my MOO.java, if you need it:

package net.minecraft.src;
 
public class MOO extends Block
{
        public MOO(int i, int j)
        {
                super(i, j, Material.rock);
        }
}

If the MOO constructor takes 2 int variables, Change the code to something like this:

int secParm = ModLoader.addOverride("/terrain.png","/MOO/MOOBlock.png").setHardness(3F).setResistance(5F).setStepSound(Block.soundStoneFootstep).setBlockName("MOO");
 
public static final Block MOOBlock = new MOO(123, secParm);

Then fix the value of secParm to be valid for the compile.

Originally Posted by Norm

If the MOO constructor takes 2 int variables, Change the code to something like this:

int secParm = ModLoader.addOverride("/terrain.png","/MOO/MOOBlock.png").setHardness(3F).setResistance(5F).setStepSound(Block.soundStoneFootstep).setBlockName("MOO");
 
public static final Block MOOBlock = new MOO(123, secParm);

Then fix the value of secParm to be valid for the compile.

Now heres what it says:

Cannot invoke setHardness(float) on the primitive type int

and my code:

package net.minecraft.src;
 
public class mod_MOO extends BaseMod
{
	static int MOOthing = ModLoader.addOverride("/terrain.png","/MOO/MOOBlock.png").setHardness(3F).setResistance(5F).setStepSound(Block.soundStoneFootstep).setBlockName("MOO");
	public static final Block MOOBlock = new MOO(123, MOOthing);
 
        public mod_MOO()
        {
 
        }
 
        public void load()
        {
                ModLoader.RegisterBlock(MOOBlock);
                ModLoader.AddName(MOOBlock, "MOO");
                ModLoader.AddShapelessRecipe(new ItemStack(MOOBlock, 123), new Object[] {Item.leather, Item.leather});
        }
 
        public String getVersion()
        {
                return "1.0.0";
        }
}

and the underlined red thing is this:

ModLoader.addOverride("/terrain.png","/MOO/MOOBlock.png").setHardness(3F)

You need to read the documentation for all the methods in that statement and see what they do, what values they return etc.
As I said before you need to break that chain of method calls up into single statements. Something like this:
<datatype> val = <objectRef>.<the method>(<the methods args>);

Where you replace the values in <>with the values from the chained statement being broken up.