C++ SKLEP ONLINE

Question

Mam problem z funkcją LogowanieKlienta

plik klienci.txt jest budowany tak:

Imie //1linijka

Nazwisko //2linjka

ID // linijka

Imie //4lijka

itd

NP.

Piotr

Nowak

1000

Julia

Kowal

1001

 

#include <iostream>
#include <cstdlib>
#include <string>
#include <fstream>
#include <Windows.h>
#include <algorithm>
#include <conio.h>
#include <stdio.h>

using namespace std;

char MainMenu();
void PandC();
void Default();
void Exit();
void LogowanieKlienta();
void MenuKlienta();

int main()
{
	while (1)
	{
		char mainmenu = MainMenu();
		switch (mainmenu)
		{
		case '1':
			MenuKlienta(); break;
		case '2':
			//Menupersonel();
		case '0':
			Exit(); break;
		default:
			Default(); break;
		}
	}

	system("PAUSE");
	return 0;
}
char MainMenu()
{
	char o;
	cout << "Witamy w Lidl online!" << endl;
	cout << "Dostepne opcje:" << endl;
	cout << "1. Dzial dla klienta" << endl;
	cout << "2. Dzial dla pracownika" << endl;
	cout << "0. Wyjdz" << endl;
	o = _getch();
	return o;
}
void PandC()
{
	system("PAUSE");
	system("cls");
}
void Default()
{
	cout << "Nie ma takiej opcji!" << endl;
	PandC();
}
void Exit()
{
	char leave;
	do
	{
		cout << "Czy na pewno chcesz opuscic nasz sklep? (T/N)" << endl;
		leave = _getch();
		if ((leave == 'T') || (leave == 't'))
		{
			system("cls");
			cout << "Dziekuje za skorzystanie z naszych uslug." << endl;
			cout << "Zapraszamy ponownie" << endl;
			system("PAUSE");
			exit(0);
		}
		else if ((leave != 'N') && (leave != 'n'))
		{
			cout << "Nie ma takiej opcji!" << endl;
			PandC();
		}
	} while ((leave != 'N') && (leave != 'n'));
	system("cls");
}
void LogowanieKlienta()
{
	system("cls");
	char o;
	fstream klienci;
	int id;
	string linia;
	string imie[100], nazwisko[100];
	int numer[100];
	int nr_lini = 0;
	while (1)
	{
		cout << "Witamy na zakupach online" << endl;
		cout << "Dostepne opcje:" << endl;
		cout << "1. Zaloguj" << endl;
		cout << "2. Stworz konto" << endl;
		o = _getch();
		switch (o)
		{
		case '1':
			cout << "Prosze podac swoj numer klienta: ";
			cin >> id;
			klienci.open("klienci.txt", ios::in);
			while (!klienci.eof())
			{
				getline(klienci, linia);
				imie[nr_lini] = linia;
				nazwisko[nr_lini] = linia;
				numer[nr_lini] = atoi(linia.c_str());
				system("PAUSE");
				if (id == numer[nr_lini])
				{
					klienci.close();
					cout << "Witamy " << imie[nr_lini] << " " << nazwisko[nr_lini] << ". Zapraszamy na zakupy" << endl;
				}
				nr_lini ++;
			}
			cout << "Takiego numeru nie ma w systemie. Wprowadz numer klienta ponownie lub zaloz nowe konto." << endl;
			PandC();
			klienci.close();
		case '2':
			cout << "2" << endl; break;
		default:
			Default(); break;
		}
	}

}
void MenuKlienta()
{
	LogowanieKlienta();

}

 

Answer 1

1. W wewnętrznej pętli wczytujesz tylko jedną linię i przypisujesz ją do wszystkich zmiennych (imie, nazwisko, numer)

2. Nie potrzebujesz tablic. Nic nie wnoszą, tylko zamieszanie robią

Wprowadziłem kilka poprawek
 

void LogowanieKlienta()
{
    system("cls");
    while (1)
    {
        cout << "Witamy na zakupach online" << endl;
        cout << "Dostepne opcje:" << endl;
        cout << "1. Zaloguj" << endl;
        cout << "2. Stworz konto" << endl;
        char o = _getch();
        switch (o)
        {
        case '1': {
            cout << "Prosze podac swoj numer klienta: ";
            int id;
            cin >> id;
            fstream klienci;
            klienci.open("klienci.txt", ios::in);
            while (!klienci.eof())
            {
                string imie, nazwisko;
                int numer;
                
                getline(klienci, imie);
                getline(klienci, nazwisko);
                cin >> numer;
                if (cin.bad()) // sprawdzamy czy nie było błędu odczytu
                    break;
                cin.ignore(); // usuwamy koniec lini ze strumienia
                
                system("PAUSE");
                if (id == numer)
                {
                    klienci.close();
                    cout << "Witamy " << imie << " " << nazwisko << ". Zapraszamy na zakupy" << endl;
                }
            }
            cout << "Takiego numeru nie ma w systemie. Wprowadz numer klienta ponownie lub zaloz nowe konto." << endl;
            PandC();
            klienci.close();
        } break;
        case '2':
            cout << "2" << endl; 
            break;
        default:
            Default(); 
            break; // ten break jest niepotrzebny
        }
    }
}

Comments

Co Wy z tym eofem macie?

string imie, nazwisko;
int id;

while (getline(klienci >> ws, imie) && 
        getline(klienci, nazwisko) && 
        klienci >> id) {
    /* Tutaj masz pewność, że wszystkie pola zostały odczytane poprawnie */
}