, используя список для хранения объектов

Question

Я хочу знать, правильно ли это или есть ли способ хранения разных объектов одного класса и использования операций класса?

class Camera_ { 

//function1 
//function2 
//function3 

} 

int main() 
{ 
    int n ; 
    std::cout <<"\n How many Cameras you are using ? \n"; 
    std ::cin >> n; 
    std:: cout << "The number of camera is " << n << std ::endl ; 

    std::list<Camera_> list1; // creating a list of camera 



    for(int j=0; j<n; j++) //transfer array 
    camera = new Camera_;// camera object   
    list1.push_back(camera);// storing camera objects 
    list<Camera_>::iterator it; 
    for(it = iList.begin(); it != iList.end(); it++) 
    *it->function1(); 
    *it->function2(); 
    *it->function3(); // I am trying to access 3 function of the class for each objects of camera class. 

    return 0; 
} 

Answer 1

То, что вы показали, не соответствует действительности. Это должно выглядеть примерно так:

class Camera { 
public: 
    void function1() { 
     // ... 
    } 
    void function2() { 
     // ... 
    } 
    void function3() { 
     // ... 
    } 
}; 

int main() 
{ 
    int n; 
    std::cout << "How many Cameras you are using? "; 
    std::cin >> n; 
    std::cout << "The number of Cameras is " << n << std ::endl; 

    std::list<Camera> list1; // creating a list of camera 

    for (int j = 0; j < n; ++j) { 
     list1.push_back(Camera()); // storing camera objects 
     // or, if using C++11 or later: 
     // list1.emplace_back(); 
    } 
    // or simply: 
    // list1.resize(n); 

    list<Camera>::iterator it; 
    for(it = list1.begin(); it != list1.end(); ++it) { 
     it->function1(); 
     it->function2(); 
     it->function3(); 
    } 

    return 0; 
} 

Answer 2

Here'r некоторые предложения, которые могут улучшить эффективность и другие вещи:

1. Если вы не делаете частые вставки в конкретных индексов, использование вектора вместо список, вы получите случайные acces (Это зависит от вас usecase).

2. Использование смарт-указатель - вы сохраняете указатель класса Camera_

   камера = новая камера _; // объект камера
   list1.push_back (камера); // хранение камеры объекты

Измените это на:

shared_ptr<Camera_> sptr = make_shared(new Camera_); 
list1.push_back(sptr);// storing camera objects 

Лучше использовать интеллектуальные указатели, поскольку вы избегаете бремени удаления память. Использование raw-указателя может привести к утечке памяти даже после надлежащего ухода (например, во время исключений).