ohsome-api icon indicating copy to clipboard operation
ohsome-api copied to clipboard
verified publisher icon GIScience

Rename and restructure classes and packages

Open bonaparten opened this issue 4 years ago • 2 comments

A general review of all classes' and packages' names and structures would be useful.

For example, AggregateRequestExecutor.java and DataRequestExecutor.java could be renamed to AggregationRequestExecutor.java and ExtractionRequestExecutor.java. So these classes will follow the same naming criteria used in other packages, i.e. in the package output: DefaultAggregationResponse.java and ExtractionResponse.java .

bonaparten avatar Feb 16 '21 11:02 bonaparten

I agree, it would definitely make sense to review again the current structures and names. After we've merged https://github.com/GIScience/ohsome-api/pull/124 and https://github.com/GIScience/ohsome-api/pull/121 we can think about which other classes and packages would need to be renamed/restructured.

FabiKo117 avatar Feb 16 '21 13:02 FabiKo117

I pushed this old commit. It is an example of how we can restructure some packages and classes.

bonaparten avatar Jul 29 '21 09:07 bonaparten