Zadanie C++ z pętlami.

Question

Witam,

ostatnio zająłem się nauką programowania i nalazłem takie zadanie: Pan Kowalski zbudował ścianę z cegieł. Każda cegła ważyła 2 kilogramy. U podstawy ściany znajdowało się 15 cegieł. Każda warstwa wyższa posiadała o jedną cegłę mniej. Próbowałem je wykonać jednak napotkałem kilka trudności, oto kod:

#include <iostream>

using namespace std;

int main()
{
   int waga, ilosc, c1;

    cout << "Wpisz wage jednej cegly w kilogramach: ";
    cin>>waga;
    cout << "Wpisz ilosc cegiel u podstawy piramidki: ";
    cin>>ilosc;

    for (int i=ilosc; i=0; i--)
    {
        c1=waga*ilosc*i;
    }

    cout<<"Masa scianki wynosi "<<c1<<" kg.";

    return 0;
}

Zadanie odrobinę przerobiłem, aby zwiększyć poziom trudności. Zamiast ustalonych wartości dałem możliwość wpisania własnych. Niestety kod nie działa, myślę, że to powód złego zapisu pętli lub samych zmiennych. Nieważne co wpiszę to zawsze "c1" wynosi 1989087586. Prosiłbym aby ktoś wytłumaczył mi co jest źle i najlepiej pozmieniał to w kodzie. Będę wdzięczny za pomoc.  

Answer 1

Po pierwsze w twojej pętli znajduje się zły zapis. Zainicjowałeś jakieś i = ilości, a następnie podałeś warunek dopóki i = 0, więc pętla się nie wykona ani razu. Wyskakuje Ci wartość 1989087586 ponieważ nie nadałeś żadnej wartości zmiennej "c1", a że pętla się nie wykonywała to wyskakiwała Ci taka magiczna liczba. Więc pierwsze co powinieneś zrobić to nadać swojej zmiennej początkową wartość równą 0.

int waga, ilosc, c1 = 0;

Następnie pętla for powinna posiadać inny warunek w tym przypadku powinno wyglądać to tak :

for (int i=ilosc; i>0; i--)

Ostatnią sprawą jest wzór wewnątrz pętli. Dodałeś tam niepotrzebną zmienną "ilość". Zauważ że dla przykładowych danych : cegła 2kg i 3 cegły w pierwszym rzędzie przyporządkujesz zmiennej "c1" wartość 2kg*3*3 - ponieważ twoje i początkowo jest równe ilości, później będzie to 2kg*3*2 bo zmniejszasz i o 1 i tak ciągle. Ostatnią sprawą jest fakt, iż zapis :

c1=waga*ilosc*i;

ciągle zmienia wartość zmiennej c1, a nie ją sumuje. Abyś otrzymał poprawny wynik musisz kolejne wyniki sumować więc powinieneś zapisać to w taki sposób :

 c1 +=(waga*i);

Cały zmodyfikowany kod prezentuje się tak :

#include <iostream>

using namespace std;

int main()
{
   int waga, ilosc, c1 = 0;

    cout << "Wpisz wage jednej cegly w kilogramach: ";
    cin>>waga;
    cout << "Wpisz ilosc cegiel u podstawy piramidki: ";
    cin>>ilosc;

    for (int i=ilosc; i>0; i--)
    {
        c1 +=(waga*i);

    }
         cout<<"Masa scianki wynosi "<<c1<<" kg.";


    return 0;
}

 

Answer 2

W warunku w pętli nie powinno być przypadkiem i > 0? Poza tym:

c1=waga*ilosc*i;

Chodzi chyba o dodawanie i przypisywanie, a nie samo przypisywanie:

c1+=waga*ilosc*i;

Poza tym po co mnożysz przez ilość? Przecież ilość cegieł w jednej warstwie siedzi w zmiennej i:

c1+=waga*i;

I jeszcze jedno: próbowałeś robić to zadanie na kartce? Pętle nie są tu potrzebne, gdyż masa takiej ściany to:

Gdzie m to masa jednej cegły, a s to suma liczb całkowitych z przedziału [1, ilość_cegieł_przy_podstawie].

Answer 3

Masz kilka błędów

1. Musisz nadać początkową wartość równą 0 zmiennej c1
2. Źle postawiony warunek w pętli
3. Źle obliczasz c1, po co to mnożenie przez ilość? :D

Answer 4

W warunku masz przypisanie (=) zamiast porównania (==).