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

Kod C++ do oceny.

VPS Starter Arubacloud
0 głosów
7,141 wizyt
pytanie zadane 24 września 2017 w C i C++ przez s3r31n Początkujący (340 p.)
zmienione kategorie 25 września 2017 przez s3r31n

Witam,

Bardzo proszę o sugestie i ewentualne porady. Zaznaczam, że to mój pierwszy program i jest pisany jako ćwiczenie. 
Wiem, że wymaga dokończenia. Chodzi mi o to co jest już na ten moment.  

#include <iostream>
#include <stdio.h>
#include <stdlib.h>
#include <fstream>
#include <string>

using namespace std;

int a,b,c,d;
int wynik;
char tn;

float zestaw(int suma);

int main()
{
    for(;;)
    {
        cout<<"Witaj w kreatorze cen zestawow firmy LEGO!"<<endl<<endl;
        cout<<"----------------->>MENU<<-----------------"<<endl;
        cout<<"1. Stworz wlasny zestaw."<<endl;
        cout<<"2. Wczytaj zapisany projekt."<<endl;
        cout<<"3. Ponoc."<<endl;
        cout<<"4. Wyjscie."<<endl;
        cout<<"=========================================="<<endl;
        cout<<"Wybierz opcję: ";
        cin>>a;

        switch(a)
        {
        case 1:
        {
            string napis;
            cout<<"Podaj nazwe zestawu: ";
            cin.ignore();
            getline (cin,napis);
            cout<<"Ilu bohaterow pierwszoplanowych na byc w zestawie?: ";
            cin>>b;
            cout<<"Ilu bohaterow drugoplanowych ma byc w zestawie?: ";
            cin>>c;
            cout<<"Ile bedzie specjalnych klockow w zestawie?: ";
            cin>>d;
            cout<<"Wynik = "<<zestaw(wynik)<<endl;
            cout<<"Czy zapisac wynik? [t/n]: ";
            cin>>tn;
            if (tn=='t')
            {
                fstream plik;
                plik.open("Zestawy.txt",ios::out | ios::app);
                plik<<"Zestaw: "<<napis<<":"<<endl;
                plik<<"Dodano "<<b<<" pierwszoplanowych postaci."<<endl;
                plik<<"Dodano "<<c<<" drugoplanowych postaci."<<endl;
                plik<<"Dodano "<<d<<" specjalnych klockow."<<endl;
                plik<<"Tyle bedzie kosztowal caly zestaw."<<zestaw(wynik)<<" PLN"<<endl;
                plik.close();
                cout<<"Zapis wykonany pomyślnie!\n Wcisnij ENTER";
            }
            else
                system ("clear");

        }
        break;
        case 2:
            cout<<"b";
            break;
        case 3:
            cout<<"c";
            break;
        case 4:
        {
            cout<<"Do zobaczenia!";
            exit(0);
        }
        break;
        }

        getchar();
        getchar();
        system("clear");

    }

    return 0;
}

float zestaw(int suma)
{
    suma=((b*20)+(c*10)+(d*1));

    return suma;
}

 

komentarz 24 września 2017 przez Haker2004 Użytkownik (590 p.)
Zamiast system("clear"); użyj system("cls") mi wyskakiwało gdy wstawiłem clear że to nie jest polecenia a używam code block 16.01. i dodaj conio.h  i cstdlib.
5
komentarz 24 września 2017 przez MetGang Nałogowiec (34,360 p.)

Nie dawaj takich rad jeśli nie znasz powodu problemu. Widać, że kod pisany był na Linuxie, który zamiast cls posiada clear i nie udostępnia conio.h (która powinna być omijana przy poważniejszych projektach).

komentarz 24 września 2017 przez criss Mędrzec (172,590 p.)
edycja 28 września 2017 przez criss
@OP W zasadzie mogłabyś zmienić kategorie na Programowanie->C++. "Nasze projekty" jest przeznaczona do pochwalenia się jakimś swoim gotowym projektem.
komentarz 25 września 2017 przez s3r31n Początkujący (340 p.)
Dzięki. Dokładnie tak jak piszesz. Koduję na Linuksie.

3 odpowiedzi

+5 głosów
odpowiedź 24 września 2017 przez criss Mędrzec (172,590 p.)
wybrane 25 września 2017 przez s3r31n
 
Najlepsza
  1. Unikaj zmiennych globalnych i sytuacji gdy wiele funkcji może wpływać na stan jednej zmiennej. Kod staje się dużo trudniejszy w kontroli. Co jeśli okazałoby się, że któraś z twoich zmiennych globalnych przyjmuje nieprzewidziane wartości? Nie jesteś w stanie powiedzieć gdzie szukać przyczyny, bo absolutnie wszystko może ją modyfikować.
  2. Nie używaj std::exit. Przeczytaj odpowiedź z linku! Tym bardziej tutaj (w mainie) nie ma to sensu, bo mogłaś po prostu napisać return 0;
  3. Nie mieszaj C i C++. stdio.h i stdlib.h to nagłówki z C. Odpowiedniki dla C++ to cstdio i cstdlib - na początku dodajesz c i usuwasz '.h' na końcu.
  4. Zdaje się, że nie wiesz jak działają funkcje. Skoro już wszystkie potrzebne do obliczenia zestawu zmienne są globalne, to funkcja zestaw nie ma potrzeby przyjmować jakichkolwiek argumentów. Dobrze napisana funkcja powinna jednak przyjmować argumenty b, c, d (przydałyby się jednak więcej mówiące nazwy) zamiast korzystać ze zmiennych globalnych. Nie ma w tym nic skomplikowanego - dajesz funkcji to czego potrzebuje do obliczeń, a ona zwraca wynik.
  5. Wspomniane nazwy zmiennych. Powinny więcej mówić o ich przeznaczeniu. Pojedyncze litery alfabetu, 'wynik' czy 'tn' znaczą wszystko i nic.
komentarz 25 września 2017 przez s3r31n Początkujący (340 p.)
Wielkie dzięki!

ad. 1. Słuszna uwaga.
ad. 2. Nie wiedziałam...
ad. 3. Też nie wiedziałam. Ale będę się od tej pory stosowała.
ad. 4. Czyli powinnam podać zmienne, które funkcja przyjmie, bez użycia zmiennych globalnych. Tylko bezpośrednio do funkcji, która zwróci mi wynik do funkcji main, tak? Ale sama funkcja powinna posiadać swoje zmienne? (nazwy a, b, c, d są robocze, krócej mi się je wpisywało jak kombinowałam. Przy większym projekcie od razu nadam właściwe).
ad. 5. == ad. 4
komentarz 25 września 2017 przez criss Mędrzec (172,590 p.)

Czyli powinnam podać zmienne, które funkcja przyjmie, bez użycia zmiennych globalnych.

Tak, dokładnie.

 Ale sama funkcja powinna posiadać swoje zmienne?

Chodzi ci o zmienne utworzone wewnątrz ciała funkcji? To już wewnętrzny świat funkcji bez znaczenia dla pozostałych (a przynajmniej powinien być bez znaczenia, co zapewnia brak zmiennych globalnych :P).

 nazwy a, b, c, d są robocze, krócej mi się je wpisywało jak kombinowałam. 

Często ludzie tak mówią, ale nie do końca ma to sens. Opisowe nazwy nie mają tylko ładnie wyglądać, ale pomagają ci się właśnie odnaleźć w kodzie. Więc najbardziej przydatne będą dla ciebie właśnie na etapie kombinowania. 

PS: zmieniłem końcówki z -eś na -aś w mojej odpowiedzi. Twój nick nie pomaga w określeniu płci :P

komentarz 25 września 2017 przez s3r31n Początkujący (340 p.)
Bardzo dziękuję za pomoc.

Nick może nie ale jego etymologia już tak. To skrót od Serial Experiment Lane. No i awatar co nieco też mówi ;p Ale nic się nie stało :)
+1 głos
odpowiedź 24 września 2017 przez 10kw10 Pasjonat (22,880 p.)

1. zmienne globalne

2. uzywanie przestrzeni nazw std

3. nazwa zmiennych, ktora nic nie mowi

int a,b,c,d;

4. Mozesz od razu przeslac przez referencje

float zestaw(int suma)
{
    suma=((b*20)+(c*10)+(d*1));
 
    return suma;
}

 

komentarz 25 września 2017 przez s3r31n Początkujący (340 p.)
ad 2. O co chodzi? Możesz mi to wyjaśnić?

ad. 4 Teraz rozumiem, dziękuję!
0 głosów
odpowiedź 24 września 2017 przez QizmoPL Stary wyjadacz (11,440 p.)
1. Poslkie nazwy

2. Nazwy ktore nic nie znacza

3. using namespce

4. stdio.h ->cstdio, stdlib.h ->cstdlib

5. system ("clear"); exit(0);

6. brak defualt w case

7. Brak kontroli bledow

8. nie for, a while

9. Globalne zmienne
komentarz 25 września 2017 przez s3r31n Początkujący (340 p.)
ad 7. Możesz wyjaśnić?

ad 8. A dlaczego? Wydaje mi się, że do opcji wyboru tak - nie pętla for jest lepsza. Wyjaśnij mi proszę dlaczego uważasz inaczej.

Podobne pytania

+1 głos
2 odpowiedzi 986 wizyt
pytanie zadane 8 lutego 2016 w C i C++ przez garyore Początkujący (370 p.)
+2 głosów
1 odpowiedź 421 wizyt
pytanie zadane 9 października 2021 w C i C++ przez <młody_programista> Początkujący (450 p.)
0 głosów
1 odpowiedź 375 wizyt
pytanie zadane 30 lipca 2019 w C i C++ przez martin1724 Użytkownik (690 p.)

92,454 zapytań

141,262 odpowiedzi

319,089 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!

...