Seite 2 von 2

Re: Style-Verstoß

Verfasst: Fr 12. Dez 2008, 10:53
von Jonny
Auszug aus der Newsgroup:

Leonard Graf wrote:
> Hallo zusammen,
>
> ich habe heute im Zuge der Aufgabe 5 auf dem Übungsblatt Nr. 3 die Aufgabe 0 im Praktomat gelöst.
> Das Programm scheint soweit in Ordnung zu sein, nur meckert Praktomat bei der CheckStyle Prüfung rum, und zwar wie folgt:
>
>
> Prüfung 5: CheckStyle (1/2)
> Ihr Programm hat diese Prüfung nicht bestanden:
>
> Starting audit...
> GgT.java:25:26: '}' should be on a new line.
> GgT.java:25:26: '}' should be on the same line.
> GgT.java:30:28: '}' should be on a new line.
> GgT.java:30:28: '}' should be on the same line.
> Audit done.
>
>
> Meiner Meinung nach widersprechen sich die Aussagen jeweils...

Ich glaube, die widersprechen sich nicht, sondern "new" und "same" beziehen sich jeweils auf Code in anderen Zeilen.
Du hast an der Stelle etwas dieser Form geschrieben:

if (condition) { statement; }
else if (......)

Das "}" sollte vom "if" aus betrachtet auf eine _neue_ Zeile, aber es sollte auf die _gleiche_ Zeile wie das else. Also so etwa:

if (condition) {
statement;
} else {
statement;
}

Probier mal das und guck, ob das Problem weiterhin besteht.
[...]

Re: Style-Verstoß

Verfasst: Fr 12. Dez 2008, 15:14
von lupus
Dein Problem war das "else", das muss mit auf die Zeile vom "}" also

Code: Alles auswählen

} else {
also komplette Syntax:

Code: Alles auswählen

if (bedingung) {
..
} else if (bedingung) {
...
} else {
...
}
Wenn auf eine Bedingung nur eine Anweisung folgt, dann kannst du die Klammern auch weglassen.

Code: Alles auswählen

if (bedingung)
anweisung;

weiter_im_code...;

Re: Style-Verstoß

Verfasst: Mo 15. Dez 2008, 15:45
von fuku
ah okay, danke lupus