search

averbraeck / opentrafficsim

Open Source Multi-Level Traffic Simulator
BSD 3-Clause "New" or "Revised" License
28 stars 8 forks source link

Remove xxxSI() methods and use xxx().si instead #75

Closed WJSchakel closed 1 year ago

WJSchakel commented 1 year ago

There are quite a number of methods that are of the form:

double distanceSI()

Next to:

Length distance()

The former can be removed, and usage can be shifted to the other using:

distance().si

To reduce code and increase consistency, this should be implemented.

WJSchakel commented 1 year ago

Using the pattern (?<!(get|instantiate))SI\( all methods ending with SI not being getSI() or instantiateSI() were found. Those that only return SI values have been removed. Methods that receive SI values have been maintained.