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

Odcinek 11 praca domowa C++ ocena kodu

VPS Starter Arubacloud
0 głosów
170 wizyt
pytanie zadane 7 stycznia 2017 w C i C++ przez Arkadiusz Kruk Nowicjusz (120 p.)
Witam wklejam tutaj swoja pracę domową z odcinka 11 kursu C++. Może ktoś ocenić? Wiem, że kod nie jest idealny i na pewno można go bardzo skrócić ale starałem się posługiwać tylko tym co do tej pory umiem. W internecie jedynie wyszukiwałem jak obliczyć wartość bezwzględną, pozdrawiam :)

PS. Zadanie polegało na stworzeniu programu, w którym będzie trzeba wpisać 5 liczb, a program wyliczy z nich średnią i pokaże liczbę (bądź 2 liczby) najbliższe średniej.

http://wklej.org/id/3008620/

1 odpowiedź

0 głosów
odpowiedź 7 stycznia 2017 przez playerony Pasjonat (15,240 p.)
Już mniejsza z tym skracaniem kodu, to co najbardziej rzuca się w oczy i to co jest złym nawykiem to nazywanie zmiennych "a", "b", itd., już estetyczniejszym rozwiazaniem będzie zastosowanie tablicy po czym kod od razu zyska na tym, że będzie nie tyle co czytelniejszy, ale również skalowalny. Na dodatek czemu są to zmienne globalne ? Tu masz małą lekture na temat zmiennych globalnych oraz lokalnych.

http://www.ithelpdesk.pl/kurs-c++/poziom-3/zasieg-zmiennych-w-c++-zmienne-globalne-zmienne-lokalne.html
komentarz 7 stycznia 2017 przez niezalogowany
Aha. Płatny.
komentarz 7 stycznia 2017 przez playerony Pasjonat (15,240 p.)
komentarz 7 stycznia 2017 przez DragonCoder Nałogowiec (36,500 p.)

Ja dodam, ze czytelnosc kodu moglbys takze poprawic, tak jak up mowi, usun globalne uzyj lokalnych, dodatkowo mozesz usunac using namespace i pisac std::, co jeszcze mi sie rzuca w oczy to: endl; uzyj zamiast tego na koncu jeszcze w cudzyslowiu \n, dziala tak samo i wyglada jak dla mnie lepiej.

if (e1<n)
    {
        n=e1; m=e;
    }

zrob tego entera zeby bylo w nowej lini, bardziej czytelne bedzie

 

komentarz 7 stycznia 2017 przez niezalogowany

@krrr123 endl nie działa do końca tak samo jak "\n":

http://en.cppreference.com/w/cpp/io/manip/endl

Inserts a newline character into the output sequence os and flushes it as if by calling os.put(os.widen('\n')) followed by os.flush().

komentarz 7 stycznia 2017 przez DragonCoder Nałogowiec (36,500 p.)
Ale mam na mysli efekt wizualny, ktory jest taki sam. Aczkowloiek dziekuje za zrodlo, zaraz przeczytam.

Podobne pytania

0 głosów
1 odpowiedź 150 wizyt
pytanie zadane 13 października 2016 w C i C++ przez KamilGG Użytkownik (690 p.)
0 głosów
1 odpowiedź 156 wizyt
0 głosów
1 odpowiedź 435 wizyt
pytanie zadane 26 stycznia 2018 w C i C++ przez jezyk2796 Nowicjusz (150 p.)

92,453 zapytań

141,262 odpowiedzi

319,088 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!

...