[Code Review] Aplikacja WinForms

Question

Cześć wszystkim! Prosiłbym jedynie o codeReview, na pewno można coś w nim poprawić i wiem że do niektórych metod nie ma jeszcze testów jednostkowych trochę się z ich implementacją pośpieszyłem.

Cały kod na githubie: https://github.com/Slaw145/TomsadApplication

Answer 1

Kwestia upodobań lecz większość osób zgadza się ze ify nawet jedno linijkowe powinny mieć klamerki. Problem jest na tyle popularny ze w PHP powstał nawet standard mówiący ze każdy if ma mieć klamerki. Twoje nie mają. 

https://github.com/Slaw145/TomsadApplication/blob/master/CodeFromWinFormsApplication/Form1.cs#L37

Masz sporo nie używanych funkcji.

https://github.com/Slaw145/TomsadApplication/blob/master/CodeFromWinFormsApplication/Form1.cs#L135-L153

https://github.com/Slaw145/TomsadApplication/blob/master/CodeFromWinFormsApplication/Form1.cs#L240-L258

Nie wiem jak powinna wyglądac aplikacja ale może jesteś w stanie wrzucić kontrolki do wspólnego panelu lub groupBoxa ? Wtedy bedziesz miał znacznie mniej kodu tutaj ;)

https://github.com/Slaw145/TomsadApplication/blob/master/CodeFromWinFormsApplication/Form1.cs#L277-L296

Ta funkcja skoro nic nie zwraca powinna być chyba prywatna ;) 

https://github.com/Slaw145/TomsadApplication/blob/master/CodeFromWinFormsApplication/MessgeBoxSaveAndRead.cs#L40

Trochę tego kodu jest a ja się w sumie czepiam tylko drobnostek myślę ze wyszło ci całkiem całkiem ;)

Comments

Dzięki za rady postaram się poprawić co do nieużywanych funkcji to one będe później używane ale masz rację powinienem je jakoś oddzielić