Pierwszy samodzielny projekt w c++

Question

Witam, jestem uczniem technikum informatycznego (1 klasa) w c++ jestem samoukiem, jest to pierwszy samodzielny projekt który stworzyłem bo nie lubie drukować/pisać planów lekcji, mam takie wrażenie że mogłem zrobić mniejszą ilość ifów, co wy o nim myślicie ?

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

using namespace std;

int dzien, nr_linii ;
string linia;

int main()

{
    cout<<"------PLAN LEKCJI--------"    <<endl;
    cout<<"PONIEDZIALEK------------1"    <<endl;
    cout<<"WTOREK------------------2"    <<endl;
    cout<<"SRODA-------------------3"    <<endl;
    cout<<"CZWARTEK----------------4"    <<endl;
    cout<<"PIATEK------------------5"    <<endl;
    cout<< "podaj dzien"; cin>>dzien;
    system("cls");
    if (dzien==1)
    {
        cout<<"Poniedzialek: "<<endl;
        Sleep(500);
        char tekst[100];
        ifstream plik;
        plik.open("1.txt", ios::in);
        if(plik.good() == true)
            while(!plik.eof())
        {
            getline(plik,linia);
            cout << linia<<"\a"<<endl;
        }
        plik.close();
    }
    if (dzien==2)
    {
        cout<<"Wtorek: "<<endl;
        Sleep(500);
       char tekst[100];
       ifstream plik;
       plik.open("2.txt", ios::in);
       if(plik.good()==true)
        while(!plik.eof())
       {
           getline(plik,linia);
           cout <<linia<<"\a"<<endl;
       }
    }
    if (dzien==3)
    {
        cout<<"Sroda: "<<endl;
        Sleep(500);
       char tekst[100];
       ifstream plik;
       plik.open("3.txt", ios::in);
       if(plik.good()==true)
        while(!plik.eof())
       {
           getline(plik,linia);
           cout <<linia<<"\a"<<endl;
       }
    }
    if (dzien==4)
    {
        cout<<"Czwartek: "<<endl;
        Sleep(500);
        char tekst[100];
       ifstream plik;
       plik.open("4.txt", ios::in);
       if(plik.good()==true)
        while(!plik.eof())
       {
           getline(plik,linia);
           cout <<linia<<"\a"<<endl;
       }
    }
    if (dzien==5)
    {
        cout<<"Piatek: "<<endl;
        Sleep(500);
        char tekst[100];
       ifstream plik;
       plik.open("5.txt", ios::in);
       if(plik.good()==true)
        while(!plik.eof())
       {
           getline(plik,linia);
           cout <<linia<<"\a"<<endl;
       }
    }
    getchar();getchar();
    system("cls");
    return 0;
}

 

Answer 1

We wszystkich "if" powtarzasz ten sam kod a można by zrobić jedną funkcję, która by przyjmowała nazwę dnia i nazwę pliku do wypisania. 

W sumie nazwa pliku mogłaby być tworzona na podstawie zawartości zmiennej "dzien" bo nazwa pliku to prawie dzien + ".txt"

Jeśli "dzien" jest równy 1 to nie ma sensu sprawdzać pozostałych "if" więc można by zrobić "else if(dzien == 2)". Podobnie z pozostałymi dniami. Można by nawet użyć "switch/case"

Nazwy dni można by przechowywać w tablicy np. nazwa_dnia i potem wypisywać "nazwa_dnia[dzien]"

 

Answer 2

Popatrz sobie na ten kod... czy nie widzisz, że coś okresowo się w nim powtarza? Taka nadmiarowość kodu nazywa się redundancją. Właśnie dlatego powinieneś używać pętli oraz funkcji.

Comments

Doczepię się do twojego wpisu @Patrycjerz i dodam, że jestem przeciwnikiem zmiennych globalnych w tym przypadku.

---

@Patrycjerz,  może stworzyć funkcje do wczytania pliku i w  if’ach je wywoływać 

Answer 3

switch(dzien)
{
case 1:
//Poniedziałek
break;
case 2:
//Wtorek
break;

itd.
}

 

Answer 4

fajny projekt tylko mugłbyś dać jeszcze ifa który sprawdzi czy użytkownik podał prawidłowy dzień .

I zamist ifów dać switcha .