Pourquoi est-ce une mauvaise pratique de créer un objet dans une fonction en C++?

Question

Je suis débutant en C++. L'intervieweur a dit que c'est une mauvaise pratique de créer un objet dans une fonction. Est-ce vrai?

#include <iostream.h> 

class linkedlist { 
    int value; 
    linkedlist *next; 
    static linkedlist *p=NULL; 

    void insert(int data) { 
     linkedlist node; 
     node.value=data; 
     node.next=NULL; 

     if(p==NULL) 
      p=node; 
     else { 
      p->next=node; 
      p=p->next; 
     } 
    } 
} 

int main() { 
    linkedlist h; 
    h.insert(10); 
    h.insert(20); 
    return 0; 
} 

Answer 1

Le problème est "durée de vie"; le noeud que vous avez créé ne vit que dans le cadre de pile de la fonction et lorsque la fonction se termine, sa mémoire est récupérée. Le pointeur statique p le liera quand même et cela conduira à un "comportement indéfini" quand quelqu'un accédera plus tard à la liste chaînée.

L'utilisation d'objets locaux n'est pas mauvaise en général, c'est seulement mauvais si vous attendez qu'ils survivent à la fin de la fonction, parce qu'ils ne le font pas.

Vous devez allouer des objets longue durée l'objet sur le magasin gratuit, en utilisant new.