Kouhei Sutou
null+****@clear*****
Thu Oct 4 10:41:41 JST 2012
Kouhei Sutou 2012-04-09 15:06:56 +0900 (Mon, 09 Apr 2012) New Revision: 613ff01b730e52843b40e44363e5a7f0a392569a https://github.com/mroonga/mroonga/commit/613ff01b730e52843b40e44363e5a7f0a392569a Log: doc coding-style: add about cast Modified files: doc/source/developer/coding_style.rst Modified: doc/source/developer/coding_style.rst (+39 -1) =================================================================== --- doc/source/developer/coding_style.rst 2012-04-09 14:07:30 +0900 (e64473d) +++ doc/source/developer/coding_style.rst 2012-04-09 15:06:56 +0900 (99fd6fe) @@ -598,6 +598,45 @@ C++では真偽値に ``bool`` を使うためこのような状況は発生し for (int i = 0; i < 10; ++i) { } +キャスト +-------- + +C++のスタイルを用いる +^^^^^^^^^^^^^^^^^^^^^ + +Cスタイルのキャストはなんでもキャストできてしまうため、意図しないキャストにも気付かない可能性がある。例えば、単に ``const`` を外したいだけなのに、間違って違う型に変換していても気付けない。C++のキャストでは ``const`` を外したいときは ``const_cast`` を使用し、型を変換するときは ``static_cast`` を指定する。こうすれば、 ``static_cast`` で間違って ``const`` を外してしまっている場合も気付ける。 ``reinterpret_cast`` はどうしても必要なときのみ注意して使う。 + +よい例（ ``const_cast`` を使っている）: + + uchar *to_key; + const ucahr *from_key; + KEY *key_info; + uint key_length; + key_copy(to_key, const_cast<uchar *>from_key, key_info, key_length); + +よい例（ ``static_cast`` を使っている）: + + int n_hits = 1; + int n_documents = 10; + float hit_ratio = (float)(n_hits) / n_documents; + +よい例（ ``static_cast`` では無理なので ``reinterpret_cast`` を使っている）: + + THD *thread = current_thd; + my_hash_delete(&mrn_allocated_thds, reinterpret_cast<uchar *>(thread)); + +悪い例（Cスタイルのキャストを使っている）: + + int n_hits = 1; + int n_documents = 10; + float hit_ratio = (float)(n_hits) / n_documents; + +悪い例（ ``static_cast`` で十分なのに ``reinterpret_cast`` を使っている）: + + void *value = get_value(key); + char *name; + name = reinterpret_cast<char *>(value); + その他 ------ @@ -621,4 +660,3 @@ C++では真偽値に ``bool`` を使うためこのような状況は発生し * typeidを使わない。 * 例外はMySQLで問題がないようであればOK。mroongaから外の世 界（MySQLの世界）にはださないようにする。 - * castはCのキャストではなくC++のキャストを使う。 -------------- next part -------------- HTML����������������������������...Download