Czysty kod funkcji C++

Question

Cześć. Kiedyś ktoś gdy uczył mnie programować w C++ powiedział, że złą praktyką jest używanie komendy cout i cin w funkcji int. Czy to prawda?

Answer 1

Tak w skrócie: istnieje pojęcie funkcji "czystej". To taka funkcja która operuje tylko na swoich parametrach i na zmiennych lokalnych. Tak napisana funkcja zwykle nie ma problemów z używaniem jej w rekurencji i współbieżności. cout i cin to są zmienne globalne (nie są to komendy). Nie ma to co wiele wspólnego ze zwracanym typem, jednak sam fakt zwracania czegoś (oprócz void) sugeruje, że chodzi o funkcje których rezultatem to zwracany wynik, a nie tzw skutki uboczne. Oczywiście wynik można też dostarczyć "na zewnątrz" innymi sposobami. Wiele instrukcji w C ma swoje skutki uboczne, np. operator przypisanie zwaraca wartość prawego argumentu i tyle, jego skutkiem ubocznym jest przypisanie wartości do zmiennej. A operator + to taki przykład funkcji czystej.

EDIT: drobne literówki

Answer 2

Dobrą praktyką jest tworzenie funkcji:

• krótkich
• z minimalną liczbą parametrów
• wykonujących jedno zadanie

Przykład funkcji, która powinna zostać podzielona z powodu zbyt wielu wykonywanych zadań:

int suma_najmniejszych_pieciu_elementow_tablicy_wczytana_z_wejscia_z_ekstremalnie_dluga_nazwa() {
    std::vector<int> ints(10);
    for (int& i : ints) {
        std::cin >> i;
    }

    std::partial_sort(ints.begin(), ints.begin() + 5, ints.end());
    std::cout << "Najmniejsze: ";
    for (auto it = ints.begin(); it != ints.begin() + 5; ++it) {
        std::cout << *it << " ";
    }

    int sum{};
    for (int i = 0; i < 5; ++i) {
        sum += ints[i];
    }

    return sum;
}

• wczytuje dane
• wyszukuje poprzez sortowanie 5 najmniejszych liczb 
• wypisuje 5 najmniejszych liczb 
• zwraca sumę 5 najmniejszych liczb 

Przykład poprawnie napisanej funkcji z std::cout, std::cin i zwracającej inta:

#include <iostream>     
#include <string>       

int readint(std::string msg = "") {
    int num;
    std::cout << msg;
    std::cin >> num;
    return num;
}

int main() {
    int myInt = readint("Podaj liczbę: ");
}

Answer 3

Nigdy nie słyszałem o używaniu cin i cout w funkcji main. Może chodziło o używanie:

using namespace std; ?

Wtedy może dojść do kolizji przy większych plikach. Szczególnie kiedy zrobimy to w global scope (przed funkcjami).

https://stackoverflow.com/questions/1452721/why-is-using-namespace-std-considered-bad-practice

Comments

Przerabiałem już kilka kursów i na wszystkich zalecają tą komendę, żeby nie pisać wszędzie std:: . Chętnie się dowiem o co tu chodzi.

---

Takie "otwieranie" przestrzeni nazw samo w sobie nie jest czymś złym, tylko trzeba mieć świadomość tego, co się robi.

Jak otworzysz przestrzeń w pliku źródłowym (.cpp), to nic wielkiego się nie stanie, bo to otwarcie ograniczone jest tylko do tego pliku. W przypadku nagłówków sprawa wygląda inaczej, bo nagłówki załączane są/mogą być w wielu miejscach, gdzie nie zawsze chcemy, by jakaś przestrzeń nazw była wymieszana z inną, to może powodować konflikty.

Dodam, że using namespace można używać lokalnie wewnątrz funkcji.