Style-Verstoß
Style-Verstoß
Hi,
habe gestern das erste mal den Praktomat getestet. Dabei hat er mir ein paar Styleverstösse gesenden, mit denen ich nichts anfangen kann.
Hier mal die Meldungen:
GgT.java:1: Missing a Javadoc comment.
GgT.java:8:36: Expected @param tag for 'args'.
bezieht sich auch die Zeilen:
1:
1 public class GgT {
2
3 /**
4 * @param args
5 */
8:
8 public static void main(String args[]) {
9 // TODO Auto-generated method stub
10 Terminal.println("$ java Shell");
11 while (true) {
Bei der zweiten Style-Überprüfung kamen auch noch ein paar Verstösse:
GgT.java:1:1: Utility classes should not have a public or default constructor.
GgT.java:8:44: Array brackets at illegal position.
(Code: siehe oben)
Wenn mir jemand sagen könnte, was da ein verstoß sein soll, würde mich das echt weiterbringe....
Mfg Michi
habe gestern das erste mal den Praktomat getestet. Dabei hat er mir ein paar Styleverstösse gesenden, mit denen ich nichts anfangen kann.
Hier mal die Meldungen:
GgT.java:1: Missing a Javadoc comment.
GgT.java:8:36: Expected @param tag for 'args'.
bezieht sich auch die Zeilen:
1:
1 public class GgT {
2
3 /**
4 * @param args
5 */
8:
8 public static void main(String args[]) {
9 // TODO Auto-generated method stub
10 Terminal.println("$ java Shell");
11 while (true) {
Bei der zweiten Style-Überprüfung kamen auch noch ein paar Verstösse:
GgT.java:1:1: Utility classes should not have a public or default constructor.
GgT.java:8:44: Array brackets at illegal position.
(Code: siehe oben)
Wenn mir jemand sagen könnte, was da ein verstoß sein soll, würde mich das echt weiterbringe....
Mfg Michi
Re: Style-Verstoß
1. Auch die Klasse soll einen Javadoc-Kommentar bekommen.
2. fällt mit 4. zusammen: Du hast einen String[] args und nicht String args[] (Java ist ähnlich wie C, aber eben nicht genau wie)
3. Doofe Meldung die du loswerden kannst, indem du einfach einen Konstruktor anlegst, der nichts tut (oder eine Exception wirft). Siehe Terminal-Klasse als Beispiel. Musste ich auch erst ergooglen.
mfG
Markus
Edit: Noch ein Tipp - Installiert euch das Plugin Checkstyle, wenn ihr Eclipse verwendet, und stellt es dann auf die an Eclipse angepassten Java Code Conventions um. Den Autoformatter solltet ihr natürlich ebenfalls auf diesen Stil umstellen.
Der Praktomat verwendet Checkstyle intern für diese Überprüfungen, es ist also ein guter Anfang ...
2. fällt mit 4. zusammen: Du hast einen String[] args und nicht String args[] (Java ist ähnlich wie C, aber eben nicht genau wie)
3. Doofe Meldung die du loswerden kannst, indem du einfach einen Konstruktor anlegst, der nichts tut (oder eine Exception wirft). Siehe Terminal-Klasse als Beispiel. Musste ich auch erst ergooglen.
mfG
Markus
Edit: Noch ein Tipp - Installiert euch das Plugin Checkstyle, wenn ihr Eclipse verwendet, und stellt es dann auf die an Eclipse angepassten Java Code Conventions um. Den Autoformatter solltet ihr natürlich ebenfalls auf diesen Stil umstellen.
Der Praktomat verwendet Checkstyle intern für diese Überprüfungen, es ist also ein guter Anfang ...
Re: Style-Verstoß
so folgende meldungen kommen aber immer noch:
http://rapidshare.de/files/40778168/epp_result.html
ich glaube, ich habe all deine tipps befolgt... oder nicht??? was mach ich noch falsch???
michi
http://rapidshare.de/files/40778168/epp_result.html
ich glaube, ich habe all deine tipps befolgt... oder nicht??? was mach ich noch falsch???
michi
Re: Style-Verstoß
Mach GgT() mal protected.
Und schreibe hinter params auch noch, was das für Params sind ... deswegen die andere Meldung. Leerlassen ist nicht gewünscht.
Hast du das Checkstyle-Plugin mal installiert? Dann bekommst du eigentlich alle Fehler die hier bemängelt werden auch in Eclipse angezeigt.
Mich wundert es, dass er dir die Terminal-Klasse so ankreidet. Bei mir hat er die einfach ignoriert.
mfG
Markus
Und schreibe hinter params auch noch, was das für Params sind ... deswegen die andere Meldung. Leerlassen ist nicht gewünscht.
Hast du das Checkstyle-Plugin mal installiert? Dann bekommst du eigentlich alle Fehler die hier bemängelt werden auch in Eclipse angezeigt.
Mich wundert es, dass er dir die Terminal-Klasse so ankreidet. Bei mir hat er die einfach ignoriert.
mfG
Markus
Re: Style-Verstoß
Und lösche mal die Leerzeilen zwischen dem Javadoc-Kommentar und der Methode, die du kommentieren willst.
Re: Style-Verstoß
Hi,
Wenn man das style plugin installiert hat, findet man das irgend wo in der leiste oben oder so??? ich habe die Dateien mal in meinen plugin ordner geschoben, finde aber die funktion in eclipse nicht
micha
Wenn man das style plugin installiert hat, findet man das irgend wo in der leiste oben oder so??? ich habe die Dateien mal in meinen plugin ordner geschoben, finde aber die funktion in eclipse nicht
micha
Re: Style-Verstoß
Edit: Eclipse-Plugin-Link anstelle vom Projektlink eingefügt.
http://eclipse-cs.sourceforge.net/
Steht eigentlich alles gut erklärt, inklusive der Einrichtung.
mfG
Markus
PS: Es gibt noch ein weiteres Plugin, Checkclipse ... damit kenne ich mich aber nicht aus.
http://eclipse-cs.sourceforge.net/
Steht eigentlich alles gut erklärt, inklusive der Einrichtung.
mfG
Markus
PS: Es gibt noch ein weiteres Plugin, Checkclipse ... damit kenne ich mich aber nicht aus.
Re: Style-Verstoß
okay, konnte alle fehler korregieren. nur an einem häng ich noch.
was heißt denn
gruß fuku
was heißt denn
und wie fix ich das?'}' should be on the same line.
gruß fuku
Re: Style-Verstoß
meine glaskugel is grad kaputt... was steht in der zeile und was steht in der zeile davor/danach?
Re: Style-Verstoß
http://pastebin.com/m494469ea
Starting audit...
LCM.java:21:13: '}' should be on the same line.
LCM.java:25:13: '}' should be on the same line.
Audit done.
werd daraus nicht schlau, die klammern sind sauber gesetzt.
Starting audit...
LCM.java:21:13: '}' should be on the same line.
LCM.java:25:13: '}' should be on the same line.
Audit done.
werd daraus nicht schlau, die klammern sind sauber gesetzt.