Question

当我尝试将for_each算法与std::unique_ptr一起使用时，我遇到以下错误。我在下面的 profile.h 部分放大了它。

奇怪的是，如果我将其更改为std::shared_ptr，我就能编译，我怀疑它是按值获取容器的元素，因此不喜欢对unique_ptr的引用。但是，我希望它理想情况下为unique_ptr，因为这些任务应该放在ToRun_容器中，并在任务执行后转移到Completed_容器，因此shared_ptr不会在这里做任何好事。

我得到的错误是：

没有匹配函数来调用类型'的对象（Lambda at Profile.cpp：429：54）'

__ F（* __第一）;

这是指这行代码：

for_each(ToRun_.begin(), ToRun_.end(), [&os](std::unique_ptr<Task>& e){
  if (e){
    os << e->getName() <<'\n';
  }
});

在我将其转换为使用智能指针之前，我使用了原始指针，我可以保证e->getName() 100％正常工作。我的困惑是为什么在这种情况下它不起作用？我该怎么做才能使它正常工作？

Profile.h

#include <algorithm>
#include <vector>
#include <map>
#include <iostream>
#include "Task.h"
#include "Global.h" //Where my user defined global functions go

class Profile{
  std::vector<std::unique_ptr<Task>>ToRun_;
  std::vector<std::unique_ptr<Task>>Completed_;
  std::vector<std::string>menu_;
  //Ownership of ofstream object
  std::ofstream& of_;
public:
  Profile (const char* filename, std::ofstream& os, ARAIG_sensors& as);
  virtual ~Profile();
  void run();
  //Executes specified number of tasks user specifies
  void execute (unsigned long tasks);
  void load_menu();
  long show_menu()const;
  long getInput(std::string prompt, int min, long max, menuOption option = NONE);
  //Display all tasks to the screen
  std::ostream& display_todo_tasks (std::ostream& os) const;
  //Display completed tasks to the screen
  std::ostream& display_completed_tasks (std::ostream& os)const;
  //Display next task to the screen
  std::ostream& display_next_task (std::ostream& os) const;
  //Display last completed task
  std::ostream& display_last_task(std::ostream& os)const;
};

Profile.cpp

std::ostream& Profile::display_todo_tasks(std::ostream& os)const{
  //Display all tasks in ToRun container
  if(ToRun_.size() > 0){
    new_line(user_interface_skip_screen);
    std::cout << "\nTasks to be completed\n";
    print_dash(29);
    for_each(ToRun_.begin(), ToRun_.end(), [&os](std::unique_ptr<Task>& e){
      if (e){
        os << e->getName() <<'\n';
      }
    });
    new_line(user_interface_system_message_skip_line - 1);
  }else{
    std::cerr << "There are no tasks to be performed in the task list.";
    std::cerr.flush();
    new_line(user_interface_system_message_skip_line);
  }
  return os;
}

Answer 1

std::ostream& Profile::display_todo_tasks(std::ostream& os)const

您的功能是const功能。因此，ToRun_是const个对象。因此，您从begin和end获得的迭代器是const_iterator。因此，当您取消引用它们时所获得的是 const引用。

你的lambda采用非const引用。因此不兼容。它需要const引用，它应该没问题。

将for_each与std :: unique_ptr

Profile.h

Profile.cpp

1 个答案: