C++ DAL - Powrót referencyjny lub wypełnić Przekazywana w odniesieniu

Question

[EDIT 1 - dodał składnia trzeci wskaźnik (Dzięki Alex)]

Która metoda wolisz dla DAL i dlaczego z:

Car& DAL::loadCar(int id) {} 
bool DAL::loadCar(int id, Car& car) {} 
Car* DAL::loadCar(int id) {} 

Jeśli nie uda się znaleźć samochodu, pierwsza metoda zwróci wartość null, druga metoda zwróci wartość false.

Druga metoda spowoduje utworzenie obiektu Car na stercie i wypełnienie danymi wyszukanymi z bazy danych. Przypuszczalnie (mój C++ jest bardzo zardzewiały) oznaczałoby to kod wzdłuż linii:

Car& DAL::loadCar(int id) 
{ 
    Car *carPtr = new Car(); 
    Car &car= *carPtr; 
    car.setModel(/* value from database */); 
    car.setEngineSize(/* value from database */); 
    // etc 
    return car; 
} 

Thanks

Answer 1

Drugi jest zdecydowanie korzystne. Zwracasz odwołanie do obiektu, który został nowy. Dla użytkownika końcowego korzystającego z oprogramowania nie jest oczywiste, że zwracany obiekt wymagałby usunięcia. PLUS jeśli użytkownik robi coś takiego, to wskaźnik zgubiłby się.

Druga metoda polega zatem na sterowaniu pamięcią osoby dzwoniącej i powstrzymuje wszelkie dziwne błędy.

Edycja: Powrót przez referencję jest sensowny, ale tylko wtedy, gdy klasa rodzica, tj. DAL, kontroluje długość okresu odniesienia. tzn. jeśli klasa DAL zawiera wektor obiektów Car, wtedy zwrot referencji byłby sensowną rzeczą do zrobienia.

Edycja2: Nadal wolałbym drugi zestaw. Trzecia jest znacznie lepsza niż pierwsza, ale kończy się na tym, że wywołujący zakłada, że ​​obiekt jest inicjowany.

Można również

Car DAL::loadCar(int id); 

i nadzieją przyjąć kopię stosu.

Pamiętaj również, że możesz utworzyć rodzaj obiektu samochodu o wartości NULL, dzięki czemu zwrócisz obiekt będący "poprawnym" hash, ale nie zwróci on żadnych użytecznych informacji we wszystkich polach (i dlatego jest oczywiście inicjalizowany do danych śmieciowych)). To jest wzór obiektu zerowego.

Answer 2

Ponieważ tak czy inaczej przydzielasz obiekty na stercie, dlaczego nie rozważyć Car * LoadCar(), która zwraca NULL, jeśli wystąpi problem. W ten sposób nie ma żadnych ograniczeń z typami referencyjnymi (każda referencja musi być zainicjalizowana), a także mają środki do sygnalizowania przypadku błędu.