Problem z panelem...

pytanie zadane 11 czerwca 2015 w C i C++ przez SicoL332 Nowicjusz (120 p.)
edycja 11 czerwca 2015 przez SicoL332

Witajcie, ucze sie c++ na kursie miroslawa zelenta i jestem przy 7lekcji... czyli zapis i odczyt, ponizej jest zarys tzn. poczatek skryptu ktory zaczalem dzis pisac i mam problem taki ze gdy odpale kompilator i sie zarejestruje a nast. przejde do logowania to dziala a gdy wylacze i wlacze ponownie to chodz haslo i login jest poprawnie wpisany tak jak w txt to i tak nie chce zalogowac.

konto.open("nickss.txt", ios::in); - co zrobic aby zapisac to jako string czyli nick".txt", zapisywalo by kazde konto w innym folderze.

Kod:

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

using namespace std;
fstream konto;
string imie, nazwisko, nick, haslo, nicks, haslos, linia;
int wiek, nrlini, ser;
char wybor;

int main()
{
    for (;;)
    {

    cout << "  - Menu Gry -" << endl;
    cout <<endl<< "   1.Logowanie"<< endl;
    cout << "  2.Rejestracja"<< endl;
     cout << " 3.Wyjscie z gry"<< endl;
     cout << "   4.Menu Gry"<<endl;
    cout << endl<< "Komenda: ";

    wybor=getch();

 switch(wybor)
 {
 case'4':
    system("cls");
    cout <<endl<< "  - Menu Gry -" << endl;
    cout <<endl<< "   1.Logowanie"<< endl;
    cout << "  2.Rejestracja"<< endl;
     cout << " 3.Wyjscie z gry"<< endl;
     cout << "   4.Menu Gry"<<endl;
    cout << endl<< "Komenda: ";
    break;

     case '1':
        cout <<endl<<"Loading...";
        Sleep(3000);
        cout <<endl<<"Przechodenie do logowania..."<<endl;
        Sleep(1000);
        system("cls");
        cout <<" - Logowanie - ";

        konto.open("nickss.txt", ios::in);

        if(konto.good()==false)
        {
            cout<<"Plik nie istnieje!"<<endl;
            exit(0);
        }
        while(getline(konto, linia))
        {

            switch(nrlini)
{

            case 1: imie = linia; break;
            case 2: nazwisko = linia; break;
            case 3: nick = linia; break;
            case 4: haslo = linia; break;

        nrlini++;
}
        }
konto.close();
            cout<<endl<<"Nick: ";
        cin >> nicks;
        cout<<endl<<"Haslo: ";
        cin >> haslos;

if (nicks==nick&&haslos==haslo)
        {
            cout << "Zalogowano do panelu uzytkownika!"<<endl;
            cout<<endl<<"Imie: "<<nick<<endl;

            cout<<endl<<"Komenda: ";
        }
        else
        {
            cout << "Nick lub haslo sie nie zgadza!"<<endl;
        }

 

        break;

        case '2':
           cout <<endl<<"Loading...";
        Sleep(3000);
        cout <<endl<<"Przechodzenie do rejestracji...";
        Sleep(1000);
        system("cls");
        cout <<" - Rejestracja - "<<endl;
        cout <<endl<<"Imie: "; cin>>imie;
        cout<<endl<<"Nazwisko: "; cin>>nazwisko;
        cout<<endl<<"Nick: "; cin>>nick;
        cout<<endl<<"Haslo: ";cin >> haslo;

        konto.open("nickss.txt", ios::out | ios::app);

        konto<<imie<<endl;
        konto<<nazwisko<<endl;
        konto<<nick<<endl;
        konto<<haslo<<endl;

        konto.close();
        Sleep(1000);
        cout <<"Rejestracja przebiegla pomyslnie!"<<endl;
        Sleep(1000);
        cout<< "Wpisz 4 aby przejsc do menu...";
        break;

        case '3':
          cout <<endl<<"Loading...";
        Sleep(3000);
        cout <<endl<<nick<<" odlaczyl sie od serwera.";
        exit(0);
       break;
        default:
            cout <<endl<<"Nie ma takiej komendy.";
            break;

 

 }

    getchar();getchar();
    system("cls");
    }
    return 0;

}

2 odpowiedzi

odpowiedź 11 czerwca 2015 przez Jejkej Obywatel (1,030 p.)

polecam używanie tego do wstawiania kodu ;) Łatwiej się dogadać z odpowiadającym chociażby ze względu na numerowanie linii.

komentarz 11 czerwca 2015 przez SicoL332 Nowicjusz (120 p.)

Poprawione! Mozesz teraz zobaczyc.

odpowiedź 11 czerwca 2015 przez Radfler VIP (101,030 p.)
edycja 11 czerwca 2015 przez Radfler

Według mnie za wcześnie zabrałeś się za taką aplikację. Poczekaj do odcinka 9 (chyba!), kiedy to MZ omawia funkcje. Twój kod stanie się uporządkowany i łatwiejszy do zrozumienia :)

PS. Daj kolorowanie i zrób lepsze wcięcia, bo kod jest totalnie nieczytelny :p

EDIT:

Coś znalazłem:

Zmienna nrlini (linia 10) jest niezainicjonowana, a ty posługujesz się nią w switchu (linia 57). Dodaj przed tym switchem nrlini = 1;, bo póki co posługujesz się bezużyteczną wartością :)

komentarz 11 czerwca 2015 przez SicoL332 Nowicjusz (120 p.)

Poprawilem to juz... powinno byc czytelne

komentarz 11 czerwca 2015 przez Radfler VIP (101,030 p.)

Jak coś znalazłem błąd. Masz w edicie :)

komentarz 11 czerwca 2015 przez SicoL332 Nowicjusz (120 p.)

ah tak bo ja przenioslem to wyzej do intow tylko zapomnialem dopisac wartosci 1 ale teraz znowu cos przed wklejeniem kodu zjechalem bo blad przy wybor=getch();