C_and_CPP 板


LINE

我有一個需求是這樣的, 我要使用 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







like.gif 您可能會有興趣的文章
icon.png[問題/行為] 貓晚上進房間會不會有憋尿問題
icon.pngRe: [閒聊] 選了錯誤的女孩成為魔法少女 XDDDDDDDDDD
icon.png[正妹] 瑞典 一張
icon.png[心得] EMS高領長版毛衣.墨小樓MC1002
icon.png[分享] 丹龍隔熱紙GE55+33+22
icon.png[問題] 清洗洗衣機
icon.png[尋物] 窗台下的空間
icon.png[閒聊] 双極の女神1 木魔爵
icon.png[售車] 新竹 1997 march 1297cc 白色 四門
icon.png[討論] 能從照片感受到攝影者心情嗎
icon.png[狂賀] 賀賀賀賀 賀!島村卯月!總選舉NO.1
icon.png[難過] 羨慕白皮膚的女生
icon.png閱讀文章
icon.png[黑特]
icon.png[問題] SBK S1安裝於安全帽位置
icon.png[分享] 舊woo100絕版開箱!!
icon.pngRe: [無言] 關於小包衛生紙
icon.png[開箱] E5-2683V3 RX480Strix 快睿C1 簡單測試
icon.png[心得] 蒼の海賊龍 地獄 執行者16PT
icon.png[售車] 1999年Virage iO 1.8EXi
icon.png[心得] 挑戰33 LV10 獅子座pt solo
icon.png[閒聊] 手把手教你不被桶之新手主購教學
icon.png[分享] Civic Type R 量產版官方照無預警流出
icon.png[售車] Golf 4 2.0 銀色 自排
icon.png[出售] Graco提籃汽座(有底座)2000元誠可議
icon.png[問題] 請問補牙材質掉了還能再補嗎?(台中半年內
icon.png[問題] 44th 單曲 生寫竟然都給重複的啊啊!
icon.png[心得] 華南紅卡/icash 核卡
icon.png[問題] 拔牙矯正這樣正常嗎
icon.png[贈送] 老莫高業 初業 102年版
icon.png[情報] 三大行動支付 本季掀戰火
icon.png[寶寶] 博客來Amos水蠟筆5/1特價五折
icon.pngRe: [心得] 新鮮人一些面試分享
icon.png[心得] 蒼の海賊龍 地獄 麒麟25PT
icon.pngRe: [閒聊] (君の名は。雷慎入) 君名二創漫畫翻譯
icon.pngRe: [閒聊] OGN中場影片:失蹤人口局 (英文字幕)
icon.png[問題] 台灣大哥大4G訊號差
icon.png[出售] [全國]全新千尋侘草LED燈, 水草

請輸入看板名稱,例如:Tech_Job站內搜尋

TOP