作者yoco315 (眠月)
看板C_and_CPP
标题[问题] 继承 std::string 的注意事项?
时间Tue Mar 17 23:32:20 2009
我有一个需求是这样的,
我要使用 unordered_map < std::string, U >,
但是我针对某个集合的字串查找的频率非常高,
我想每次都计算 std::string 的 hash 很没效率,
所以我想我可以把 hash 值保存起来。
於是想到一个方案,
我设计一个 hashed_string 继承 std::string,
新增一个栏位 size_t hash_value_ 把 hash 值好,
然後我就可以宣告成 unordered_map < hashed_string, U >,
这样每次查找的时候,只要直接提领出里面存放的 hash 值就可以,
要注意的是我在字串内容变动的时候记得重新用 hashing() 更新 hash_value 就好。
class hashed_string : public std::string {
public:
size_t hash_value () ;
private:
size_t hash_value_ ;
} ;
那这部份倒是没问题,
建构子也没问题,这个都算容易写,
但是我对於 operator=() 跟 operator+=() 的实作不是很清楚,
我目前是这样写,我觉得效能有问题,但是我不知道怎麽改进。
hashed_string& operator=(const hashed_string& hstr) {
std::string::operator=(static_cast<std::string>(hstr)) ;
hashing () ; // 计算 hash 值并保留
}
我接收一个 hashed_string,
然後把他转型成 std::string,
呼叫 std::string::operator=() 来指派继承而来的 std::string 的部份,
感觉上这样好累赘,但是我不知道这部份要怎麽改进,
还是说这边本来就该这样写?
另外我的 operator+=() 是这样
hashed_string& operator+=(const hashed_string& hstr) {
std::string::operator+=(static_cast<std::string>(hstr)) ;
hashing () ;
}
请大师开示一下如何改善,多谢 orz
--
To iterate is human, to recurse, divine.
递回只应天上有, 凡人该当用回圈. L. Peter Deutsch
--
※ 发信站: 批踢踢实业坊(ptt.cc)
◆ From: 118.160.110.89
1F:→ yoco315:还是我根本不该继承 std::string orz 03/17 23:42
2F:推 legnaleurc:可是std::string没有virtual destructor ... ? 03/17 23:42
3F:→ yoco315:嗯,对,所以果然不该继承,应该包住… 03/17 23:42
4F:→ legnaleurc:没错,我个人认为继承它是个坏主意 03/17 23:43
5F:→ yoco315:嗯,我现在跟你个人想法一样了 XD 03/17 23:45
6F:推 StubbornLin:要不要考虑lazy evaluation? 新增一个dirty flag 03/18 00:12
7F:→ StubbornLin:有改动字串就把dirty设为true 然後在get hash时 03/18 00:13
8F:→ StubbornLin:看是否改过 需要重新计算hash 算完设dirty为false 03/18 00:13
9F:→ StubbornLin:没改动 就直接丢出旧的hash 03/18 00:14
10F:→ yoco315:重点完全不是这个.. 03/18 00:22
11F:→ StubbornLin:阿囧 歹势 没看清楚 03/18 00:23
12F:推 Ebergies:很多时候 aggregation/composition 比继承好得多 03/18 00:38
13F:→ yoco315:天哪.. 我改程式改到一半,发现这样 iterator 等等东西 03/18 00:46
14F:→ yoco315:就全都要包... 该死 orz 03/18 00:46
15F:→ yoco315:好想死.... 03/18 00:48
16F:推 softwind:扩充 STL ... 原po真强者 03/18 01:35
17F:→ yoco315:刚刚上网爬了一下,发现继承 std::string 是大忌 XDDDD 03/18 02:02
18F:→ yoco315:扩充 STL 没这麽难阿. STL 就是设计的让你很好扩充 ^^ 03/18 02:02
19F:推 littleshan:所有改变 string 内容的操作都要重算 hash 03/18 09:22
20F:→ littleshan:包括用 iterator 也可能改变 string 的内容 03/18 09:22
21F:→ littleshan:所以这部份的确要全部重包吧? 03/18 09:23
22F:→ littleshan:另外关於 operator=,在你的例子中子类别可以向上转型 03/18 09:27
23F:→ littleshan:若 string::operator= 吃ref,就不需要static_cast吧 03/18 09:28
24F:→ littleshan:虽然这和你要做的事已经无关了... 03/18 09:29
25F:推 saxontai:C++ Coding Standard Chapter 35 中,正是以 string 为例 03/18 10:11
26F:→ saxontai:说明「继承自非 base class」的缺点。XD 03/18 10:12
27F:→ yoco315:我看完以後完全忘记了 XD 03/18 20:16
28F:→ yoco315:自己回自己:注意事项「请不要继承 std::string」 XD 03/18 20:41