Witam, proszę o opisanie/podsumowanie mojej pracy :D

Question

#include <iostream>

using namespace std;

int rzeczy, pudelka,x;

main ()
{
    cout << "Program wyliczajacy ilosc rzeczy przypadajacych na pudelko\n";
    cout << "podaj ilosc rzeczy:";
        cin >> rzeczy;
    cout << "podaj ilosc pudelek:";
        cin >> pudelka;

    x = rzeczy / pudelka;

    cout << "\n\nNa jedno pudelko przypada " << x << " rzeczy.";
    return 0;
}

W C++ jestem na poziomie "Pierwszego odcinka pana Mirosława Zelenta (https://www.youtube.com/watch?v=Kc98tH59A_U)

proszę o ocenienie, czy nie wiem co, mojego programu, w sumie to chcę się pochwalić co stworzyłem :D
Program napisany od deski do deski ze ściągami na kartce co robi dana funkcja:

Chcę wiedzieć co trzeba dopisać, żeby plik w formacie .exe się sam nie zamykał po zrobieniu przeliczeń :)

Answer 1

Aby program się nie zamykał po obliczeniach masz kilka rozwiązań:

• Przed return napisz cin.get();
• Dołącz #include <"windows.h"> oraz przed return 0; system("PAUSE");
• Dołącz  #include <"conio.h"> oraz przed return 0; daj getch();

Answer 2

Zmienną x zrób float. I kod umiejszczaj korzystając z przycisku do tego przeznaczonego.

Comments

Właśnie celowo użyłem obcinania liczb po przecinku, bo wtedy by wyszło np że na jedno pudełko przypada 2,5 rzeczy a rzeczy się raczej nie dzieli
W sumie mogę jeszcze dopisać (zostanie y rzeczy bez przydzielonego do niego pudełka :)

---

Polecam nie używania using namespace std; wiem, że jest więcej pisania ale dzięki temu szybciej się nauczysz co zawiera biblioteka standardowa. 

---

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

using namespace std;

int rzeczy, pudelka,x,y;

main ()
{
    cout << "Program wyliczajacy ilosc rzeczy przypadajacych na pudelko\n";
    cout << "podaj ilosc rzeczy:";
        cin >> rzeczy;
    cout << "podaj ilosc pudelek:";
        cin >> pudelka;

    if (pudelka > 0)
    {

        x = rzeczy / pudelka;

    cout << "\n\nNa jedno pudelko przypada(ja) " << x << " rzeczy.\n";

    y = rzeczy - (x*pudelka);

    cout << "pozostanie(na) " << y << " rzeczy bez przydzielonego pudelka\n";
    system("PAUSE");
    return 0;
    }
    else
    {
         cout << "Liczba pudelek wynosi 0 lub mniej, prosze o wpisanie prawidlowej wartosci";
    }
}

Na razie mój kod wygląda tak, próbowałem zrobić to, że jeśli ilosc podelek wynosi 0 lub mniej to program przejdzie do kroku gdzie trzeba podac ilosc pudelek, lecz wyszło mi to, że program się kończy, proszę o szybką pomoc :)

Answer 3

Jeżeli doszedłeś do wyrażeń warunkowych, to możesz zabezpieczyć program, przed wpisaniem liczby 0 lub mniejszej.

Comments

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

using namespace std;

int rzeczy, pudelka,x,y;

main ()
{
    cout << "Program wyliczajacy ilosc rzeczy przypadajacych na pudelko\n";
    cout << "podaj ilosc rzeczy:";
        cin >> rzeczy;
    cout << "podaj ilosc pudelek:";
        cin >> pudelka;

    if (pudelka > 0)
    {

        x = rzeczy / pudelka;

    cout << "\n\nNa jedno pudelko przypada(ja) " << x << " rzeczy.\n";

    y = rzeczy - (x*pudelka);

    cout << "pozostanie(na) " << y << " rzeczy bez przydzielonego pudelka\n";
    system("PAUSE");
    return 0;
    }
    else
    {
         cout << "Liczba pudelek wynosi 0 lub mniej, prosze o wpisanie prawidlowej wartosci";
    }
} 

NO tak, próbowałem zabezpieczyć program, ale gdy go zabezpieczam to on się kończy, a chcę żeby znów przeszedł do punktu: podaj ilosc pudełek, jak to zrobić, proszę o szybką pomoc :D

---

Aha, to nie twoja wina. Musisz użyć pętli. W skrócie wyglądało by to tak: do {cin>>pudelka;}while(tutaj warunek trwania); Możesz to zastosować do rzeczy.

---

Możesz bardziej rozpisać? bo nie rozumiem

---

Obecnie jestem na telefonie, także większy kod odpada. Jakiś kolega Ci pomoże, lub postaram się to zrobić jutro po szkole.

Answer 4

Brak zabezpieczeń.

Comments

Czy mógłbyś napisać to jaśniej, bo jeszcze nie doszedłem do etapu zabezpieczeń :)

---

Wpisz sobie  za pudełka 0 i zobacz co dostaniesz...

powinieneś mieć zabezpieczenie przed dzieleniem przez 0, przed wpisaniem czegoś innego niż liczba itd.

---

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

using namespace std;

int rzeczy, pudelka,x,y;

main ()
{
    cout << "Program wyliczajacy ilosc rzeczy przypadajacych na pudelko\n";
    cout << "podaj ilosc rzeczy:";
        cin >> rzeczy;
    cout << "podaj ilosc pudelek:";
        cin >> pudelka;

    if (pudelka > 0)
    {

        x = rzeczy / pudelka;

    cout << "\n\nNa jedno pudelko przypada(ja) " << x << " rzeczy.\n";

    y = rzeczy - (x*pudelka);

    cout << "pozostanie(na) " << y << " rzeczy bez przydzielonego pudelka\n";
    system("PAUSE");
    return 0;
    }
    else
    {
         cout << "Liczba pudelek wynosi 0 lub mniej, prosze o wpisanie prawidlowej wartosci";
    }
}

Pomóż mi, próbowałem zrobic to co mówiłeś, ale jak będzie 0 lub mniej to wtedy program się kończy a chcę żeby wrócił tam gdzie trzeba podać ilosc pudełek, żeby zmienić liczbę, jak to zrobić?

---

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

using namespace std;

int rzeczy, pudelka,x,y;

main ()
{
    cout << "Program wyliczajacy ilosc rzeczy przypadajacych na pudelko\n";



while(true)
{


    cout << "podaj ilosc rzeczy:";
     cin >> rzeczy;
    cout << "podaj ilosc pudelek:";
    cin>>pudelka;

            if (pudelka>0)
            x = rzeczy / pudelka;
    else
    {


        cout<<"nie mozna dzielic przez 0";
        break;
    }
    cout << "\n\nNa jedno pudelko przypada(ja) " << x << " rzeczy.\n";

    y = rzeczy - (x*pudelka);

    cout << "pozostanie(na) " << y << " rzeczy bez przydzielonego pudelka\n";
}




       system("PAUSE");
    return 0;
}