Czy ten kod jest "czysty" ?

Question

Witam, jestem nowy na forum, a także nowy w programowaniu. Posłuchałem podcast Pana Mirosława dotyczący błędów w programowaniu i stwierdziłem, że jakoś muszę wkręcić się w to forum i udostępniać swoje kody oraz patrzeć na cudze. Dlatego zakładam ten post, aby się (tak jak wspomniałem) jakoś wkręcić .

Mam oczywiście pytanie, a mianowicie, czy ten kod da się jeszcze ulepszyć ? Czy należy inaczej układać zmienne, nie robić/robić spacje między różnymi znakami. Po prostu jakie są takie przynajmniej podstawowe zasady, żeby kod był schludny i tak jakby "uniwersalny", czyli łatwy w odczytaniu dla każdego.

Program liczy średnią z ocen. (Tak, dopiero jestem po kursie tablic )

#include <iostream>
#include <stdio.h>

using namespace std;

int main()
{

    int liczba_wyrazow; float suma=0; float srednia;
    cout << "Witaj, bedziemy liczyc srednia ocen!"<<"\a" << endl;
    cout<<endl<<"Wprowadz liczbe wszystkich przedmiotow:";
    cin>>liczba_wyrazow;

    float ocena[liczba_wyrazow];

    cout<<endl << "Wprowadz swoje oceny:"<<"\a"<<endl;

        for(int i=0; i<liczba_wyrazow; i++)
    {
        cout<<"Ocena "<< i+1 <<": "<<"\a";
        cin >>ocena[i];
        suma+=ocena[i];
    }
    cout<< "Suma rowna sie: "<<"\a"<<suma<<endl;

    srednia=suma/liczba_wyrazow;

    cout<<"srednia rowna sie: "<<"\a" <<srednia<<endl;

    getchar(); getchar();

    return 0;
}

P.S Jeżeli takie posty nie powinny się tu pojawiać, to proszę, żeby ktoś mnie o tym powiadomił i ewentualnie pomógł się tu jakoś zaaklimatyzować . Nie znalazłem na pierwszy rzut oka jakiegoś osobnego postu dla amatorskich kodów, żeby pooglądać, a tych bardziej skomplikowanych po prostu jeszcze nie rozumiem.

Pozdrawiam

Comments

Rozmiar tablicy musi być wartością stałą.

Druga sprawa, nie używa się jednocześnie iostream i stdio. Musisz się na coś zdecydować.

---

Ale ... ta tablica tak jakby ma wartość stałą, bo jej wartość nada użytkownik, i potem jej już nie zmieni. Program działa, więc chyba można tak zrobić. Proszę o jakieś głębsze wytłumaczenie tego.

 

Czy jeśli zrobię tylko stdio to wszystko będzie działało? W kursie pana Mirosława dołączanych jest wiele bibliotek.

Dziękuję za odpowiedż!

---

Twoje rozwiązanie dotyczące zainicjowania tablicy jest nie do końca poprawne.

W takich przypadkach należy użyć dynamicznej alokacji pamięci:

int liczba_wyrazów;

cin >> liczba_wyrazów;

float *ocena = new float[liczba_wyrazów]; //dynamiczna alokacja pamięci

/*
...
*/
//Gdy już nie potrzebujesz tablicy ocena robisz:

delete [] ocena;

 

---

Na twoim miejscu zdecydowałbym się na iostream, a wywalił stdio. Zamiast getchar można użyć cin.get()

---

> Twoje rozwiązanie dotyczące zainicjowania tablicy jest nie do końca poprawne.

A dlaczego? Kiedyś nie działało, ale od jakiejś tam wersji standardu weszło do standardu i czemu ma nie używać? Chyba że zakłada, że będzie przenosił kod na kompilatory nie obsługujące tego zapisu.

---

A dlaczego? Kiedyś nie działało, ale od jakiejś tam wersji standardu weszło do standardu i czemu ma nie używać? 

Na pewno nie standardu C++. VLA zostało wprowadzone do C wraz ze standardem C99 a od C11 jest opcjonalne. g++ ma rozszerzenie, które wprowadza obsługe VLA i dlatego to się kompiluje. Na każdym innym kompilatorze zgodnym ze standardem C++ ten kod nie powinien się skompilować.

---

Ciekawe skąd taki opór przed tym zapisem, przecież to całkiem przyjemne. Zwłaszcza w czasach mody na języki typu Python, Swift itp. powinni wychodzić na przeciw takim rzeczom, a nie je wycofywać.

---

Tu chodzi o niebezpieczeństwo takiego zapisu. Przy VLA jest bardzo łatwo o stack overflow. Jeśli chcesz mieć tablice o rozmiarze nieznanym w czasie kompilacji użyj vectora/smart pointer'ów. W przypadku zbyt dużego rozmiaru zostanie rzucony wyjątek i bedziesz mógł z tym coś zrobić.

---

Dzięki wielkie !

Answer 1

Pierwsze co przychodzi mi do głowy to zmienne/metody/klasy itp. zapisuj po angielsku. Polecam "Czysty kod", jeszcze całego nie przerobiłem, ale już widzę że książka jest bardzo dobra.

Answer 2

• Nazywaj zmienny po angielsku
• Rozdzielaj sobie operatory spacjami 

  cout << "Wszystkiego najlepszego i smacznego jajka" << endl;
  

• Nie deklaruj zmiennych w jednej lini, oraz deklaruj je odrazu pod klamerką main'a.
• Kod sformatował bym tak:

  #include <iostream>
  #include <stdio.h>
  using namespace std;
   
  int main()
  {
      int liczba_wyrazow; 
  	float suma = 0, srednia
  	
  		cout << "Witaj, bedziemy liczyc srednia ocen!" << "\a" << endl;
  		cout << endl << "Wprowadz liczbe wszystkich przedmiotow:";
  		
  		cin >> liczba_wyrazow;
   
   
  		float ocena[liczba_wyrazow];
  	 
  	 
  		cout << endl << "Wprowadz swoje oceny:" << "\a" << endl;
  	 
  	 
  		for(int i=0; i<liczba_wyrazow; i++)
  		{
  			cout << "Ocena " << i+1 << ": " << "\a";
  			cin >> ocena[ i ];
  			suma += ocena[ i ];
  			
  		}
  		
  		
  		cout << "Suma rowna sie: " << "\a" << suma << endl;
  	 
  
  		srednia = suma / liczba_wyrazow;
  	 
  	 
  		cout << "srednia rowna sie: " << "\a" << srednia << endl;
  	 
  	 
  		getchar(); getchar();
   
   return 0;
  }

  Pozdrowionka smacznego jajka, wszystkiego najlepszego, wesołych świąt z okazji oczywiście nowego roku. :3
  
  EDIT: Wypowiadam się tylko na temat czytelności kodu. :)

Comments

przestrzen nazw to ZŁO

---

for (int i = 0; i < liczba_wyrazow; i++)

Sam Visual Studio rozdziela spacjami automatycznie takie wyrażenia... Poza tym po co dwa getchar(); na końcu?

---

Wypowiedziałem się tylko na temat czytelności kodu, koledzy wyżej zajeli się resztą. :)

---

Visula przerywa prace odrazu na koncu, getchary to przeciagaja, tak to dziala w konsoli w sfml nie trzeba tego uzywac

---

Jeśli chodzi o deklarację zmiennych, to odwołam się do książki "Czysty kod". Autor zaleca deklarowanie zmiennych jak najbliżej miejsca ich użycia. Chociaż też nie ma co kurczowo trzymać się tej zasady. Jeśli uznasz, że lepszym jest deklarowanie ich bezpośrednio pod nagłówkiem funkcji, to zrób tak.

Pozdrawiam

Answer 3

Nie zauważyłem żadnych wykroczeń ;D