search

rrickfox / traffic-sim

GNU General Public License v3.0
2 stars 0 forks source link

void SetVariable() ist keine guter Namen für ne Methode die keine kein Setter von 'Variable' ist #84

Closed funketh closed 4 years ago

funketh commented 4 years ago

Sagt einem nämlich nichts außer "Irgendwas passiert mit der Variable".

Falls die Methode nur im Constructor genutzt wird wäre sowas wie InitializeVariable() gut.

Andernfalls sind es meist irgendwelche methoden die etwas updaten um Änderungen anderer Werte zu übernehmen. In dem Fall sollte UpdateVariable() genutzt werden denke ich.

funketh commented 4 years ago

Meine jetzt spezifisch:

Car.SetPosition()
EndPoint.SetWeights()

Außerdem wollte ich das generell ansprechen

rrickfox commented 4 years ago

nunja bei EndPoint.SetWeights() werden die RouteProbabilities zum ersten Mal gesetzt, was einem Setter ja ziemlich gleichkommt. Dabei wäre CalculateRouteProbabilities() vielleicht noch passend. Car.SetPosition() könnte aber in UpdatePosition() umbenannt werden.

funketh commented 4 years ago

Würde deinem ersten Punkt nicht zustimmen, Setter haben eine klare Bedeutung im Kontext von Java-ähnlichen Sprachen und akzeptieren auch immer einen Wert, zu dem die betreffende Variable gesetzt werden soll. (Mal davon abgesehen setzt die Methode nicht die weights). CalculateRouteProbabilities() ist ok

rrickfox commented 4 years ago

37c00bc