Нормальный код?

class Matrix{
private:
  int w,h;
  int *arr;
public:
  Matrix(int _w, int _h){w=_w; h=_h; arr = (int*)malloc(sizeof(int)*w*h);}
  ~Matrix(){free(arr);}
  int& elem(int x, int y){return &arr[x + y*w];}
}

// например это матрица
class Matrix2D{
private:
// имена должны быть осмысленные, и принято использовать префикс m_
    int n, m, count;
    int** arr;
public:
// явное указание void - по желанию
    Matrix2D( void ) : n(1), m(1), count(1){ // инициализация переменных в конструкторе происходит так
        arr = new int*[n];
        for( int i = 0; i < 1; ++i ){
            arr[i] = new int[m];
        };
    }
 
    Matrix2D( int n_size, int m_size ) : n(n_size), m(m_size), count(1){
        arr = new int*[n];
        for( int i = 0; i < n; ++i ){
            arr[i] = new int[m];
        };
    }
 
    ~Matrix2D( void ){
        for( int i = 0; i < n; ++i ){
            delete []arr[i];  // удаление массив. нужны квадратные скобки
            delete []arr;
        }
    }
 
// надо бы дать нормальное имя методу
    void C( void ){
        for( int i = 0; i < n; ++i ){
            for( int j = 0; j < m; ++j ){
                arr[i][j] = count++;
            }
        }
    }
 
    void show( void ){
        for( int i = 0; i < n; ++i ){
            for( int j = 0; j < m; ++j ){
                std::cout << arr[i][j] << " ";
                count++;
            }
            std::cout << '\n'; // забудьте об std::endl
        }
    }
};