Panel użytkownika
Nazwa użytkownika:
Hasło:
Nie masz jeszcze konta?

Pierwszy "program" po pół roku bez programowania, rady(?)

Ostatnio zmodyfikowano 2019-03-06 20:46
Autor Wiadomość
glof2
Temat założony przez niniejszego użytkownika
Pierwszy "program" po pół roku bez programowania, rady(?)
» 2019-03-06 16:39:13
Tak jak w tytule, nie pisałem żadnych programów od ok. pól roku. Teraz staram się wrócić do programowania jako hobby. Napisałem ten "program" i mam pytanie co mogę poprawić, oprócz braku komentarzy.
C/C++
#include <iostream>
#include <conio.h>
using namespace std;
//--------------
void ClearScreen(); //|
void Logowanie(); //{ zapowiedz funkcji
void Menu(); //|
//--------------
//--------------
int main()
{
    Logowanie(); //{Glowny kod
    Menu();
    return 0;
}
//--------------
//--------------
void Logowanie()
{ //kod menu logowania
    int loop = 0;
    string login, haslo;
    cout << "Witaj w banku.\n";
    cout << "Zaloguj sie.\n";
    while( loop == 0 ) {
        cout << "Login: "; cin >> login; if( login != "martyna" ) { cout << "Nie znaleziono loginu w bazie danych.\n"; }
        else { loop = 1; }
    }
    while( loop == 1 ) {
        cout << "Haslo: "; cin >> haslo; if( haslo != "bazyl12" ) { cout << "Nie prawidlowe haslo!\n"; }
        else { loop = 2; }
        cout << "Zalogowano pomyslnie!\n";
    }
}
//--------------
//--------------
void ClearScreen() { getch(); cout << string( 100, '\n' ); } //kod funkcji ClearScreen (Wyczysc ekran)
//--------------
//--------------
void Menu()
{
    bool zalogowano = true; int saldo = 5000, odp; string odp1, odp2;
    ClearScreen(); // czysci ekran
    //Menu
    while( zalogowano = true )
    {
        cout << string( 100, '\n' );
        cout << "Bank KFSJAIFA\n";
        cout << "-----------------\n";
        cout << "1. Przelej\n";
        cout << "2. Saldo\n";
        cout << "3. Wyloguj\n";
        cout << "Co chcesz zrobic?: ";
        cin >> odp;
        //switch w zaleznosci od inta 'odp'
        switch( odp )
        {
        case 1: {
                int loop2 = 0;
                while( loop2 == 0 )
                {
                    cout << "1.Kolega\n"; cout << "2. Cofnij\n";
                    cout << "Do kogo chcesz przelac pieniadze?: "; cin >> odp2;
                    if( odp2 == "1" )
                    {
                        int przelane;
                        cout << "Ile pieniedzy chcesz przelac?: "; cin >> przelane;
                        if( saldo - przelane <= - 1 )
                        {
                            cout << "Nie masz takiej kwoty.\n"; ClearScreen();
                        }
                        else
                        {
                            saldo = saldo - przelane; loop2 = 1;
                        }
                    }
                    else if( odp2 == "2" )
                    { loop2 = 1;
                        break;
                    }
                    else { cout << "nie ma takiej opcji\n"; }
                }
                break;
            }
        case 2: { cout << "Twoje aktualne saldo to: " << saldo << "pln.\n"; getch(); break; }
        case 3: { cout << "Wpisz t, jezeli jestes pewien.\n"; cin >> odp1; if( odp1 == "t" ) { zalogowano = false; }
                break; }
            default: { cout << "Nie ma takiej opcji!"; break; }
        }
    }
   
   
}
P-174123
pekfos
» 2019-03-06 17:10:09
Napisałem ten "program" i mam pytanie co mogę poprawić, oprócz braku komentarzy.
A w którą stronę chcesz poprawiać ten 'brak komentarzy'? Jakieś komentarze masz w kodzie i żaden nic nie wnosi. "ClearScreen() czyści ekran", duh! Komentuje się rzeczy, które wymagają komentarza, bo sam kod za słabo przekazuje intencje.
Bardziej niż komentarzami martw się formatowaniem. Ten kod wygląda beznadziejnie pod tym względem, pomimo poprawek wprowadzonych przez automatyczne formatowanie na forum. Niespójny podział na linie, niespójne wcięcia, to wszystko złe praktyki w pisaniu kodu.

C/C++
while( zalogowano = true )
» Kurs C++ / FAQNajczęściej spotykane bugi pytanie/odpowiedź

C/C++
if( saldo - przelane <= - 1 )
C/C++
if( saldo < przelane )
P-174124
glof2
Temat założony przez niniejszego użytkownika
» 2019-03-06 17:18:19
Wlasnie nie wiem czy te poprawki na forum nie zepsuly mi formatowania. Aktualnie nie moge sprawdzic czy wyglada to tak samo jak gdy pisalem, poniewaz jestem na telefonie.
P-174125
pekfos
» 2019-03-06 17:28:24
Aktualnie nie moge sprawdzic czy wyglada to tak samo jak gdy pisalem
Za to ja mogę i w oryginale jest jeszcze gorzej. Po sformatowaniu przynajmniej są wcięcia..
P-174127
DejaVu
» 2019-03-06 20:46:21
Rada: chcesz programować to rób to codziennie. Po dwóch latach intensywnego kodowania, robienia różnych projektów dopiero będzie można zacząć oceniać realną wiedzę z programowania. Taka to po prostu branża. Duży czasowo próg wejścia, a i nawet po tym czasie będziesz dopiero na początku nauki programowania (choć będziesz już umiał prawdopodobnie napisać samodzielnie proste gry).

Po 10 latach programowania zawodowego (czyli na pełen etat) nadal będziesz zdobywał nową wiedzę z inżynierii oprogramowania itd.

Podsumowując: możesz wziąć 12 lat edukacji szkolnej i wyrzucić do kosza w porównaniu do ~3-4 lat nauki programowania. Edukacja szkolna nie da Ci żadnej bonusowej wartości w jakiejkolwiek rekrutacji, a dużo energii każdy z nas w ją włożył. 4 lata samodzielnej nauki programowania natomiast będzie coś warte już na rynku pracy i da Ci szansę rozwijać się dalej.

Odpowiedz sobie na pytanie: czy ciągnie Ciebie do nauki programowania, czy masz zupełnie inne preferencje życiowe. Pół roku przerwy? Jest OK, jeżeli wracasz do programowania i widzisz, że jednak chcesz iść w tym kierunku.
P-174130
« 1 »
  Strona 1 z 1