Zadanie z wypisywaniem tekstu z funkcji(czystość kodu)

Question

Witam, właśnie zaczynam naukę programowania (tak +/- drugi raz). Kupiłem książkę Stephena Praty "Język c++ szkoła programowania wydanie VI". I w tej książce jest zadanie w którym mam stworzyć 3 funkcje. Pierwsza to main z której mam wywołać dwie następne tak aby wypisało z jednej dwa razy entliczek pentliczek a z drugiej czerwony stoliczek też dwa razy. Pokombinowałem, napisałem (3 razy poprawiłem) i działa. Ale właśnie wydaje mi się że powinno to wyglądać inaczej. Mianowicie teraz wygląda tak:

#include <iostream>

void entliczek(int);
void czerw(int);

int main()
{
    entliczek(1);
    entliczek(1);
    czerw(1);
    czerw(1);
    return 0;
}

void entliczek(int x)
{
    using namespace std;
    cout << "entliczek pentliczek" <<endl;
}
void czerw(int z)
{
    using namespace std;
    cout <<"czerowny stoliczek"<<endl;
}

I nie rozumiem dlaczego po wywołaniu funkcji np. entliczek w nawiasie musiałem wstawiać jakąś liczbę(próbowałem po prostu void i nie szło). Jeśli to przez to że w prototypie wstawiłem int w nawias, to czym to powinienem zastąpić?

W skrócie, czy ten kod jest poprawny? 

Mam nadzieje że pytanie jest dobrze sformułowane( mój pierwszy wieczór na forum ^^ ), jeśli nie to proszę o sugestie. Co do pytania to prosił bym o odpowiedzi z których dam radę wyciągnąć wnioski i się nauczyć.

Comments

Dzięki za pouczające odpowiedzi ;P poprawiłem i od razu wygląda czyściej, bardziej profesjonalnie. Teraz wiem że trzeba próbować najpierw wszystkich możliwości a dopiero potem zawracać głowy na forum. Dzięki całej trójce i temat do zamknięcia.

Answer 1

Definicja i deklaracja funkcji musi być zgodna. Jeżeli nie chcesz żeby funkcja pobierała jakieś argumenty to po prostu je usuń z definicji (i deklaracji) funkcji.
 

int moja_funkcja();

Możesz też przerobić swoje funkcje tak, aby wyświetlały napis tyle razy ile podasz w argumencie funkcji (przy użyciu pętli)
 

int main()
{
    entliczek(2);
    czerw(2);
    return 0;
}

 

Comments

Po usuwałem i stwierdzam że działa :) Dzięki, w ogóle nie pomyślałem że te nawiasy można by zostawić puste. A pomysł z pętlą faktycznie dobry(może nie teraz ale przy jakimś większym projekcie na pewno się przyda)

Answer 2

Zeby wywolac funkcje entliczek musisz podac liczbe (inta) w nawiasach, poniewaz tak zadeklarowales.

void entliczek(int);

w nawiasach podaje sie argumenty i podczas wywolania funkcja ich oczekuje, w tym przypadku jest to typ int.

Zebys mogl wywolac funkcje nie podajac argumentow to musisz... hmm, jak myslisz ? Funkcja nie zawsze musi cos przyjmowac.

Kod jest poprawny choc, mozesz zastosowac tez petle do wywolania funkcji, no i tak jak wsponialem wczesniej, po co wysylac funkcji jakos wartosc skoro z niej nie korzystasz.

 

 

Answer 3

Twój kod jest poprawny, ale liczba która przekazujesz nie ma znaczenia.

Pisząc funkcje uwzględniasz 3 rzeczy to co zwraca jej nazwę oraz jej argumenty.

Jeśli nie chcesz by funkcja przyjmowała żadnej wartości to w nawiasie nie piszesz nic

np.

void FunBezArg();

natomiast pomiędzy nawaisami możesz podać dowolną liczbę argumentów

np.

double FunZArg(int, double, char);

oczywiście definując póżniej funkcje musisz nadać im nazwy.