[CR] C++ (#9): Własne funkcje. Podejście proceduralne

Question

CR = Code Review. O co chodzi? Zajrzyj tutaj
Pełna lista wszystkich Code Review? Zajrzyj tutaj

https://www.youtube.com/watch?v=HHplT1A4_A4

Kod z odcinka:

#include <iostream>

using namespace std;

float metry;

float ile_cali(float m)
{
   return m*39.37;
}

float ile_jardow(float x);

void ile_mil(float m)
{
   cout<<"na mile: "<<m*0.0006213;
}

int main()
{
    cout << "Podaj ile metrow: ";
    cin>>metry;

    cout<<"na cale: "<<ile_cali(metry)<<endl;
    cout<<"na jardy: "<<ile_jardow(metry)<<endl;
    ile_mil(metry);

    return 0;
}

float ile_jardow(float x)
{
   return x*1.0936;
}

Paczka z odcinka: POBIERZ​

Answer 1

Musiałem obejrzeć 2 razy, żeby zrozumieć. Wydaje mi się, że przykład jest zbyt zawiły i gdyby wyglądał tak:

#include <iostream>

using namespace std;

float metry;

//FUNKCJE
float ile_cali(float x)
{
    return x*39.37;
}

float ile_jardow(float x)
{
    return x*1.0936;
}

float ile_mil(float x)
{
    return x*0.0006213;
}

//PROCEDURY
void ile_cali_proc(float x)
{
    cout<<"na cale: "<<x*39.37<<endl;
}

void ile_jardow_proc(float x)
{
    cout<<"na jardy: "<<x*1.0936<<endl;
}

void ile_mil_proc(float x)
{
    cout<<"na mile: "<<x*0.0006213<<endl;
}


int main()
{
    cout<<"KONWERSJA PRZY UZYCIU FUNKCJI"<<endl;
    cout << "Ile metrow: ";
    cin>>metry;
    cout<<"na cale: "<<ile_cali(metry)<<endl;
    cout<<"na jardy: "<<ile_jardow(metry)<<endl;
    cout<<"na mile: "<<ile_mil(metry)<<endl;
    cout<<endl;

    cout<<"KONWERSJA PRZY UZYCIU PROCEDUR"<<endl;
    ile_cali_proc(metry);
    ile_jardow_proc(metry);
    ile_mil_proc(metry);
    return 0;
}

to przerobienie tego odcinka zajęłoby mi połowę mniej czasu.

Po pierwsze proponowałbym zrobić cały konwerter w dwóch wersjach: na funkcjach i na procedurach. Po drugie wszystkie zmienne lokalne oznaczyłbym jako literę "x". Ma to sens?

Comments

Nie. Dlaczego?
1. Zmienna globalna "metry". Po pierwsze, globalna, po drugie po polsku.
2. Wszystko powinno być nazwane po angielsku.
3. Zmienne lokalne jako 'x'? Nie. Dlaczego? Bo zmienne muszą coś reprezentować.
4. Te procedury są bezsensu. Z tego co wiem nazywa się to "głośną funkcją". Funkcje raczej nie powinny robić nic więcej, niż to co mają robić. Czyli, jeśli masz funkcję, która konwertuje jednostkę na jednostkę, to powinna ona tylko to zrobić i zwrócić wynik (ew. przyjąć przez referencję/wskaźnik i zmodyfikować, ale to podejście jest rzadko używane).

---

OK, dzięki za komentarz.