DeleteAllNodes 에서 질문있습니다.

안녕하세요 .

교수님의 DeleteAllNodes 코드는 아래와같습니다.

 void DeleteAllNodes(Node*root){

if (root == NULL) { return; }

Node* pright = root->right;

DeleteAllNodes(root->left);

free(root);

DeleteAllNodes(pright);

}

하지만 이 코드는 Tree의 root 를 NULL로 바꿔주지못해 실제 활용시 에러를 발생시키는데요 따라서

Node**ptr 을 Parameter로 사용하거나  Tree를 활용해야하는데  재귀호출을 사용하다보니 Node**ptr을 Parameter로 받기는 조금 번거롭고 

따라서 Tree를 활용하기위해 아래와같이 

DeleteAllItems 라는 함수를 따로만들어 

 void DeleteAllNodes(Node*root) {

/*

if (root == NULL) { return; }

Node* pright = root->right;

DeleteAllNodes(root->left);

free(root);

DeleteAllNodes(pright);

*/

if (root == NULL) { return; }

Node* pright = root->right;

if (root->left != NULL) { DeleteAllNodes(root->left); }

free(root);

if (root->left != NULL) { DeleteAllNodes(pright); }

}

 void DeleteAllItems(Tree* tree) {

if (tree->root == NULL) { return; }

DeleteAllNodes(tree->root);

tree->root = NULL;

tree->m_items = 0;

 }

이렇게 구현하는것이 더 좋지않나 싶은데 선생님은 어떻게 생각하시나요?