Odcinek 11 praca domowa C++ ocena kodu

Question

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/

Answer 1

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

Comments

Aha. Płatny.

---

Moje niedopatrzenie:

http://drzewniak.slupsk.pl/~ks/c/c_024.html

---

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

 

---

@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().

---

Ale mam na mysli efekt wizualny, ktory jest taki sam. Aczkowloiek dziekuje za zrodlo, zaraz przeczytam.