access modifiers

[trez]

Hallo zusammen.

Ich bin in unserem Code hier auf lokale Klassen und Methoden ohne modifier gestossen.
Hier wird schon klar genug gezeigt was das ist aber in unseren Codingguidelines steht sinngemäss "keine Deklaration ohne modifier" - sollen/können wir den Punkt rausschmeissen? (Stammt vielleicht noch aus der Vor-Java-Zeit )


Wird das wirklich bewusst eingesetzt? Mich interessiert einfach eure Meinung.

trez

 

[maki]

Wird das wirklich bewusst eingesetzt?

Manchmal, ohne modifier wird das dann "package-private" genannt.

 

[MySelV]

Hi,

die Frage ist:
- Gehst du nur den Code durch? -> Da fehlt ein Modifier, der Code entspricht nicht den Guidelines.
- Sollst du die Guidelines überarbeiten? -> Abstimmung mit den Entwicklern, ob inzwischen der AccessModifier weggelassen werden soll.

Persönliche Meinung ist, dass man den Modifier schon weglassen kann, wenn man tatsächlich Package Sichtbarkeit haben möchte.

Grüße

 

[trez]

Immer wenn alter Code (der wurde 2000 zuletzte modifiziert) verändert wird, werden auch solche Sachen überprüft.

Konkret:
Bei Klassen, auch den lokalen, hat es nie Modifier
Bei Konstruktoren hat es sehr oft keine Modifier

Was ich aber wirklich schlimm finde sind Attribute ohne Modifier - das ist ein Tabuthema und wird erbarmungslos geändert -> privat oder protected - mehr gibt es da nicht

Die im Moment noch Grippe-"toten" werden dann schon sehen was ich "gebastelt" habe - die nehmen zum Glück dann auch kein Blatt vor den Mund wenn ihnen was nicht passt

 

[maki]

Grundsätzlich gilt, dass die Sichtbarkeit so klein wie möglich, so groß wie nötig ist.

Kann schon Sinn machen wenn die Alternative public lautet, kann man nicht pauschal beurteilen, sowas machen imho Leute die sehr neu in der Materie sind oder Leute, die genau wissen was sie machen.