Не работает стандартная функция std::find??

Question

[Евгений Иванович]

Есть код

std::vector<std::string> LoadedModule_Hashes;

__forceinline bool LoadGmodLibraryFromRawData(lua_State* L, const std::string& data)
{
	auto hash = MD5(data).hexdigest();
	if (std::find(
			LoadedModule_Hashes.begin(),
			LoadedModule_Hashes.end(),
			hash) != LoadedModule_Hashes.end())
		return false;
	LoadedModule_Hashes.push_back(hash);
	return true;
}

И он постоянно возвращает true, чекал хэши должно работать, в чем причина?

Comments

[Армянское Радио]

data точно всегда одинаковая?
Так - то лучше бы использовать под это дело set, честно говоря.

[Евгений Иванович]

Армянское Радио, Одинаковая, делал принты хешей

[Армянское Радио]

Евгений Иванович, a точно hash - это строка?

[Roman]

Евгений Иванович, и что после двух подряд LoadGmodLibraryFromRawData с одинаковыми входными данными, в LoadedModule_Hashes лежит 2 одинаковых значения? т.е
Начальное состояние LoadedModule_Hashes == empty
Конечное size == 2 и

true == equal(LoadedModule_Hashes[0].cbegin(), 
              LoadedModule_Hashes[0].cend(),
              LoadedModule_Hashes[1].cbegin());

[Евгений Иванович]

Армянское Радио, Да, точно

[Евгений Иванович]

Roman, Значения разные, при входном параметре если значение уже существует в массиве должен происходить выход из функции, раньше работало

[Армянское Радио]

Евгений Иванович, скорее всего, косяк где-то в другом месте - что-то портит массив

[Roman]

std::find это очень простой алгоритм, он не может работать неправильно.

vector<string> LoadedModule_Hashes;
/*
//release
auto find_hash = [](vector<string> const& hashes, string const& value){
  return find(hashes.begin(), hashes.end(), value) != hashes.end();
};
*/
// debug
auto find_hash = [](vector<string> const& hashes, string const& value){
  auto it = find_if(hashes.begin(), hashes.end(), [&](string const& hs)
  { 
    cout << "compare:\n" << hs << "\n" << value << "\n";
    return hs == value;
  });
  return it != hashes.end();
};

__forceinline bool LoadGmodLibraryFromRawData(lua_State* L, std::string const& data)
{
  if(const string hash = MD5(data).hexdigest(); !find_hash(LoadedModule_Hashes, hash)) // Если не найден
  {
    LoadedModule_Hashes.push_back(hash);
    return true;
  }
  // Хэш уже есть
  return false;
}