[C++]Stack

[Luc@s]

#include <iostream>
#include <cstdio>
#include <string>
#include <vector>

using namespace std;

template<class T>
class Stack
{
public:
// Constructor with dimension
Stack(int dim) { vet = new T[checkDim(dim)]; };
// Constructor without dimension
Stack() { vet = new T[5]; };
// Check and verify the chosed dimensione
int checkDim(int dim) { return(dim <= 0)?5:dim; };
// The destructor
~Stack(){};
// add a element
void push(T value) { vet[sp++] = value; };
// return a element (and no delete)
T pop() { return vet[sp--]; };
// return a element (and delete)
T pop_d()
{
int index =sp--;
T tmp = vet[index];
delete vet[index];
return tmp;
};
private:
T vet;
int sp;
};



Mi da (riga in neretto):
P:/C++/stack/stack.h:21: invalid types `int[int]' for array subscript
Tnk 100000000000

[Molz]

Penso ke l'errore venga xké vet é dikiarato come un elemento di tipo T e nn come un vettore di T o un puntatore a T.

A qesto punto nn capisco xké nn t da errore il costruttore.

Cmq prova a sostituire

private:
T vet;
int sp;

con

private:
T * vet;
int sp;

[cionci]

Oltre a quello segnalato...

delete vet[index];

Questo non si può fare...non puoi cancellare uno alla volta gli elementi di un vettore allocati tutti insieme...

I due costruttori li metterei in uno solo (e mancava l'inizializzazione di sp e di un'altra variabile membro che ti tiene la dimensione, altrimenti puoi andare inoverflow):

Stack(int d = 5):dim(d),sp(0) { vet = new T[checkDim(dim)]; };

Sulla push devi controllare se sp >= dim...e in tal caso devi generare un'eccezione...
Sulla pop devi controllare se sp < 1...e in tal caso devi generare un'eccezione...
La pop è sbagliata...se fai la push così:

vet[sp++] = value;

allora devi fare la pop così:

return vet[--sp];

checkDim la metterei come privata...

[verloc]

E' buona norma inoltre,indicizzare un qualsiasi container con unsigned int.
...se uno fa container[-3] ?

ciao ;)