AlessandroCogollo / SE1-Project-2022

Software Engineering Project - Politecnico di Milano - AA2021/22
2 stars 0 forks source link

main() has too many lines #8

Closed ingconti closed 2 years ago

ingconti commented 2 years ago

pls slit them in functionalities, You ha ve a bunch of if.. else .. else... if....

public static void main(String[] args) {
luca-botti commented 2 years ago

They are used only for the various possiblity of the cli parser. Do you mean to use other algorithm to choose between all the possibility, or to decrease the number of them?

ingconti commented 2 years ago

I simply say to break them down in separate methods, to better debug / read / modify /maintain.

luca-botti commented 2 years ago

Ok thanks, sorry when i write the previous comment i haven't seen yet #11. I will change it.

luca-botti commented 2 years ago

fixed