Проверка того, что элемент уже находится в векторе

Question

Так что я хочу выполнить некоторую проверку ошибок на векторе, который у меня есть в классе, чтобы увидеть, существует ли элемент, прежде чем добавлять новый элемент в вектор.

ClassA каст

void ClassA::func(std::shared_ptr<ClassB> new_item) 
{ 
    for(auto items : vector_) 
    { 
     if(items = new_item) 
     { 
      return; 
     } 
     vector_.push_back(new_item); 
    } 
} 

vector_ является членом класса члена станд :: вектор. В этой текущей реализации все new_item игнорируются, даже если это не дубликат. Я знаю, что «if (items = new_item)» является проблематичной линией, но я не знаю почему.

Answer 1

Вы назначая вместо сравнения равенства здесь:

if(items = new_item) // assigns value of new_item to items 

Такого рода проблемы легко избежать, используя хорошо проверенные стандартные библиотечные функции, такие как std::find:

#include <algorithm> // for std::find 
.... 
if (find(vector_.begin(), vector_.end(), newitem) == vector_.end()) 
    vector_.push_back(newitem); 

Answer 2

= присваиванию

== равенство

Вы хотите

if(items == new_item) 
    { 
     return; 
    } 

Answer 3

У вас есть опечатка:

if(items = new_item) 

должен быть

if (items == new_item) 

Btw вы можете написать

if (std::find(std::begin(vector_), std::end(vector_), new_item) == std::end(vector_) { 
    vector_.push_back(new_item); 
} 

или использовать std::set

Answer 4

Это то, что вам нужно:

void ClassA::func(std::shared_ptr<ClassB> new_item) 
{ 
    for(auto items : vector_) 
    { 
     if(items == new_item) 
     { 
      return; 
     }    
    } 

    vector_.push_back(new_item); 
} 

Обратите внимание, что вам нужно ==, а что =. Кроме того, ваш код добавлял new_item к вектору на каждой итерации. Теперь это просто сделано, если элемент не найден.

Answer 5

Вы используете

if(items = new_item) 
    { 
     return; 
    } 

значение new_item присваивается элементам как "= "является оператором присваивания и любым положительным значением if, если условие будет истинным. Вам необходимо использовать оператор «==» для сравнения