C++ własne funkcje. Zwracanie zmiennej.

Question

Witam, ostatnio zacząłem moją przygodę z programowaniem. Jestem na etapie wprowadzania do programu własnych funkcji, mam jednak problem - mój program do wypisywana planu lekcji (cin >> dzien. a potem switch ktory wypisuje odpowiedni tekst.) Chciałem zrobić to z własną funkcją, żeby przećwiczyć funkcje, bo program działa. Ale Wyskakuje mi błąd - nie ważne co wpiszę do zmiennej "dzien", wykonuje się default w switchu:

 

#include <iostream>
#include <conio.h>
using namespace std;




int day(int dzien)
{
    cout << "Na jaki dzien tygodnia chcesz zobaczyc plan lekcji?" << endl;
    cout << "1 - poniedzialek" << endl;
    cout << "2 - wtorek" << endl;
    cout << "3 - sroda" << endl;
    cout << "4 - czwartek" << endl;
    cout << "5 - piatek" << endl;
    cout << "6 - sobota" << endl;
    cout << endl << endl;

    cin >> dzien;

    return dzien;

}



int main()
{

    int dzien;
    day(dzien);

    switch (dzien)
{
    case 1:

      cout << "Plan lekcji na poniedzialek:" << endl;
      cout << "----------------------------" << endl;
      cout << "1 - jezyk polski " << endl;
      cout << "2 - chemia" << endl;
      cout << "3 - biologia" << endl;
      cout << "4 - matematyka" << endl;
      cout << "5 - WF" << endl;
      cout << "6 - historia" << endl;
      cout << "7 - fizyka" << endl;
      cout << "8 - wdzwr" << endl;
      break;
case 2:
      cout << "Plan lekcji na wtorek:" << endl;
      cout << "----------------------------" << endl;
      cout << "1 - fizyka" << endl;
      cout << "2 - matematyka" << endl;
      cout << "3 - plastyka" << endl;
      cout << "4 - WF" << endl;
      cout << "5 - jezyk angielski" << endl;
      cout << "6 - jezyk niemiecki" << endl;
      cout << "7 - jezyk polski" << endl;
break;

case 3:
      cout << "Plan lekcji na srode:" << endl;
      cout << "----------------------------" << endl;
      cout << "1 - religia " << endl;
      cout << "2 - jezyk angielski" << endl;
      cout << "3 - WF" << endl;
      cout << "4 - chemia" << endl;
      cout << "5 - jezyk polski" << endl;
      cout << "6 - jezyk polski" << endl;
      cout << "7 - historia" << endl;
break;

case 4:
      cout << "Plan lekcji na czwartek:" << endl;
      cout << "----------------------------" << endl;
      cout << "1 - jezyk niemiecki " << endl;
      cout << "2 - religia" << endl;
      cout << "3 - matematyka" << endl;
      cout << "4 - informatyka" << endl;
      cout << "5 - godz. wych." << endl;
      cout << "6 - geografia" << endl;
      cout << "7 - biologia" << endl;
break;


case 5:
      cout << "Plan lekcji na piatek:" << endl;
      cout << "----------------------------" << endl;
      cout << "1 - jezyk polski " << endl;
      cout << "2 - muzyka" << endl;
      cout << "3 - matematyka" << endl;
      cout << "4 - geografia" << endl;
      cout << "5 - jezyk angielski" << endl;
      cout << "6 - WF" << endl;
break;


case 6:
      cout << "nie, nie mamy lekcji w sobote :)" << endl;
break;


default:
      cout << "nie ma takiego dnia tygodnia w menu" << endl << endl;
break;

}


getch();
    return 0;
}

 

Answer 1

W tym kodzie nie przypisujesz do zmiennej dzien tego co zwracasz z funkcji day. Musisz zrobić to tak:

dzien = day(dzien);
 

Ale tak poza tym przekazywanie zmiennej do funkcji day jest niepotrzebne. Lepiej niech to będzie funkcja bez parametrów, a w jej ciele zadeklaruj zmienną lokalną dzien. Wówczas w funkcji main możesz zrobić tak:

int dzien = day();