constexpr хэш-функция

Question

Вот constexprхеш-функция, которая упаковывает строку в самый большой доступный целочисленный тип без знака. Так что ты думаешь?

#include <climits>

#include <cstdint>

#include <utility>

#include <iostream>

namespace detail
{

template <typename T, std::size_t ...I>
constexpr T hash(char const* const s, std::size_t const N,
  std::index_sequence<I...>) noexcept
{
  return ((T(s[I < N ? I : 0]) << ((I < N ? I : 0) * CHAR_BIT)) | ...);
}

}

template <typename T = std::uintmax_t>
constexpr T hash(char const* const s, std::size_t const N) noexcept
{
  return detail::hash<T>(s, N, std::make_index_sequence<sizeof(T)>());
}

template <typename T = std::uintmax_t, std::size_t N>
constexpr T hash(char const(&s)[N]) noexcept
{
  return hash<T>(s, N - 1);
}

int main()
{
  std::cout << (hash("a") == 'a') << std::endl;

  return 0;
}

https://wandbox.org/permlink/KbPiWJc434xYLL3q

Answer 1

Ваш код слишком сложен. С C ++ 17 вы можете писать более сложные функции constexpr, поэтому вам не нужны трюки с вариативными шаблонами:

template <typename T = std::uintmax_t, std::size_t N>
constexpr T hash(char const(&s)[N]) noexcept
{
  T val{};

  for (size_t i = 0; i < N; ++i)
    val |= s[i] << (i * CHAR_BIT);

  return val;
}

Кроме того, это ужасная хеш-функция! Выход сильно коррелирован с входом. Он также будет хешировать только до sizeof(T)символов, поэтому длинные строки с общим префиксом могут иметь одно и то же значение хеш-функции.