• Najnowsze pytania
  • Bez odpowiedzi
  • Zadaj pytanie
  • Kategorie
  • Tagi
  • Zdobyte punkty
  • Ekipa ninja
  • IRC
  • FAQ
  • Regulamin
  • Książki warte uwagi

[Projekt] Kalkulator

VPS Starter Arubacloud
0 głosów
697 wizyt
pytanie zadane 5 czerwca 2018 w Nasze projekty przez bicnet Gaduła (4,800 p.)
Cześć,

stworzyłem prosty kalkulator w html,css + JS, chciałbym żebyście ocenili, ewentualnie jak jeszcze da się ulepszyć to proszę napisać, na pewno się zainteresuję i najwyżej rozbuduję aktualną wersję. Co prawda nie jest mój to pierwszy projekt w życiu, mam jeszcze kilka innych na dysku, ale będę je publikować na GitHub'ie , tylko muszę je wykończyć.

Kalkulator: https://kosek1.github.io/calculator/

Repozytorium GitHub: https://github.com/kosek1/calculator

 

Pozdrawiam
komentarz 5 czerwca 2018 przez manjaro Nałogowiec (37,390 p.)
Github R.I.P.
komentarz 5 czerwca 2018 przez Tomek Sochacki Ekspert (227,510 p.)
Nie dramatyzowałbym aż tak :) a już na pewno nie w projektach typowo edukacyjnych...
komentarz 5 czerwca 2018 przez manjaro Nałogowiec (37,390 p.)
Ludzie już masowo uciekają z Githuba

3 odpowiedzi

+3 głosów
odpowiedź 5 czerwca 2018 przez Tomek Sochacki Ekspert (227,510 p.)
wybrane 6 czerwca 2018 przez bicnet
 
Najlepsza

Kilka małych uwag:

  1. może warto pozostawić ostatnie wprowadzenie (ostatnie działanie) np. nad wynikiem, to ułatwia pracę z kalkulatorem,
  2. dzielenie przez zero wg mnie nie powinno dawac Infinity bo typowemu userowi nic to nie mówi. Lepiej dać jakiś błąd bardziej czyteny.
  3. zrób sobie 1 dzielone przez zero, dostajesz Infinity, następnie pomnóż to przez 1 i dalej mamy Infinity, mając to "Infinity" dopisz jedynkę - mamy "Infinity1" i pomnóż przez jeden i mamy błąd w konsoli "Uncaught ReferenceError: Infinity1 is not defined"
  4. Nie używaj funkcji eval, widzę że często kusi ona wiele początkujących osób właśnie w kalkulatorze itp. ale to zly nawyk. Jeśli na prawdę chciałbyś zrobić kalkulator to wg mnie warto pomyśleć np. nad ONP albo chociaż jakoś porządnie zaprojektować te podstawowe działania + np. ikonka na pierwiastek, potęgę itp.
  5. Do tego przy tym rozwiązaniu pamiętaj, że wszystko działa jeśli poruszamy się w tzw. bezpiecznym zakresie liczb JS... przy bardzo dużych liczbach trzeba by obliczenia wykonywać na stringach - jak będziesz zainteresowany to poczytaj sobie w necie jest mnóstwo przykładów w wielu językach na dodawanie, mnożenie, dzielenie, odejmowanie, modulo itp.
  6. Kodu nie za bardzo jest co oceniać bo głównie bazuje on na funkcji eval() także spróbuj przepisać to na pojedyncze funkcje, może nawet warto pomyśleć nad jakimś wzorcem np. command itd.
  7. Zauważylem tez mały bląd w obliczeniach na liczbach zmiennoprzecinkowych... zrób sobie 2,8 - 3 i zobacz co dostajesz :)

 

 

0 głosów
odpowiedź 5 czerwca 2018 przez Mariusz08 Maniak (62,300 p.)
Projekt fajny prosty. Jedyne czego mi w nim brakuje to możliwości wpisywania liczb z klawiatury oraz innych działań (np. pierwiastkowanie) czy stałych (e, pi).

 

Co do kodu trudno ocenić - mała ilość - ale na pewno musisz zadbać o formatowanie w pliku js, zamiast onClick używaj addEventListener.
0 głosów
odpowiedź 5 czerwca 2018 przez CodeOwsky Początkujący (290 p.)

To chyba nie jest to czego użytkownik by oczekiwał :D

........................2+2.....................

Poza tym wszystko fajnie, moi poprzednicy napisali Ci wszystko co należało napisać :D

Pzdr :D

komentarz 5 czerwca 2018 przez Tomek Sochacki Ekspert (227,510 p.)
hehe a na to nie wpadłem analizując apkę :D

To jednak wszystko wina funkcji eval(), bo generalnie na niej bazuje cała aplikacja i nie robi walidacji.
komentarz 5 czerwca 2018 przez CodeOwsky Początkujący (290 p.)
Mały błąd :D Zdarza się :D Niemniej - projekt jest świetny :)

Powodzenia :D

Podobne pytania

+4 głosów
1 odpowiedź 607 wizyt
pytanie zadane 12 listopada 2021 w Nasze projekty przez qax Dyskutant (8,060 p.)
+5 głosów
1 odpowiedź 189 wizyt
pytanie zadane 7 czerwca 2021 w Nasze projekty przez beczkas Nowicjusz (170 p.)
+1 głos
3 odpowiedzi 1,204 wizyt

92,455 zapytań

141,263 odpowiedzi

319,100 komentarzy

61,854 pasjonatów

Motyw:

Akcja Pajacyk

Pajacyk od wielu lat dożywia dzieci. Pomóż klikając w zielony brzuszek na stronie. Dziękujemy! ♡

Oto polecana książka warta uwagi.
Pełną listę książek znajdziesz tutaj.

Akademia Sekuraka

Akademia Sekuraka 2024 zapewnia dostęp do minimum 15 szkoleń online z bezpieczeństwa IT oraz dostęp także do materiałów z edycji Sekurak Academy z roku 2023!

Przy zakupie możecie skorzystać z kodu: pasja-akademia - użyjcie go w koszyku, a uzyskacie rabat -30% na bilety w wersji "Standard"! Więcej informacji na temat akademii 2024 znajdziecie tutaj. Dziękujemy ekipie Sekuraka za taką fajną zniżkę dla wszystkich Pasjonatów!

Akademia Sekuraka

Niedawno wystartował dodruk tej świetnej, rozchwytywanej książki (około 940 stron). Mamy dla Was kod: pasja (wpiszcie go w koszyku), dzięki któremu otrzymujemy 10% zniżki - dziękujemy zaprzyjaźnionej ekipie Sekuraka za taki bonus dla Pasjonatów! Książka to pierwszy tom z serii o ITsec, który łagodnie wprowadzi w świat bezpieczeństwa IT każdą osobę - warto, polecamy!

...