стек — не имеет члена с именем Переполнение стека
У меня есть некоторые проблемы. Я создал программу с динамическим стеком и при компиляции получаю ошибки: У класса stos нет члена с именем push, у класса sto нет члена с именем pop, у класса sto нет члена с именем destroy, у класса sto нет члена с именем isempty. На языке C программа в порядке, но на C ++ нет. Я начал свое приключение с C ++, и я понятия не имею, что я делаю не так. Вы можете мне помочь ? Я ставлю код:
#include <stdio.h>
#include <stdlib.h>
#include <stdbool.h>
#include <iostream>
#include <conio.h>
using namespace std;
class stos
{
public:
int *tab;
int licznik;
int rozmiar;
};
void init (class stos* s)
{
s->licznik = 0;
s->rozmiar = 3;
//s->tab = (int*) malloc ( 2 * sizeof * s->tab );
s->tab = (int*) malloc ( s->rozmiar * sizeof(int) );
if (s->tab == NULL)
{
cout << "Blad alokacji pamieci\n";
abort ();
}
}
void push (class stos* s, int element)
{
if (s->licznik != s->rozmiar)
{
cout << "Dodaj element do stosu\n";
cin >> element;
s->tab[s->licznik] = element;
s->licznik++;
}
else
{
s->rozmiar = 2 * (s->rozmiar);
s->tab = (int*) realloc (s->tab, (s->rozmiar) * sizeof (int));
cout << "Zwiekszono pamiec\n";
cout <<"Dodaj element do stosu\n";
cin >> element;
s->tab[s->licznik] = element;
s->licznik++;
}
//system ("clear");
}
int pop (class stos* s)
{
if (s->licznik == 0)
{
cout << "Nie podales zadnego elementu\n";
abort ();
}
else
{
s->licznik--;
}
return s->tab[s->licznik];
}
void destroy (class stos* s)
{
free (s->tab);
}
bool isempty (class stos* s)
{
bool empty = true;
if (s->licznik == 0)
{
empty = true;
cout << "Stos nie posiada zadnych elementow\n";
}
else
{
empty = false;
cout << "Stos zawiera jakies elementy\n";
}
return empty;
}
int main ()
{
int element = 0, a = 0;
stos stos1;
stos stos2;do
{
cout << "\nMENU\n\n";
cout << "STOS I\nAby dodac element do stosu wcisnij '1'\nAby odjac element ze stosu wcisnij '2'\n";
cout << "Wcisnij '3' aby usunac stos\nWcisnij '4' aby sprawdzic czy stos zawiera elementy\n\n";
cout << "STOS II\nAby dodac element do stosu wcisnij '5'\nAby odjac element ze stosu wcisnij '6'\n";
cout << "Wcisnij '7' aby usunac stos\nWcisnij '8' aby sprawdzic czy stos zawiera elementy\n\n";
cout << "Wcisnij '0' aby zakonczyc dzialanie programu\n\n";
cin >> a;
//system ("clear");
if (a == 1)
{
stos1.push (element);
}
if (a == 2)
{
cout << "Usunieto element:" << stos1.pop;
}
/*if (a == 3)
{
destroy (&stos1);
printf ("Wyczyszczono stos 1\n");
}*/
if (a == 4)
{
stos1.isempty;
}
if (a == 5)
{
stos2.push (element);
}
if (a == 6)
{
cout << "Usunieto element: " << stos2.pop;
}
/*if (a == 7)
{
destroy (&stos2);
printf("Wyczyszczono stos 2\n");
}*/
if (a == 8)
{
stos2.isempty;
}
}
while (a!=0);
destroy (&stos1);
destroy (&stos2);
getch();
return 0;
}
Решение
Это потому, что функции stos, pushи т. д. являются свободными функциями, но вы называете их как методы. Вы должны назвать их так:
stos s;
init(&s)
push(&s, 3)
Или лучше, вы можете преобразовать функции в методы:
class stos {
public:
stos() // constructor instead of init
{
// access members tab, etc. directly
licznik = 0;
// ...
}
void push(int element)
{
}
// ...
private:
int *tab;
int licznik;
int rozmiar;
};
Другие решения
Проблема в том, что «в классе stos нет члена с именем push» и так далее. push а также pop не являются членами функции. Похоже, вам нужно что-то в форме
push (&stos, element);
а также
pop(&stos1);
В вашем коде вы объявляете класс stos, Класс содержит только несколько полей.
Позже в вашем коде вы определяете функции
void push (class stos* s, int element)
int pop (class stos* s)
bool isempty (class stos* s)
В основном это правильно. Вы можете просто создать экземпляр класса stos а затем вызвать одну из функций и передать stos экземпляр в качестве аргумента функции. В этом случае вам нужно сделать вызов функции следующим образом:
push (stos1, element);
вместо:
stos1.push (element);
Другой способ, как другие, возможно, уже предложили, это добавить функции в ваш класс. В этом случае ваше определение класса будет выглядеть так:
class stos
{
public:
int *tab;
int licznik;
int rozmiar;
void push (int element)
int pop ()
bool isempty ()
};
Если вы создадите класс таким образом, вы можете оставить вызовы функций в главном как они есть сейчас.
Если вы хотите иметь функции-члены, вы должны написать их, например, push может быть написано так
class stos
{
public:
void push (int element);
int *tab;
int licznik;
int rozmiar;
};
void stos::push (int element)
{
if (licznik != rozmiar)
{
cout << "Dodaj element do stosu\n";
cin >> element;
tab[licznik] = element;
licznik++;
}
else
{
rozmiar = 2 * (rozmiar);
tab = (int*) realloc (tab, (rozmiar) * sizeof (int));
cout << "Zwiekszono pamiec\n";
cout <<"Dodaj element do stosu\n";
cin >> element;
tab[licznik] = element;
licznik++;
}
//system ("clear");
}
Это похоже на код, который был переведен с C, но вы прошли только половину пути.
Просто даю подсказку для «поп» проблемы:
Сделайте это методом класса stos:
class stos
{
public:
int *tab;
int licznik;
int rozmiar;
int pop( void );
};
Реализуйте метод:
int stos::pop (void )
{
if (licznik == 0)
{
cout << "Nie podales zadnego elementu\n";
abort ();
}
else
{
licznik--;
}
return tab[licznik];
}
Но вы должны действительно узнать больше о class прежде чем идти дальше
Функции, которые вы пытаетесь вызвать, находятся не в классе stos
Вы могли бы сделать что-то вроде этого:
class stos
{
public:
int *tab;
int licznik;
int rozmiar;
void init ();
void push (int element);
int pop ();
void destroy ();
bool isempty ()
};
void stos:: init ()
{
this->licznik = 0;
this->rozmiar = 3;
//this->tab = (int*) malloc ( 2 * sizeof * this->tab );
this->tab = (int*) malloc ( this->rozmiar * sizeof(int) );
if (this->tab == NULL)
{
cout << "Blad alokacji pamieci\n";
abort ();
}
}
void stos:: push (int element)
{
if (this->licznik != this->rozmiar)
{
cout << "Dodaj element do stosu\n";
cin >> element;
this->tab[this->licznik] = element;
this->licznik++;
}
else
{
this->rozmiar = 2 * (this->rozmiar);
this->tab = (int*) realloc (this->tab, (this->rozmiar) * sizeof (int));
cout << "Zwiekszono pamiec\n";
cout <<"Dodaj element do stosu\n";
cin >> element;
this->tab[this->licznik] = element;
this->licznik++;
}
//system ("clear");
}
int stos:: pop ()
{
if (this->licznik == 0)
{
cout << "Nie podales zadnego elementu\n";
abort ();
}
else
{
this->licznik--;
}
return this->tab[this->licznik];
}
void stos:: destroy ()
{
free (this->tab);
}
bool stos:: isempty ()
{
bool empty = true;
if (this->licznik == 0)
{
empty = true;
cout << "Stos nie posiada zadnych elementow\n";
}
else
{
empty = false;
cout << "Stos zawiera jakies elementy\n";
}
return empty;
}
Учебный класс stos не имеет ни одной из этих функций-членов, изучая ваш код, я думаю, что у вас есть два возможных решения. Или вы добавляете его в класс stos, что-то вроде:
class stos
{
public:
int *tab;
int licznik;
int rozmiar;
void init ()
{
s->licznik = 0;
s->rozmiar = 3;
//s->tab = (int*) malloc ( 2 * sizeof * s->tab );
s->tab = (int*) malloc ( s->rozmiar * sizeof(int) );
if (s->tab == NULL)
{
cout << "Blad alokacji pamieci\n";
abort ();
}
}
void push (int element)
{
if (s->licznik != s->rozmiar)
{
cout << "Dodaj element do stosu\n";
cin >> element;
s->tab[s->licznik] = element;
s->licznik++;
}
else
{
s->rozmiar = 2 * (s->rozmiar);
s->tab = (int*) realloc (s->tab, (s->rozmiar) * sizeof (int));
cout << "Zwiekszono pamiec\n";
cout <<"Dodaj element do stosu\n";
cin >> element;
s->tab[s->licznik] = element;
s->licznik++;
}
//system ("clear");
}
...
};
или вы называете их правильно:
push(&stos1, element);
вместо
stos1.push(element);
С другой стороны, если вы решите не включать их в stos Класс, пожалуйста, рассмотрите возможность использования ссылок вместо указателей:
void push (class stos& s, int element);
Используйте ссылки, где вы можете, указатели, где вы должны. 😉