• Najnowsze pytania
  • Bez odpowiedzi
  • Zadaj pytanie
  • Kategorie
  • Tagi
  • Zdobyte punkty
  • Ekipa ninja
  • IRC
  • FAQ
  • Regulamin
  • Książki warte uwagi

Animacja programu rejestrującego.

VPS Starter Arubacloud
0 głosów
172 wizyt
pytanie zadane 5 października 2015 w C i C++ przez Barus Stary wyjadacz (14,120 p.)

Witam ! 

Napisałem kilka minut temu program, który "rejestruje" nowego użytkownika. 
Czy kod jest poprawnie napisany i zrefaktoryzowany ? 

Proszę o odpowiedź :)

Pozdrawiam ! 

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

using namespace std;

string name, name2, log, pasw, pasw2, email;
int code, usercode;

int main()
{
    cout<<"Zarejestruj sie!";
    getchar();
    system("cls");

    cout<<endl<<"Imie : ";
    getline(cin,name);
    system("cls");

    cout<<endl<<"Nazwisko : ";
    getline(cin,name2);
    system("cls");

    cout<<endl<<"Nazwa Uzytkownika : ";
    getline(cin,log);
    system("cls");

    do
    {
    cout<<endl<<"Haslo : ";
    getline(cin,pasw);

    cout<<endl<<"Powtorz haslo : ";
    getline(cin,pasw2);


    if(pasw==pasw2)
        {
        continue;
        }
    else
        {
        system("cls");
        cout<<"Podane hasla sie nie zgadzaja...";
        getchar();getchar();
        }

    system("cls");
    }while(pasw!=pasw2);

    system("cls");
    cout<<endl<<"E-mail : ";
    getline(cin,email);
    system("cls");

    do
    {
    srand(time(NULL));
    code=rand()%900;
    cout<<endl<<"Przepisz kod weryfikacyjny : "<<code<<endl;
    cin>>usercode;

    if(code==usercode)
        {
        continue;
        }
    else
        {
        system("cls");
        cout<<"Podany kod jest nieprawidlowy...";
        getchar();getchar();
        }
        system("cls");
    }while(code!=usercode);

    system("cls");
    cout<<"KONTO ZOSTALO POMYSLNIE ZALOZONE !"<<endl<<endl;
    cout<<"PODSUMOWANIE OPERACJI :"<<endl;
    cout<<"Imie : "<<name<<endl;
    cout<<"Nazwisko : "<<name2<<endl;
    cout<<"Login : "<<log<<endl;
    cout<<"Haslo : "<<pasw<<endl;
    cout<<"E-mail : "<<email<<endl<<endl;

    cout<<"Dziekujemy za rejestracje !";

    getchar();getchar();
    return 0;
}

 

2 odpowiedzi

+1 głos
odpowiedź 5 października 2015 przez pewolt Mądrala (5,790 p.)
Na moje oko jeste okej poza jedna rzecza:

string name, name2, log, pasw, pasw2, email;

int code, usercode;

jak bys to zrobil na tablicach to by ladniej wygladalo i bylo bardziej czytelne
komentarz 5 października 2015 przez Barus Stary wyjadacz (14,120 p.)
Dzięki za odpowiedź ! Ja również uważam, że z tablicami będzie lepiej :)

Pozdrawiam !
+1 głos
odpowiedź 5 października 2015 przez criss Mędrzec (172,590 p.)

Nie bardzo jest sens, żeby pasw2 i code były globalne. Wystarczy jak będą zmiennymi lokalnymi w pętli (deklarowane w pętli).
Jeszcze continue możnaby zamienić na break. Widocznie niczego to nie zmieni, ale po prstu jest bez sensu. Różnica jest taka, że continue kończy aktualną iteracje i przechodzi do kolejnego obrotu pętli, znowu sprawdzany jest warunek i dopiero wtedy wychodzimy z pętli. Break po prostu zakończy cała pętle.
Mógłbyś tez zadbać o wcięcia w kodzie, bo ciężko się to czyta (mówie o pętlach).

komentarz 5 października 2015 przez Barus Stary wyjadacz (14,120 p.)
Dziękuję za odpowiedź i porady !  Kod zostanie zmieniony według Twoich podpowiedzi :)

Pozdrawiam !

Podobne pytania

0 głosów
1 odpowiedź 102 wizyt
pytanie zadane 11 sierpnia 2020 w C i C++ przez PirchHD Obywatel (1,730 p.)
0 głosów
1 odpowiedź 356 wizyt
pytanie zadane 22 stycznia 2018 w C i C++ przez Damskii Nowicjusz (120 p.)
0 głosów
2 odpowiedzi 2,058 wizyt
pytanie zadane 20 lutego 2017 w C i C++ przez WireNess Stary wyjadacz (11,240 p.)

92,845 zapytań

141,786 odpowiedzi

320,861 komentarzy

62,178 pasjonatów

Motyw:

Akcja Pajacyk

Pajacyk od wielu lat dożywia dzieci. Pomóż klikając w zielony brzuszek na stronie. Dziękujemy! ♡

Oto polecana książka warta uwagi.
Pełną listę książek znajdziesz tutaj.

Wprowadzenie do ITsec, tom 2

Można już zamawiać tom 2 książki "Wprowadzenie do bezpieczeństwa IT" - będzie to około 650 stron wiedzy o ITsec (17 rozdziałów, 14 autorów, kolorowy druk).

Planowana premiera: 30.09.2024, zaś planowana wysyłka nastąpi w drugim tygodniu października 2024.

Warto preorderować, tym bardziej, iż mamy dla Was kod: pasja (użyjcie go w koszyku), dzięki któremu uzyskamy dodatkowe 15% zniżki! Dziękujemy zaprzyjaźnionej ekipie Sekuraka za kod dla naszej Społeczności!

...