[s.....a]

zrobiłem pierwszą wersję kalkulatora i wrzuciłem ją tu https://github.com/slucham-psa-jak-gra/Kalkulator
możecie sprawdzić kod czy jest ok?
to jest taka pierwsza wersja i będę ją rozwijał

#programowanie #java #robiekalkulator

[sorhu]

@slucham-psa-jak-gra:
Dodaj .idea do .gitignore.
Zrób jakiś podział na metody.
Long jest ok, ale do dzielenia się nie nadaje. 1/2 = 0.

[Darjo]

@slucham-psa-jak-gra: jak wspomnieli poprzednicy - typy calkowite sie nie nadaja, poza tym juz tylko analiza leksykalna wyrazen, ttworzenie drzewa operacji, wykonanie ich i voila, kalkulator gotowy :) (zerknij na np http://www.dobreprogramy.pl/Piszemy-troche-mniej-zlozony-kalkulator,Blog,52021.html ) - jesli kalkulator ma byc uzyteczny i rozwijajacy to idz ta droga :)

[t0fu]

@sorhu: .gitignore wydaje mi się spoko, jest tak wygenerowany by pousuwać elementy zależne od platformy programisty.

Ja bym podbił sourceCompatibility w gradlu do 1.8.

Kolejnym usprawnieniem jakie bym zrobił w twoim kodzie to rozbiłbym maina na kilka funkcji o niezależnych zadaniach np:
- String wczytajLinie() // Scanner może być póki co jako stała statyczna wartość w klasie
- Operacja parsujLinie(String linia) // gdzie Operacja to nowa klasa zawierająca (n1, operator, n2)

[y.....n]

To co poprzednicy, plus: Poczytaj o konwencjach nazweniczych pakietowi zmien je odpowiednio. Te 4 ify mozna zamienoc na wzorzec projektowy strategia.

[sorhu]

@t0fu: Tylko te nazwy to nie po polsku.

[y.....n]

Do fego takie rzeczy to BigDecimal i troche pozmieniaj nazwy zmiennych na wiecej mowiace

[razornfs]

@slucham-psa-jak-gra: błąd dla 1+1, lol

[s.....a]

@razornfs: bo napisałeś bez spacji, powinno być 1 + 1

[razornfs]

@slucham-psa-jak-gra: nikt tak nie pisze, nie powinieneś wymagać żeby użytkownik wpisywał input w jeden konkretny sposób. parę wskazówek:

- ignoruj białe znaki (spacje, tabulatory itd.) w inpucie np. 1+1, 1 + 1, 1+ 1; dają ten sam wynik
- co jeśli użytkownik wpisze niepoprawny input? np. "1+", narazie zakładasz że input ma co najmniej 3 znaki
- używaj doubli zamiast long

[Patres]

@slucham-psa-jak-gra zrób zabezpieczenie i daj string.trim() usuwa Ci wtedy białe znaki.

[razornfs]

@Patres: trim usuwa białe znaki z początku i końca, nie ze środka

[razornfs]

@slucham-psa-jak-gra: możesz też wprowadzić kolejność działań, linki do poczytania:

https://en.wikipedia.org/wiki/Shunting-yard_algorithm
https://en.wikipedia.org/wiki/Reverse_Polish_notation

[s.....a]

@GuardianOfTheBlind: @FedoraTyrone: @sorhu: @Darjo: @t0fu: @yelen: @razornfs: @Patres:
pojawiła sie nowa wersja w repozytorium, za chwile zaczne myśleć nad tą odwrotną notacją i jak to zrobić

[s.....a]

@GuardianOfTheBlind: @FedoraTyrone: @sorhu: @Darjo: @t0fu: @yelen: @razornfs: @Patres:
no i dzięki za wskazówki

[razornfs]

@slucham-psa-jak-gra: dalej nie działa dla 1+1 ;)

[sorhu]

@slucham-psa-jak-gra:
throw new CalculatorException(new IllegalArgumentException("Illegal number of tokens"))
Usuń ten illegal argument. Nie złapałeś tego wyjątku. Nie jest więc on tu potrzebny.

Po prostu:
throw new CalculatorException("Illegal number of tokens")

A teraz.
Wyrzuć if else przy operatorach.
Użyj switcha.

A jeszcze lepiej:
Stwórz interfejs reprezentujący operację. Ma przyjąć dwa double i zwrócić wynik.

Stwórz implementacje.

Stwórz klasę, która zamienia string (operator) na operację.

[s.....a]

@sorhu: update https://github.com/slucham-psa-jak-gra/Kalkulator

[sorhu]

@slucham-psa-jak-gra: Wyrzuć to .idea z repo. ( ͡° ͜ʖ ͡°)