C++ - funkcja, która zwraca obiekt

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; 
} 

Czy będzie „po użyciu wolnego” błędu na molly->eatFood()?

Answer 1

Poprawiono program i stworzył przykładową realizację 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; 
} 

To będzie produkować wyjście:

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

proponuję nauczyć się podstawowej książki o C++ deski do deski przed kontynuowaniem .

Answer 2

Nie ma błędu w zakresie nieprawidłowego użycia pamięci innej niż wyciek pamięci na końcu programu. Jeśli coś jest tworzone na stercie (na przykład z new), musisz zadzwonić pod numer delete, aby go zwolnić.

Masz również wiele błędów składniowych, poprawionych poniżej.

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

myślę, że powinno być Cat *lady i Cat *molly, ale inaczej powinno być w porządku.

Answer 4

Po utworzeniu lady na sterty (z new), nie zostanie ona zniszczona po wyjściu z metody makeCat. Dlatego połączenie pod numerem molly jest całkowicie poprawne.

ALE, masz wyciek pamięci. Musisz usunąć molly po użyciu (czasami w przyszłości). Ponieważ twój program się kończy, nie jest to wielka sprawa. W większym programie byłaby to bardzo ważna sprawa.

Answer 5

new Cat("fluffy") tworzy wskaźnik. Będziesz musiał określić Cat* jako typ zwracany. Ponieważ obiekt jest tworzony w stercie, będzie on nadal dostępny po zwróceniu funkcji.

Answer 6

Problem nie jest "Użyj po bezpłatny"; bardziej prawdopodobne, że nie usuwasz nowej instancji.

Answer 7

Jeśli kompilator obsługuje C++ 11 można użyć unique_ptr tutaj:

#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; 
} 

Teraz nie trzeba się martwić o usunięcie utworzonego obiektu. Zostanie on usunięty, gdy tylko molowy wskaźnik wykracza poza zakres:

Cat created 
Cat is eating food 
Cat destroyed