Czy takie coś jest dopuszczalne:
lista.Add(Metoda(double.Parse(tablica[...].Substring(...), CultureInfo.InvariantCulture)));
Czy rozbić/refaktoryzować taki twór?
Czy takie coś jest dopuszczalne:
lista.Add(Metoda(double.Parse(tablica[...].Substring(...), CultureInfo.InvariantCulture)));
Czy rozbić/refaktoryzować taki twór?
Ja bym wyciągnął przynajmniej double.Parse
do oddzielnej linijki, będzie czytelniejsze i łatwiejsze w debugowaniu.
Potwierdzam!
Dzięki. Początkujący takich rzeczy nie wyłapuje, ja chciałem to bardziej rozbijać i kombinować. Jednak takie proste rady bardzo dużo dają.
Ja się staram tak pisać kod, że jak później go czytam to od razu rozumiem po kolei linijki i całość układa mi się w głowie. Staram się nie dopuszczać sytuacji w których muszę myśleć minutę co się właśnie dzieje w danej linijce.