Prosiłbym o ocenę kodu programu do rozwiązywania równań. Zaimplementowałem na razie tylko liniowe i kwadratowe. Program parsuje proste równania np. 3*2x+4+4=5x-1*2i zwraca wynik -10.0.
Nie mam dużego doświadczenia w Javie, więc nie znam dobrych praktyk. Nie testowałem jakoś bardzo kodu, więc mogą być jakieś błędy.
Dobrze zrobiłem umieszczając klasę Operator w tym samym pliku co klasa RPN? Czy powinna to być osobna, niezależna klasa?
Mam też pytanie co do unit testów. Czy prywatne metody też powinny być jakoś testowane? Bo czasem zastanawiałem się, czy nie zrobić jakieś metody publicznej tylko dla testów.
https://github.com/jerylee/equation-solver
0
0
Nikt nie skomentuje? Czyli jest tak dobrze (raczej nie) albo tak źle, że nikomu się nie chce przeglądać kodu :D
0
- Brak komentarzy w kodzie.
- Dziwnie poustawiane pola - powinny znajdować się przed metodami, a nie między / po.
- Konstruktor zawsze na początku klasy.
Oprócz tego znacznie ciekawiej by było, gdybyś: - Napisał parser, zamiast bawił się w odwrotną notację polską.
- Dodał możliwość obliczeń na (potencjalnie) nieskończenie wielkich liczbach, na przykład wykorzystując GMP (jeśli jest na Javę) bądź też implementując to samemu, tak aby mieć możliwość wyświetlenia rozwiązania jako przykładowo
sqrt(3) + 1zamiast2.73.