这就是为什么您应该使用std::vector<>，因为它将为您正确地处理所有内存管理。
问题1：new从不返回NULL（对于pedants，如下所示的普通new）。还要注意的是，在C++中，我们使用nullptr而不是NULL来保证类型安全。

map[i] = new int [columns];
 if (map[i] == NULL)                // This will never be NULL
     cout << “No enough memory”;    // So this will never print anything.

如果fillmaze()是从构造函数调用的，那么如果它抛出异常，析构函数就不会被调用，这意味着你必须处理任何分配失败。

map = new int* [rows];              // could throw.
    for (int i = 0; i < rows; i++) {
        map[i] = new int[columns];      // could throw
    }

因此，您必须处理并能够检测到已经分配了哪些内容，以及哪些内容需要重新分配。

try
{
    map = nullptr;                      // Init to nullptr to make sure
                                        // you can distinguish between valid state
                                        // and what could have randomly been in memory.

    map = new int* [rows]{nullptr};     // Initialize all elements to nullptr.
                                        // If any throw you don't want your code
                                        // to start releasing random pointers.
                                        // calling delete on nullptr is fine.

    for (int i = 0; i < rows; i++) {
        map[i] = new int[columns]{PASSAGE};
    }
}
catch(...)   // catch all exceptions.
{
    if (map != nullptr) {                   // You should only loop over
                                            // map if it has been allocated.
        for (int i = 0; i < rows; i++) {
            delete [] map[i];
        }
    }
    delete [] map;                         // delete the map.
    throw;                                 // rethrow exception

}

你的析构函数是丁。
但你可以用一个向量来简化这个过程：

std::vector<std::vector<int>>   maze{std::vector<int>{PASSAGE, columns}, rows};