SPOJ-STOS-BŁĘDNA ODPOWIEDŹ.

Question

Witam, zrobiłem zadanie pt. "Stos" i wyświetla mi błędną odpowiedź, a mam wrażenie że wszystko jest jak należy, chyba że błędem jest gdy wpisuje tam ostatnią liczbę którą mam dodać czyli mam napisać "+" i później "1" to po zaakceptowaniu "+" enterem wyświetla się ":(" przed wpisaniem "1". Ale mogę wpisać "+1"razem i wtedy się minka wyświetla po 1.
Nie wiem, czy to przez to i nie mam pomysłu jak to naprawić, miałem to samo napisane z tym że wybór był getchar(),

ale miałem błąd kompilacji, i przeczytałem gdzieś żeby usunąć bibliotekę conio.h.

Proszę o udzielenie wskazówki, z góry dziękuje za pomoc.

#include <iostream>
using namespace std;
int ile;
int *dane;
int rozmiar;
void push()
{
    if(rozmiar>=ile)
     cout<< ":("<<endl;
    else
    {
    rozmiar= rozmiar+1;
    cin>>dane[rozmiar];
    cout<<":)"<<endl;
    }
}
void pop()
{
    if(rozmiar>0)
    {
        cout<< dane[rozmiar]<<endl;
        rozmiar=rozmiar-1;
    }
    else
        cout<<":("<<endl;
    }
int main()
{
    cin>> ile;
    dane =new int [ile];
    char wybor;
    rozmiar=0;
    while(wybor!=ile)
    {
    cin>>wybor;
    switch(wybor)
    {
        case '+':
        push();
        break;
        case '-':
        pop();
        break;

    }
    }

    delete [] dane;
    return 0;
}

 

Comments

W zdaniu powiedziane jest, że na wejście podana jest nieokreślona liczba zestawów danych, więc niepotrzebnie wczytujesz wartość do zmiennej ile. No i stos powinien być symulowany na tablicy rozmiaru 10.

---

Zrobiłem tak jak mówiłeś, dalej nie przyjmuje zadania, wydaje mi się że problemem jest tak jak wspominałem minka ":(" która wyświetla się przed wpisaniem "1" . Próbowałem kombinować, zamieniać ale nie działa ;/

#include <iostream>
using namespace std;
int dane[10];
int rozmiar;
void push()
{
    if(rozmiar>=10)
        cout<<":(";


      else
        {
         cin>>dane[rozmiar];
         rozmiar=rozmiar+1;
        cout<<":)"<<endl;

        }
}
void pop()
{
    if(rozmiar>=1)
    {

        cout<<rozmiar[dane]<<endl;
         rozmiar=rozmiar-1;

    }
    else
    {
        cout<< ":("<<endl;
    }
}
int main()
{
int ile;
cin>>ile;
while(ile--)
{
    char wybor;
    rozmiar=0;
    while(wybor!=10)
    {
        cin>>wybor;
        switch(wybor)
          {
          case '+':
            push();
          break;
          case '-':
            pop();
          break;
          }
    }
}
    return 0;
}

 

---

Oj znowu wczytujesz wartość do zmiennej ile. W zadaniu nie ma mowy o takiej wartości. Dostaniesz po prostu nieokreśloną liczbę poleceń i musisz je wykonać. To co robi się w przypadkach gdy nie ma z góry narzuconej ilości zapytań to piszesz:

while(cin >> wybor)
{
      switch(wybor)
      {...}
}

A co do tego wypisywania ":(" przed wczytaniem wartości to nie ma się co martwić, gdyż dla SPOJa nie robi to większej różnicy

---

Bardzo dziękuje za pomoc.