Rerport system -- Nachricht an alle mit rechte senden.

[ZockerBoysHD]

Halöle.

Ich ahbe mich gerade ebend an ein report system für meinem server gesetzt.

Soweit funktioniert auch alles, nur wenn ich z.b.: /report ZockerBoysHD 1 mache also Grund 1= Hacks dann bekommt man als mod keine anchricht, obwohl alles eig. richtig sein dürfte.

Die Klasse:
http://pastebin.com/wPxCj7Pw

 

[LeokoDev]

Da sind n paar Sachen n bissel falsch gelaufen:

Line 41:
if(args.length == 1)
zu
if(args.length == 2)
Da /report NAME GRUND ja 2 argumente hat

Line 42:
if(args[0].matches(Bukkit.getOnlinePlayers().toString()))
zu
if(Bukkit.getOfflinePlayer(args[0]).isOnline())

Line 43
if(args[1].matches("1") || args[1].matches("2") || args[1].matches("3") || args[1].matches("4"))
kannst du ganz simpel machen mit (ist aber nicht falsch)
if(args[1].matches("1-4"))

und ganz neben bei wenn du zwei Strings vergleichen will nimmt man eig. .equals("str") und nicht .matches("str") weil matches mit Regex arbeitet und equals einfach nur die Strings vergleicht.
Und wenn du .equalsIgnoreCase("str") nimmst wird Groß- und Kleinschreibung ignoriert.

Das würde man zB in line 24 machen
if(args[0].matches("reasons"))
zu
if(args[0].equalsIgnoreCase("reasons"))

 

[ZockerBoysHD]

Da sind n paar Sachen n bissel falsch gelaufen:

Line 41:
if(args.length == 1)
zu
if(args.length == 2)
Da /report NAME GRUND ja 2 argumente hat

Line 42:
if(args[0].matches(Bukkit.getOnlinePlayers().toString()))
zu
if(Bukkit.getOfflinePlayer(args[0]).isOnline())

Line 43
if(args[1].matches("1") || args[1].matches("2") || args[1].matches("3") || args[1].matches("4"))
kannst du ganz simpel machen mit (ist aber nicht falsch)
if(args[1].matches("1-4"))

und ganz neben bei wenn du zwei Strings vergleichen will nimmt man eig. .equals("str") und nicht .matches("str") weil matches mit Regex arbeitet und equals einfach nur die Strings vergleicht.
Und wenn du .equalsIgnoreCase("str") nimmst wird Groß- und Kleinschreibung ignoriert.

Das würde man zB in line 24 machen
if(args[0].matches("reasons"))
zu
if(args[0].equalsIgnoreCase("reasons"))

Habe es jetzt so geamcht. Funzt abebr imemrnochnicht... Und es komtm KEIN error. nichts

 

[LeokoDev]

Habe es jetzt so geamcht. Funzt abebr imemrnochnicht... Und es komtm KEIN error. nichts

Poste pls nochmal deinen ganzen Code, so wie er jz ist und beschreb mal genau was passiert wenn man /report NAME 1 macht

 

[ZockerBoysHD]

Poste pls nochmal deinen ganzen Code, so wie er jz ist und beschreb mal genau was passiert wenn man /report NAME 1 macht

eeeeh hier: http://pastebin.com/mSePskC9

Es passiert nichts, wenn man das macht. nochnichtmal wird die nachricht, dass man den reportet hat ausgegeben...
Die klasse ist abebr importiert

 

[LeokoDev]

eeeeh hier: http://pastebin.com/mSePskC9

Es passiert nichts, wenn man das macht. nochnichtmal wird die nachricht, dass man den reportet hat ausgegeben...
Die klasse ist abebr importiert

Schick mal deine onEnable() Methode wo du den command registrierst

 

[ZockerBoysHD]

Schick mal deine onEnable() Methode wo du den command registrierst

Bitte:
http://pastebin.com/fDBuYqhb
Habe da wos genau ist mit // makiert

 

[LeokoDev]

Bitte:
http://pastebin.com/fDBuYqhb
Habe da wos genau ist mit // makiert

Oke das sieht so gut aus
1. Gehen die anderen Commands?
2. hast du den command in der plugin.yml registriert?
3. funkt /report reasons?
4. Mach vllt. noch mehr else-Statements
wie in line 73 bei http://pastebin.com/mSePskC9:
}else p.sendMessage("§cDer User muss online sein um ihn zu reporten!");
dann siehst du vllt wieso der code nicht ausgeführt wird.

 

[ZockerBoysHD]

Oke das sieht so gut aus
1. Gehen die anderen Commands?
2. hast du den command in der plugin.yml registriert?
3. funkt /report reasons?
4. Mach vllt. noch mehr else-Statements
wie in line 73 bei http://pastebin.com/mSePskC9:
}else p.sendMessage("§cDer User muss online sein um ihn zu reporten!");
dann siehst du vllt wieso der code nicht ausgeführt wird.

1.Ja
2.Ja
3. Ja
4. oke

 

[jens1o]

Du hast hier einen kleinen Fehler, da du die 2. If-Abfrage(wegen deinem wiederlichem Codestil...) in die erste verschachtelt hast, was nie erreicht werden kann.

Mal in schön. Das dicke Zeichen fehlt bei dir:

if (args.length === 1) {
    if (args[0].equalsIgnoreCase("reasons")) {
        p.sendMessage("§e========> §8Report Gründe §e<========");
        p.sendMessage("§8             Gründe             |      Bestrafung    ");
        p.sendMessage("§8-----------------------------------------------------");
        p.sendMessage("§81.: §7Hacking/Hackclient        |   20 Tage Bann -> 30 Tage -> Permanent ");
        p.sendMessage("§82.: §7Werbung                   |   Kick -> 24h Mute -> 1 Tag -> 5 Tage ");
        p.sendMessage("§83.: §7Spamming                  |   Kick -> 2 Tage Mute -> 1 Tag Bann -> Permanenten Bann");
        p.sendMessage("§84.: §7Beleidigungen             |   2h Mute -> 8h Mute -> 1 Tag Bann");
        p.sendMessage("§e=================><=================");
    } else {     
        p.sendMessage(KilltimeMain.prausrufe + "§8/report §cNAME §bZAHL DES GRUNDES");
        p.sendMessage(KilltimeMain.prausrufe + "Du Brauchst eine Liste alle Gründe? §b/report reasons");
    }        
} <== Dieses fehlt

Ansonsten kann die dritte If-Abfrage [if (args.length == 2)] nie erreicht werden, weil ja die Argumente nicht gleichzeitig aus einem bestehen UND aus zwei sein können.

Daran sieht man mal, wie wichtig ordentlich strukturierter Code ist.

// cc: Verschraubt

Kleiner Tipp abseits davon:

Der Ausführer des Commands kann auch bspw. die Console sein, bei dir würde das dann jetzt crashen. Nimm einfach statt Player die CommandSender Klasse, die verfügt auch über die Methode: ```CommandSender#sendMessage(string msg)```

Edit: Ups, das ist hier ja kein GitHub...

 

[ZockerBoysHD]

Du hast hier einen kleinen Fehler, da du die 2. If-Abfrage(wegen deinem wiederlichem Codestil...) in die erste verschachtelt hast, was nie erreicht werden kann.

Mal in schön. Das dicke Zeichen fehlt bei dir:

if (args.length === 1) {
    if (args[0].equalsIgnoreCase("reasons")) {
        p.sendMessage("§e========> §8Report Gründe §e<========");
        p.sendMessage("§8             Gründe             |      Bestrafung    ");
        p.sendMessage("§8-----------------------------------------------------");
        p.sendMessage("§81.: §7Hacking/Hackclient        |   20 Tage Bann -> 30 Tage -> Permanent ");
        p.sendMessage("§82.: §7Werbung                   |   Kick -> 24h Mute -> 1 Tag -> 5 Tage ");
        p.sendMessage("§83.: §7Spamming                  |   Kick -> 2 Tage Mute -> 1 Tag Bann -> Permanenten Bann");
        p.sendMessage("§84.: §7Beleidigungen             |   2h Mute -> 8h Mute -> 1 Tag Bann");
        p.sendMessage("§e=================><=================");
    } else {    
        p.sendMessage(KilltimeMain.prausrufe + "§8/report §cNAME §bZAHL DES GRUNDES");
        p.sendMessage(KilltimeMain.prausrufe + "Du Brauchst eine Liste alle Gründe? §b/report reasons");
    }       
} <== Dieses fehlt

Ansonsten kann die dritte If-Abfrage [if (args.length == 2)] nie erreicht werden, weil ja die Argumente nicht gleichzeitig aus einem bestehen UND aus zwei sein können.

Daran sieht man mal, wie wichtig ordentlich strukturierter Code ist.

// cc: Verschraubt

Kleiner Tipp abseits davon:

Der Ausführer des Commands kann auch bspw. die Console sein, bei dir würde das dann jetzt crashen. Nimm einfach statt Player die CommandSender Klasse, die verfügt auch über die Methode: ```CommandSender#sendMessage(string msg)```

Edit: Ups, das ist hier ja kein GitHub...

Ich habe es zwar nochnicht ausprobiert, abber es klingt, als ob es jz funken würde...

Ich nehme eig. nur den sender weil ich den spielr auch brauhcen weder, da ich auch noch nicht alle featueres im report system drin habe. da sollen auch z.b. noch andere sachen passieren usw. und why sollte die Konsole einen Reporten?!? die kann odhc garnicht überfprüfen ob jmd hackt? xD

 

[ZockerBoysHD]

Du hast hier einen kleinen Fehler, da du die 2. If-Abfrage(wegen deinem wiederlichem Codestil...) in die erste verschachtelt hast, was nie erreicht werden kann.

Mal in schön. Das dicke Zeichen fehlt bei dir:

if (args.length === 1) {
    if (args[0].equalsIgnoreCase("reasons")) {
        p.sendMessage("§e========> §8Report Gründe §e<========");
        p.sendMessage("§8             Gründe             |      Bestrafung    ");
        p.sendMessage("§8-----------------------------------------------------");
        p.sendMessage("§81.: §7Hacking/Hackclient        |   20 Tage Bann -> 30 Tage -> Permanent ");
        p.sendMessage("§82.: §7Werbung                   |   Kick -> 24h Mute -> 1 Tag -> 5 Tage ");
        p.sendMessage("§83.: §7Spamming                  |   Kick -> 2 Tage Mute -> 1 Tag Bann -> Permanenten Bann");
        p.sendMessage("§84.: §7Beleidigungen             |   2h Mute -> 8h Mute -> 1 Tag Bann");
        p.sendMessage("§e=================><=================");
    } else {   
        p.sendMessage(KilltimeMain.prausrufe + "§8/report §cNAME §bZAHL DES GRUNDES");
        p.sendMessage(KilltimeMain.prausrufe + "Du Brauchst eine Liste alle Gründe? §b/report reasons");
    }      
} <== Dieses fehlt

Ansonsten kann die dritte If-Abfrage [if (args.length == 2)] nie erreicht werden, weil ja die Argumente nicht gleichzeitig aus einem bestehen UND aus zwei sein können.

Daran sieht man mal, wie wichtig ordentlich strukturierter Code ist.

// cc: Verschraubt

Kleiner Tipp abseits davon:

Der Ausführer des Commands kann auch bspw. die Console sein, bei dir würde das dann jetzt crashen. Nimm einfach statt Player die CommandSender Klasse, die verfügt auch über die Methode: ```CommandSender#sendMessage(string msg)```

Edit: Ups, das ist hier ja kein GitHub...

Ich habe es zwar nochnicht ausprobiert, abber es klingt, als ob es jz funken würde...

Ich nehme eig. nur den sender weil ich den spielr auch brauhcen weder, da ich auch noch nicht alle featueres im report system drin habe. da sollen auch z.b. noch andere sachen passieren usw. und why sollte die Konsole einen Reporten?!? die kann odhc garnicht überfprüfen ob jmd hackt? xD

Nice!
Bisher als ich es alleine getestet habe, komtm schonmal soweit, bis da, wo der user geprüft wird ob er online ist!
ES GEHT!!!!! DANKE!!!!!!!!!!


Und. Ich ahbe das so freikopf soweit programmeirt... :O
wusste garnicht, dass ich sowas kann
Danke wegen den fixes^^

 

[TheSimufreak]

Noch nen kleinen Tipp für deine Hauptklasse:
In Zeile 128 bzw. 129 hast du

getCommand("help").setExecutor(new help());
getCommand("hilfe").setExecutor(new help());

Da würde ich dir raten mit Aliasen in der plugin.yml Datei zu arbeiten.
Auch empfehle ich dir in Zukunft eine bessere Namensgebung zu nutzen, bei Klassen, Methoden als auch Variablen. Dazu gibt es eine sehr häufig genutzte Konvention. Mal davon abgesehen dass du die Hauptklasse aus einem anderen Projekt (bei dem ich dir bei der Fehlersuche helfen wollte) übernommen hast, was auch die nichtssagenden Methodenname erklären würde.
Und bevor ich's vergesse, les dir bitte deine Beträge noch einmal durch bevor du sie abschickst, du scheinst da nämlich eine Schwäche für Rechtschreibfehler zu haben...

 

[ZockerBoysHD]

Noch nen kleinen Tipp für deine Hauptklasse:
In Zeile 128 bzw. 129 hast du

getCommand("help").setExecutor(new help());
getCommand("hilfe").setExecutor(new help());

Da würde ich dir raten mit Aliasen in der plugin.yml Datei zu arbeiten.
Auch empfehle ich dir in Zukunft eine bessere Namensgebung zu nutzen, bei Klassen, Methoden als auch Variablen. Dazu gibt es eine sehr häufig genutzte Konvention. Mal davon abgesehen dass du die Hauptklasse aus einem anderen Projekt (bei dem ich dir bei der Fehlersuche helfen wollte) übernommen hast, was auch die nichtssagenden Methodenname erklären würde.
Und bevor ich's vergesse, les dir bitte deine Beträge noch einmal durch bevor du sie abschickst, du scheinst da nämlich eine Schwäche für Rechtschreibfehler zu haben...

Ja. Das Stimmt. Ich kopiere oft einfach die main, damit ich das pl shcneller fertig bekomme

Und ds mit den fehlern, kommt davon, weil ich sehr schnell tippe.

 

[TheSimufreak]

Ja. Das Stimmt. Ich kopiere oft einfach die main, damit ich das pl shcneller fertig bekomme

Und du bist nicht der Meinung dass du die Fehler umgehen könntest wenn du dir beim 'Programmieren' die nötige Zeit lässt und nicht (schier) wahllos bereits vermurksten Code kopierst?! (Qualität > Quantität)
Sorry für den Sarkasmus, aber mit

wegen deinem wiederlichem Codestil...

hat @jensIO absolut recht.

 

[ZockerBoysHD]

Und du bist nicht der Meinung dass du die Fehler umgehen könntest wenn du dir beim 'Programmieren' die nötige Zeit lässt und nicht (schier) wahllos bereits vermurksten Code kopierst?! (Qualität > Quantität)
Sorry für den Sarkasmus, aber mit

hat @jensIO absolut recht.

SAGMAL WAS WILLST DU EIG: VON MIR:
DAS IST DOCH MINE SAHCE; WIE ICH PROGRAMMIERE; ODER DAS DA "AUSSEHEN" LASSE.
ICH MECKER JA AUCH NICHT; NUR WEIL DU MORGENS MIT DEM LINKEN ANSTATT RECHTEM FUS AUFGESTANDEN BIST:
ODER WEIL DU DEIN PC NICHT MIT DER RECHTEN HAND SONDERN MIT DER LINKEN HAND ANGEMACHT HAST:

ALSO WAS WILLT DU

 

[TheSimufreak]

Ich mecker' ja auch nicht

Sorry für die aggressive Haltung, manchmal muss man dem einen oder anderen mal die Meinung geigen. Ob das gerechtfertigt ist oder nicht sei dahin gestellt, sowas solltest du nie persönlich nehmen. Dennoch solltest du dir die Ratschläge zu Herzen nehmen (oder zumindest das was du für sinnvoll hältst). Deine Quellcode ist nicht das, was man als guten Programmierstil bezeichnen würde. Ob dir das jetzt wichtig ist, ist deine Angelegenheit, da hast du vollkommen recht.

Das ist doch meine Sache, wie ich [...] das da "aussehen" lasse.

Wenn du selber das Ziel verfolgst deine Programmierfähigkeiten auszubauen ist es ratsam, sich an die allgemeine Konvention zu halten.

Auch wenn es dir jetzt lächerlich erscheint die Namen deiner Instanzen nach einem vorgegeben Scheme zu wählen, bringt dieser Standard eine Menge Vorteile: So kann jeder auf einen Blick anhand des Namens erkennen wobei es sich bei einer Variable handelt oder was für einen Zweck eine Methode erfüllt. Das ist nicht nur für dich übersichtlicher sondern auch für andere Programmierer, denen du deinen Quellcode z.B. in einem Forum wie diesem präsentierst. Auch die Formatierung (z.B. Einrückungen) spielt hier eine Rollte, die bei dir aber recht vernünftig ausschaut.

Ich mache mir im Folgenden mal die Mühe deine Code zu kommentieren. Ich will dich damit nicht angreifen sonder dir konstruktive Ratschläge geben, wie du deine Programme in Zukunft besser gestalten kannst:

Deine Mainklasse:
- die hast du offensichtlich Kopiert, ist beim Programmieren ein absolutes no-go
- entsprechend ist der Name irreführend, aber zumindest heißt die nicht Main
- die Main implementiert das Interface Listener, obwohl du die Events (korrekterweise) ausgelagert hast
- Für die Config-Files ist eine eigene Klasse ebenfalls sinnvoller
- genau wie für die Listen/Maps, Scoreboard und die Prefixe von Zeile 66-69
- lobenswert ist, dass du für das Registrieren von Commands und den Listenern eigene Methoden geschrieben hast
- auch wenn die Namen createConfig() und createSConfig() ungünstig gewählt sind; createConfig() und createSpawnConfig() wären angebrachter und gehören ebenfalls in die separate Config-Klasse
- und für die Listener sowie die Commands zwei eigene Methoden besser wären


Deine Report-Command-klasse
- Klassenname sollten groß geschrieben werden, will an dieser Stelle aber nicht genauer drauf eingehen (s.O.)
- das Senden der 'Informationstabelle' geschieht bei dir Zeilenweise. Hier solltest du (meiner Meinung nach) alles in einen String packen und mit \n arbeiten
- du gibst diese Tabelle mehrfach an, obwohl du hier Redundanzen recht einfach vermeiden könntest indem du den String in einer Variable oder in einer eigenen Klasse speicherst.
- bei Bukkit/Spigot empfiehlt es sich die ChatColor Klasse zu nutzen anstatt die Codes direkt anzugeben
- Umlaute werden nicht immer Korrekt dargestellt, deshalb ist auch hier Vorsicht geboten

Ich denke, dass der angeschlagene Tonfall u.A. daher rührt, dass viele unerfahrene Programmierer ähnliche Fehler machen, sich dessen nicht bewusst sind und übersehen, dass Programmieren schwerer ist als sie meinen (Stichwort Scriptkiddie, auch wenn der Begriff aus einem etwas anderen Kontext stammt)