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

Zapraszam do dyskusji na temat programiku :)

VPS Starter Arubacloud
–2 głosów
289 wizyt
pytanie zadane 27 stycznia 2017 w C i C++ przez Szczurk3y Nowicjusz (200 p.)

Generalnie dopiero się uczę i chciałem napisać program z wykorzystaniem tego, co pokazywał P. Mirosław Zelent w swoich 5 pierwszych filmach z C++ w celu utrwalenia. Chciałem poprosić o jakieś uwagi, jak można by było skrócić program, ulepszyć etc. Miałem na celu zrobienie (pseudo) formularza z wykorzystaniem mojego aktualnego stanu wiedzy. Pozdrawiam i zapraszam do dyskusji :)

oto on:

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

using namespace std;

string imie, nazwisko, m_zam, ulica, nr_domu;
int wiek, liczba, strzal;
int main()
{
   cout << "Witaj! Wypelnij pola za: " << "\a" <<endl;

       for (int i=3; i>0; i--)
         {
           Sleep (1000);
            cout << i << endl;
         }
     Sleep(1000);
     system ("cls");

    cout << "Ile masz lat "<< endl;
    cin>> wiek;
    cout << "\a";
    system ("cls");
        if (wiek<16)
        {
            cout<< "Przepraszamy! Jestes za mlody. ";
            Sleep (2000);
            exit(EXIT_SUCCESS);
        }
        else if (wiek>=16)
 {
    cout << "Imie: "<<endl;
    cin>> imie;
    cout << "\a";
    system ("cls");

    cout << "Nazwisko: "<< endl;
    cin>> nazwisko;
    cout << "\a";
    system ("cls");

    cout << "Miejsce zamieszkania: "<< endl;
    cin>> m_zam;
    cout << "\a";
    system ("cls");

    cout << "Ulica: "<< endl;
    cin>> ulica;
    cout << "\a";
    system ("cls");

    cout <<"numer domu: ";
    cin>> nr_domu;
    cout << "\a";
    system ("cls");

    cout<< "Formularz zgloszeniowy: "<<endl;
    cout<<endl;

        cout<< "Twoj wiek: "<<wiek<<endl;
          Sleep(1500);

        cout<< "Twoje imie: "<< imie<<endl;
          Sleep(1500);

        cout<< "Twoje nazwisko: "<< nazwisko <<endl;
          Sleep(1500);

        cout<< "Twoje miejsce zamieszkania: "<<m_zam<<endl;
          Sleep(1500);

        cout<< "Twoja ulica: "<<ulica<<endl;
          Sleep(1500);

        cout<< "Twoj numer domu: "<< nr_domu<<endl;
          Sleep(3000);
          system ("cls");

    cout<<"\a"<< "Aby zakonczyc rejestracje udowodnij, ze nie jestes robotem!"<<endl;
       Sleep(3000);
       system("cls");

    cout<<"\a"<< "Odgadnij liczbe, ktora wymyslilem!"<<endl;
    Sleep (2000);
     srand(time(NULL));
     liczba=rand()%100+1;

      while (strzal!=liczba)
        {
           cout<< "zgadnij jaka: "<<endl;
           cin>> strzal;

           if (strzal==liczba)
            cout<< "Gratuluje, udalo ci sie udowodnic, ze nie jestes robotem!"<<endl;

               else if (strzal>liczba)
                cout<< "To za duzo!"<<endl;

               else if (strzal<liczba)
                cout<< "To za malo!"<<endl;
        }
        Sleep(2000);
       system("cls");
        cout<<"Aby wyslac zgloszenie nacisnij enter"<<"\a"<<endl;
 }
    return 0;
}

komentarz 27 stycznia 2017 przez koczurekk Gaduła (3,420 p.)
Wiem, że w kursie Zelenta tego nie było, ale pls, sformatuj kod.
komentarz 27 stycznia 2017 przez Szczurk3y Nowicjusz (200 p.)
Cześć, przeczytaj ze zrozumieniem mój wstęp w poście. Generalnie wiem, że to "coś" co napisałem to gówno :D Ale chciałbym dowiedzieć się nowych rzeczy, m.in właśnie jak mógłbym go skrócić, dlatego tutaj napisałem. Pozdrawiam. :)
2
komentarz 27 stycznia 2017 przez koczurekk Gaduła (3,420 p.)

Ech… przeczytaj to.

2 odpowiedzi

+4 głosów
odpowiedź 27 stycznia 2017 przez Molester Bywalec (2,920 p.)
wybrane 28 stycznia 2017
 
Najlepsza

Używaj nazw zmiennych, funkcji po angielsku, to dobry nawyk i warto go przyswoić. Po 2 , rozumiem że program ma formę edukacyjną bardziej niż praktyczną, ale staraj się nie używać funkcji Sleep() kiedy jest to niepotrzebne, może to sprawa dyskusyjna i moje widzimisię , ale to bardziej wygląda jakby program się zwieszał. Zresztą kiedyś widziałem podobną opinię na tym forum w sprawie tego. W jednej linijce  niepotrzebnie dałeś else if, wystarczyłby po prostu else (przy "To za mało"). Nie stosuj też  zmiennych globalnych, włóż je pod maina (to też dobry nawyk) oraz wyposaż strumień wejścia w kontrolę błędów (np ktoś poda literę zamiast liczby, i żeby ponowił próbę wpisania). Dobrze jest też uczynić swój kod uniwersalnym czyli czyszczenie ekranu zastosować i do windowsa i do innych systemów. Z tym też można się zapewne sprzeczać ale ja bym nie używał przestrzeni nazw std. Tu masz więcej na ten temat :

http://cpp0x.pl/kursy/Kurs-C++/Poziom-1/Przestrzen-nazw-std/10

Równolegle z kursem MZ możesz przerabiać tez ten kurs, jest to zalecane bo z całym szacunkiem, ale niektóre bardzo ważne rzeczy są pomijane przez pana MZ :

http://cpp0x.pl/kursy/Kurs-C++/Poziom-1/24

Nie musisz mówić że To co napisałeś to gówno - każdy kiedyś zaczynał i to normalne . No i to w zasadzie tyle, program jest bardzo mały - nie ma tu co większych rzeczy do sprawdzenia.

+1 głos
odpowiedź 27 stycznia 2017 przez maciek061 Gaduła (4,490 p.)

Do zamieszczania kodu używaj bloków kodu, albo podrzuć link do jakiegoś takiego serwisu. 

Generalnie, moim zdaniem, kod jest spoko, nie ma żadnych nieprawidłowości.

Możesz go sformatować klikając prawym przyciskiem gdzieś na kod i wybierając z menu kontekstowego "Format use AStyle"

Jeszcze jedna sprawa. Skoro zaczynasz przygodę z programowaniem, to możesz zastanowić się co jest twoim celem. Po opanowaniu podstaw C++, który jest bardzo dobry na start, możesz rozpocząć uczyć się technologii webowych, bo mogą okazać się przydatniejsze (zobacz)

Podobne pytania

+30 głosów
8 odpowiedzi 1,451 wizyt
pytanie zadane 7 sierpnia 2016 w Sprawy forum przez ScriptyChris Mędrzec (190,190 p.)
0 głosów
3 odpowiedzi 260 wizyt
pytanie zadane 29 czerwca 2020 w Nasze projekty przez _Pita_ Stary wyjadacz (11,410 p.)
0 głosów
2 odpowiedzi 4,649 wizyt
pytanie zadane 20 października 2020 w Java przez lukasz07it Początkujący (290 p.)

92,455 zapytań

141,263 odpowiedzi

319,099 komentarzy

61,854 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.

Akademia Sekuraka

Akademia Sekuraka 2024 zapewnia dostęp do minimum 15 szkoleń online z bezpieczeństwa IT oraz dostęp także do materiałów z edycji Sekurak Academy z roku 2023!

Przy zakupie możecie skorzystać z kodu: pasja-akademia - użyjcie go w koszyku, a uzyskacie rabat -30% na bilety w wersji "Standard"! Więcej informacji na temat akademii 2024 znajdziecie tutaj. Dziękujemy ekipie Sekuraka za taką fajną zniżkę dla wszystkich Pasjonatów!

Akademia Sekuraka

Niedawno wystartował dodruk tej świetnej, rozchwytywanej książki (około 940 stron). Mamy dla Was kod: pasja (wpiszcie go w koszyku), dzięki któremu otrzymujemy 10% zniżki - dziękujemy zaprzyjaźnionej ekipie Sekuraka za taki bonus dla Pasjonatów! Książka to pierwszy tom z serii o ITsec, który łagodnie wprowadzi w świat bezpieczeństwa IT każdą osobę - warto, polecamy!

...