Solved Java wont cooperate with me anymore ^^

http://docs.oracle.com/javase/tutorial/

It doesn't.

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

 

EvilKittyCat123 You certainly should worry about if statements that do nothing, especially if they're supposed to do something.

 

Garris0n
I typed the command /nodrop, and it does what it is suppose to do. How is that not working?

 

you did an if and ignore its result, not sure how its deemed working.

If you tried to add another command, it will always print nodrop message.

 

Necrodoom
Okay, so, I'm gonna try something different. I deleted the class noDrop, and decided to put the event in my main class. But I get a red line under onPlayerDrop saying "void is an invalid type for the variable onPlayerDrop".

Here's my main class:
PS (Instead of yelling and screaming at me that I suck, just give me a copy of exactly how my class should look like.. please...)

Code:java

package me.stripa.NoDrop;
 
import java.util.logging.Logger;
 
import org.bukkit.Bukkit;
import org.bukkit.ChatColor;
import org.bukkit.command.Command;
import org.bukkit.command.CommandSender;
import org.bukkit.entity.Player;
import org.bukkit.event.Listener;
import org.bukkit.event.player.PlayerDropItemEvent;
import org.bukkit.plugin.PluginDescriptionFile;
import org.bukkit.plugin.java.JavaPlugin;
 
public class Main extends JavaPlugin implements Listener
{
  public final Logger logger = Logger.getLogger("Minecraft");
  public static Main plugin;
 
  public void onDisable()
  {
    PluginDescriptionFile pdfFile = getDescription();
    this.logger.info(pdfFile.getName() + " Has been disabled! ");
  }
 
  public void onEnable()
  {
    PluginDescriptionFile pdfFile = getDescription();
    this.logger.info(pdfFile.getName() + " Version " + pdfFile.getVersion() + " Has been enabled! ");
    getServer().getPluginManager().registerEvents(this, this);
  }
 
  public boolean onCommand(CommandSender sender, Command cmd, String label, String[] args) {
      Player player = (Player) sender;
 
 
      {
 
          if(cmd.getName().equalsIgnoreCase("nodrop"));
          if(player.hasPermission("nodrop.main")) {
          player.sendMessage(ChatColor.BLUE +"["
                  + ChatColor.GRAY + "NoDrop"
                  + ChatColor.BLUE + "] "
                  + ChatColor.YELLOW + "============ "
                  + ChatColor.GRAY + "NoDrop version "
                  + ChatColor.GREEN + "1.0 "
                  + ChatColor.YELLOW + "============    "
                  + ChatColor.GRAY + "Bukkit version:    "
                  + ChatColor.GREEN + Bukkit.getVersion() + "    "
                  + ChatColor.GRAY + "Plugin Developer: "
                  + ChatColor.GREEN + "EvilKittyCat123");
 
 
          } else {
              player.sendMessage(ChatColor.BLUE + "["
                          + ChatColor.GRAY + "NoDrop"
                          + ChatColor.BLUE + "] "
                          + ChatColor.RED + "You do not have permission!");
 
      }
 
 
 
          {
 
              @EventHandler
              public void onPlayerDrop(PlayerDropItemEvent evt);
 
 
          }
 
 
 
 
    return false;
 
}
}
}
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 

 

You provided invalid code 4 times in row, it wont compile and wont work.

look closely on line 37 of your paste one more time

 

There is no other solution. I could go over a few other things you shouldn't be doing that you likely got from outdated tutorials (logger, enable/disable messages, other stuff), however there's no point. You do not know enough Java to continue with Bukkit yet. There's not much you're going to be able to do without copying other people's code, and if you do manage to get something to work it will probably be luck. I'm not trying to insult you, by the way, that's simply the truth. You must learn Java, at least basic Java, before doing this. You can't just learn Bukkit.

 

EvilKittyCat123 You now came back to the original error of the beginning of this thread.
line 39: if statement doing nothing
line 67: method inside another method (playerdropitemevent inside oncommand)
line 67: empty method

If you are unsure how to fix it i recommend looking up a java tutorial.

 

RawCode Necrodoom
How would my class look like then?

 

This is exactly what I mean. You need to be able to write it for yourself.

 

EvilKittyCat123
I suggest to take a look here:
http://thenewboston.org/list.php?cat=31
start at 10, its where you go wrong in the command part.
And read this:
http://wiki.bukkit.org/Event_API_Reference

 

Garris0n
Oh, you mean like I did 4 times before and everything worked successfully? Good times, so fucking good times.

Listen up everybody! I am just trying to make thus 1 simple bukkit plugin which I have successfully managed to make many times before! I do not want to spend 2-3 years learning java in order to make a bukkit plugin which should take minimum 2 minutes! Again, I'VE DONE THIS SO MANY TIMES!

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

 

http://forums.bukkit.org/threads/repeating-the-same-command.209400/

You have the exact same error..

 

So what happened, exactly? You just magically forgot Java? It might have "hackily" worked, but I still remember perfectly what it was like to try to use Bukkit without knowing Java. Not actually knowing what you're doing and coming up with bizarre solutions to problems is not good. I remember doing some stupid things (like discovering the static keyword and thinking it was the solution to all my problems). Seriously, learn Java. I've told many people this because I speak from experience. Learn. Java.

 

If you have done it so many times before and made it work, how come you make these obvious mistakes now?
Like we said: Learn some java first.

 

Necrodoom
Mhm?
Garris0n
I am already learning java, 10-12 days ago I bought a few Java books..
Aqua
That's the whole reason I made this thread. I've made this sort of plugin so many times.

 

EvilKittyCat123
Look at line 39, then at http://forums.bukkit.org/threads/repeating-the-same-command.209400/#post-2106464, then back at line 39.

 

Necrodoom
I really don't see it.
Garris0n
I do read and learn java by using the books. But at the same time I am trying to make a plugin that I have made before.

 

Agh.

You do an if(something) then close it with ; instead of { }, which means you just did a line of code which checks a variable and does nothing. You asked the exact same question 4 months ago, but yet you dont see the problem in this code, despite being almost an identical duplicate of it.

I highly recommend reading a basic java tutorial before proceeding.

 

fuck this, I give up. Thanks a lot for the help!

I could have learned if somebody just gave me the correct code. Because then I would be able to see how it's done, and then proceed doing it right. I still don't understand why it is not working, but all I know is that it worked before.

- Cya

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

 

By the way everyone here has tried to help you, coding takes a long time but in the end its always worth it and makes you feel like you accomplished something awesome i dont think you are willing to take that time.
take a look here like Necrodoom suggested ! http://forums.bukkit.org/threads/repeating-the-same-command.209400/#post-2106464

 

DeluXeNationMC
I've already looked at it. But I just don't get it.. I am stupid. All I am asking for, is for somebody to tell me what to do step by step on how to make this shit work. But nobody wants to. So I just gave up.. Everything worked before..

 

If you need this plugin i suggest making a thread at plugin request section or looking up existing plugins.

 

EvilKittyCat123 First of all could you stop cursing because it is not necessary.

Code:java

 package me.stripa.NoDrop;
 
import java.util.logging.Logger;
 
import org.bukkit.Bukkit;
import org.bukkit.ChatColor;
import org.bukkit.command.Command;
import org.bukkit.command.CommandSender;
import org.bukkit.entity.Player;
import org.bukkit.event.Listener;
import org.bukkit.event.player.PlayerDropItemEvent;
import org.bukkit.plugin.PluginDescriptionFile;
import org.bukkit.plugin.java.JavaPlugin;
 
public class Main extends JavaPlugin implements Listener
{
public final Logger logger = Logger.getLogger("Minecraft");
public static Main plugin;
 
public void onDisable()
{
PluginDescriptionFile pdfFile = getDescription();
this.logger.info(pdfFile.getName() + " Has been disabled! ");
}
 
public void onEnable()
{
PluginDescriptionFile pdfFile = getDescription();
this.logger.info(pdfFile.getName() + " Version " + pdfFile.getVersion() + " Has been enabled! ");
getServer().getPluginManager().registerEvents(this, this);
}
 
public boolean onCommand(CommandSender sender, Command cmd, String label, String[] args) {
Player player = (Player) sender;
 
 
{
 
if(cmd.getName().equalsIgnoreCase("nodrop"));
if(player.hasPermission("nodrop.main")) {
player.sendMessage(ChatColor.BLUE +"["
+ ChatColor.GRAY + "NoDrop"
+ ChatColor.BLUE + "] "
+ ChatColor.YELLOW + "============ "
+ ChatColor.GRAY + "NoDrop version "
+ ChatColor.GREEN + "1.0 "
+ ChatColor.YELLOW + "============ "
+ ChatColor.GRAY + "Bukkit version: "
+ ChatColor.GREEN + Bukkit.getVersion() + " "
+ ChatColor.GRAY + "Plugin Developer: "
+ ChatColor.GREEN + "EvilKittyCat123");
 
 
} else {
player.sendMessage(ChatColor.BLUE + "["
+ ChatColor.GRAY + "NoDrop"
+ ChatColor.BLUE + "] "
+ ChatColor.RED + "You do not have permission!");
 
}
 
 
 
{
 
@EventHandler
public void onPlayerDrop(PlayerDropItemEvent evt); //The ; should be a {
 
 
}
 
 
 
 
return false;
 
}
}
}
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 

In this code the problem is the ; on the end of public void onPlayerDrop(PlayerDropItemEvent evt); should be a { braket

 

EvilKittyCat123 Giving up would require trying in the first place.

 

DeluXeNationMC Missed one.

 

Let's not turn this into a hate thread...
The OP doesn't seem to understand - we're here to help, not judge.

 

Errm, it wont let me put event.SetCancelled(true);

 

I'm not trying to hate on him. But I'll agree that the OP doesn't understand - but he doesn't seem to want to accept our help, he wants us to tell him what the correct version is. Insisting that he's ready or that what he's doing worked a while ago when we know for a fact it didn't, and then just giving up? Not the kind of attitude somebody we can help has.