Created on 2010-02-10.00:00:00 last changed 89 months ago
Proposed resolution:
In Function objects [function.objects] in paragraph 2 at the end of the Header <functional> synopsis insert:
// convenience functions template <class T> size_t make_hash (const T&); template <class T, class... Types> size_t make_hash (const T&, const Types&...);
In Class template hash [unord.hash] add:
20.7.16.1 Hash creation functions [hash.creation]
template <class T> size_t make_hash (const T& val);Returns: hash<T>()(val);
template <class T, class... Types> size_t make_hash (const T& val, const Types&... args);Returns: hash<T>()(val) + std::make_hash(args...)
Rationale:
There is no consensus to make this change at this time.
[ LEWG Kona 2017 ]
Recommend NAD: Feature? Needs a paper. (This is LEWG21)
[ 2010 Pittsburgh: Moved to NAD Editorial, rationale added below. ]
Currently, the library lacks a convenient way to provide a hash function that can be used with the provided unordered containers to allow the usage of non trivial element types.
While we can easily declare an
std::unordered_set<int>
or
std::unordered_set<std::string>
we have no easy way to declare an unordered_set for a user defined type. IMO, this is a big obstacle to use unordered containers in practice. Note that in Java, the wide usage of HashMap is based on the fact that there is always a default hash function provided.
Of course, a default hash function implies the risk to provide poor hash functions. But often even poor hash functions are good enough.
While I really would like to see a default hash function, I don't propose it here because this would probably introduce a discussion that's too big for this state of C++0x.
However, I strongly suggest at least to provide a convenience variadic template function make_hash<>() to allow an easy definition of a (possibly poor) hash function.
As a consequence for a user-defined type such as
class Customer {
   friend class CustomerHash;
   private:
     string firstname;
     string lastname;
     long   no;
   ...
 };
would allow to specify:
class CustomerHash : public std::unary_function<Customer, std::size_t>
{
  public:
    std::size_t operator() (const Customer& c) const  {
       return make_hash(c.firstname,c.lastname,c.no);
    }
};
instead of:
class CustomerHash : public std::unary_function<Customer, std::size_t>
{
  public:
    std::size_t operator() (const Customer& c) const  {
       return std::hash<std::string>()(c.firstname) +
              std::hash<std::string>()(c.lastname) +
              std::hash<long>()(c.no);
    }
};
Note that, in principle, we can either specify that
make_hash returns the sum of a call of std::hash<T>()(x) for each argument x of type T
or we can specify that
make_hash provides a hash value for each argument, for which a std::hash() function is provided
with the possible note that the hash value may be poor or only a good hash value if the ranges of all passed arguments is equally distributed.
For my convenience, I propose wording that describes the concrete implementation.
| History | |||
|---|---|---|---|
| Date | User | Action | Args | 
| 2018-06-22 06:38:21 | admin | set | messages: + msg9967 | 
| 2018-06-22 06:38:21 | admin | set | status: lewg -> nad | 
| 2014-11-24 15:11:58 | admin | set | status: nad future -> lewg | 
| 2010-10-21 18:28:33 | admin | set | messages: + msg1579 | 
| 2010-10-21 18:28:33 | admin | set | messages: + msg1578 | 
| 2010-10-21 18:28:33 | admin | set | messages: + msg1577 | 
| 2010-02-10 00:00:00 | admin | create | |