C++ - función que devuelve objeto

Question

// Assume class definition for Cat is here. 

Cat makeCat() { 
    Cat lady = new Cat("fluffy"); 
    return lady; 
} 

int main (...) { 
    Cat molly = makeCat(); 
    molly->eatFood(); 
    return 0; 
} 

Habrá un "uso después de liberación" error en molly->eatFood()?

Answer 1

ha corregido su programa y crearon un ejemplo de implementación class Cat:

#include <iostream> 
#include <string> 

class Cat { 
public: 
     Cat(const std::string& name_ = "Kitty") 
     : name(name_) 
     { 
       std::cout << "Cat " << name << " created." << std::endl; 
     } 
     ~Cat(){ 
       std::cout << "Cat " << name << " destroyed." << std::endl; 
     } 
     void eatFood(){ 
       std::cout << "Food eaten by cat named " << name << "." << std::endl; 
     } 
private: 
     std::string name; 
}; 

Cat* makeCat1() { 
     return new Cat("Cat1"); 
} 

Cat makeCat2() { 
     return Cat("Cat2"); 
} 

int main(){ 
     Cat kit = makeCat2(); 
     kit.eatFood(); 

     Cat *molly = makeCat1(); 
     molly->eatFood(); 
     delete molly; 

     return 0; 
} 

Se producirá la salida:

Cat Cat2 created. 
Food eaten by cat named Cat2. 
Cat Cat1 created. 
Food eaten by cat named Cat1. 
Cat Cat1 destroyed. 
Cat Cat2 destroyed. 

le sugiero que aprender un libro básica sobre el C++ principio a fin antes de continuar .

Answer 2

No hay ningún error en cuanto al uso no válido de memoria que no sea una pérdida de memoria al final de su programa. Si se crea algo en el montón (como con new), entonces necesita llamar al delete para liberarlo.

También tiene un montón de errores de sintaxis, corregidos a continuación.

Cat* makeCat() 
{ 
    Cat *lady = new Cat("fluffy"); 
    return lady; 
} 

int main (int argc, char** argv) 
{ 

    Cat* molly = makeCat(); 
    molly->eatFood(); 

    delete molly;//This was added 
    return 0; 
} 

Answer 3

creo que debe ser Cat *lady y Cat *molly, pero de lo contrario debería estar bien.

Answer 4

Dado que lady se crea en el montón (con un new), no se destruirá cuando salga del método makeCat. Entonces la llamada en molly es perfectamente válida.

PERO, tiene una pérdida de memoria. Debe eliminar molly después de usarlo (en el futuro). Ya que su programa finaliza, esto no es un gran problema. En un programa más grande, esto sería un gran problema.

Answer 5

new Cat("fluffy") crea un puntero. Tendrá que especificar Cat* como tipo de devolución. Como el objeto se crea en el montón, seguirá estando disponible después de que se devuelva la función.

Answer 6

El problema no es un "uso después de gratis"; más probablemente, no estás eliminando la nueva instancia.

Answer 7

Si su compilador compatible con C++ 11 puede utilizar unique_ptr aquí:

#include <iostream> 
#include <memory> 

using namespace std; 
class Cat { 
    public: 
     Cat() { 
      cout << "Cat created" << endl; 
     } 
     ~Cat() { 
      cout << "Cat destroyed" << endl; 
     } 
     void eatFood() { 
      cout << "Cat is eating food" << endl; 
     } 
}; 

unique_ptr<Cat> makeCat() { 
    unique_ptr<Cat> lady(new Cat); 
    return lady; 
} 

int main() { 
    unique_ptr<Cat> molly = makeCat(); 
    molly->eatFood(); 
    return 0; 
} 

Ahora, usted no tiene que preocuparse acerca de cómo eliminar objeto creado. Se eliminará tan pronto como el puntero molly salga del alcance:

Cat created 
Cat is eating food 
Cat destroyed