Refaktoryzacja kodu.

Question

Bardzo proszę o rady jak mogę ten kod zrefaktoryzować

Oto kod : 

#include <iostream>
#include <iomanip>
#include <cstdlib>
#include <stdio.h>
#include <windows.h>

using namespace std;
int wybor;
float srednia, j, d, t, cz, p, sz;
int main()
{

for (;;)
   {system("cls");
    cout << "MENU GLOWNE" << endl;
    cout << "--------------" << endl;
    cout << "1.Srednia ocen" << endl;
    cout << "2.Wyjscie" << endl;

    cout << "Wybierz: " ;
    if (!(cin >> wybor))
    {
        cerr<<"To nie jest wlasciwa liczba! Wlacz program jeszcze raz";
        Sleep (3000);
    exit(0);

    }
system("cls");

    switch(wybor)
      {
      case 1:
        {cout << "Program policzy srednia ocen ucznia. \n";
        cout << "Podaj kolejno ilosc ocen, o ktore zapytam: \n";
        do
    {
        cout << "Ile jedynek ma uczen? ";
        cin >> j;
    }
    while( j < 0 );

    do
    {
        cout << "Ile dwojek ma uczen? ";
        cin >> d;
    }
    while( d < 0 );

    do
    {
        cout << "Ile trojek ma uczen? ";
        cin >> t;
    }
    while( t < 0 );

    do
    {
        cout << "Ile czworek ma uczen? ";
        cin >> cz;
    }
    while( cz < 0 );

    do
    {
        cout << "Ile piatek ma uczen? ";
        cin >> p;
    }
    while( p < 0 );

    do
    {
        cout << "Ile szostek ma uczen? ";
        cin >> sz;
    }
    while( sz < 0 );

system("cls");
    srednia =(( j * 1 ) +( d * 2 ) +( t * 3 ) +( cz * 4 ) +( p * 5 ) +( sz * 6 ) ) /( j + d + t + cz + p + sz );
    cout << "Srednia ocen tego ucznia wynosi: " << fixed << setprecision( 2 )<< srednia<<endl;
    cout << "Jesli chcesz obliczyc inna srednia wcisnij klawisz ENTER";
  getchar(); getchar();}
    break;
      case 2:
        exit(0);
        break;
        default:
        cout <<"Nieprawidlowa liczba!";

        break;}
   }
getchar(); getchar();

    return 0;
}

 

Answer 1

Radziłbym wykorzystać do tego tablice i wprowadzać dane w pętli.

for (i=0; i<6; i++)
{
        cout << "Ile " << (i+1) << " ma uczen? ";
        cin >> tablica[i];
}

Podobnie w pętli wyliczyć średnią.

Answer 2

Przede wszystkim nazywaj zmienne tak, żeby ich nazwa oddawała do czego służą.

+ wcięcia ładne zrób.

Answer 3

A ja bym chciał wiedzieć, co rozumiesz poprzez refaktoryzację? 

Comments

Sprowadzenie kodu do najprostszej postaci :)

Answer 4

jest to kalkulator średniej ocen,

Answer 5

Pokrótce:

1. Używanie using namespace std (ale to tylko moja uwaga)
2. Zmienne globalne
3. Nazwy zmiennych nie mówiące niczego konkretnego
4. Brak podziału programu na funkcje
5. Często powtarzające się bloki programu (czyli powinieneś użyć pętli i funkcji)
6. Podwójne użycie getchar (wiem, że pan Mirosław tak pokazywał, ale to nie jest odpowiednie rozwiązanie; powinieneś użyć wcześniej std::cin.sync - czyści bufor strumienia, z którego korzysta także getchar)

Comments

A jak mam napisane to using namespace std to czy muszę pisać std::cin.sync?

---

Nie. 12 znaków, blablabla