Totalny nowicjusz [zerknij w kod].

Question

Cześć.

Tak dla odmiany przed świętami... choinka.

Na wstępie trzy (4) słowa. Jestem zupełnym nowicjuszem. Po obejrzeniu czterech odcinków kursu C++ Pana Mirosława, wykonując znalezione w internecie ćwiczenia trafiłem na słynną prostą choinkę. Przez pierwsze minuty nie szło, więc napisałem trochę więcej dla utrwalenia. Oczywiście jest to zlepek poznanych instrukcji, na pewno w części wykorzystanych nie nie do końca do tego, do czego zostały stworzone :). Ot tak dla zabawy.

Wygarnij mi proszę przed Świętami co razi w kodzie od samego początku lub wskaż błąd/problem, ale bez rozwiązania.

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

using namespace std;

int liczba, czas=100;
string znak;

int main()
{
    cout<<"Wpisz jeden znak, z ktorego chcesz zbudowac choinke: "<<endl;
    cin>>znak;
    cout<<"Podaj liczbe pieter choinki: "<<endl;
    cin>>liczba;
    system("cls");

    if(liczba<20)
    {
        for(int m=1;m<=liczba*2;m++)
            cout<<" ";
        cout<<"[]";

        for(int i=0;i<=liczba ;i++)
        {
            czas=czas+50;
            Sleep(czas);
            for(int l=liczba*2;l>=i;l--)
                cout<<" ";
            for(int k=1;k<=i*2;k++)
                cout<<znak;
            cout<<endl;
        }

        for(int m=1;m<=liczba*2;m++)
            cout<<" ";
        cout<<znak<<znak;
        cout<<endl<<endl;
        Sleep(2000);
        system("cls");
        for(int p=1;p<=5;p++)
        {
            system("cls");
            cout<<"W E S O L Y C H   S W I A T   !";
            Sleep(500);
            system("cls");
            Sleep(500);

        }
    }
    else
    {
        cout<<"Choinka moze miec maksymalnie 20 pieter"<<endl;
        cout<<"Wyzsza niestety sie przewroci. Zmniejsz liczbe pieter o "<<liczba-20<<" i sprobuj ponownie.";
    }

    return 0;
}

Pozdrawiam

Answer 1

U mnie ten kod się nie kompiluje. Podobnie nie skompiluje się na przykład na ideone

Spróbuj dowiedzieć się dlaczego, a później napisać kod w taki sposób aby kompilował się wszędzie.

Błędów jest sporo jedne mniejsze inne większe, ale na wszystko przyjdzie czas.

Spróbuj napisać jakieś proste, krótkie programiki tym razem z mniejszą ilością forów ;) Skup się bardziej na ifach bo są ważniejsze i częściej wykorzystywane.

W każdym razie jak na początek to nie jest źle. 

Comments

Dodaj bibliotekę #include <string>. Ogólnie wydaje mi się, że lepiej jest użyć "char" zamiast "string", jeśli chce się używać jednego znaku do budowy choinki, ale poprawcie mnie, jeśli się mylę. Wtedy przynajmniej nie trzeba też tej biblioteki dodawać.

---

Ale kolega przerobił dopiero 4 lekcje Zelenta więc na tym na razie bazuje.