これはベクターを初期化する悪い方法ですか？

Question

vector<vector<int>> levelOrder(TreeNode* root) { 
    vector<vector<int>> result; 
    queue<TreeNode *> que; 

    if (root != nullptr) { 
     que.emplace(root); 
    } 

    while (!que.empty()) { 
     vector<int> level; 
     int size = que.size(); 
     for (int i = 0; i < size; i++) { 
      auto *front = que.front(); 
      que.pop(); 
      level.emplace_back(front->val); 
      if (front->left != nullptr) { 
       que.emplace(front->left); 
      } 
      if (front->right != nullptr) { 
       que.emplace(front->right); 
      } 
     } 
     result.emplace_back(move(level)); 
    } 

    return result; 
} 

問題：上記https://leetcode.com/problems/binary-tree-level-order-traversal/description/

>はベクトルを返す関数です。

しかし、ベクトルをローカル変数vector<vector<int>> result;として初期化したので、それを返すコードの匂いであることを意味していますか？

ベクトルはローカル変数であるため、スタック上に割り当てられ、この関数呼び出しが終了するとベクターは破棄されます。ソフトウェアは、私はその時に停止することはできませんので

私が代わりにauto results = new vector<vector<int>>

Answer 1

号

これを行っているならば、私は詳しく説明します。 「いいえ」は質問にのみ適用されます。私はエラーのコードをチェックしていません。それで、いいえ、書いたように結果を返すのは間違いではありません。費用はかかりません。最新のコンパイラはreturn-value optimizationを使用して結果を返します。

operator newを使用することは非常に悪いフォームです。そうすることで、呼び出し関数にオブジェクトを削除する負担がかかります。 "C++ RAII"を検索してください。