CareerCup

There's a major problem in the implementation of the following class. Can you spot it? How can you fix
the problem? You can propose more than one solution, depending on the requirement specifications of
the class.

#include<iostream>
using namespace std;
template <class T>                            
class Array                                  
{                                              
private:                                        
    T *m_pData;                                   
    unsigned int m_nSize;                         
                                                
public:                                         
  Array(unsigned int nSize) : m_nSize(nSize)      
  {                                           
    if(m_nSize > 0)
     m_pData = new T[m_nSize];                
  }                                           
                                                
  virtual ~Array()                                
  {                    
    if(m_pData != NULL)                           
     delete [] m_pData;                          
  }
                                            
bool Set(unsigned int nPos, const T& Value)
{
  if(nPos < m_nSize)
  {
  	m_pData[nPos] = Value;
  	return true;
  }
  else
  {
  	return false;
  }
}
  
T Get(unsigned int nPos)
{
        if(nPos < m_nSize)
             return m_pData[nPos]; 
        else     
            return T();
}
};