07/07/12 23:14:10
C++で、構造体にメンバ変数を追加する必要があって、
その構造体を扱っているコードをみていたら、わざわざ
メンバ変数を一個ずつコピーしてる関数が。
HogeCopy(Hoge* dst, Hoge* src)
{
dst->a = src->a;
dst->b = src->b;
dst->c = src->c;
:
:
}
これ、*dst = *src; でいいだろと思ったけど、念のためにコードを確認してみると、
メンバ変数は31個なのに、コピーしてるのは29個だけ。
ただのバグだと思うけど、*dst = *src;に置き換えて全部コピーするようにして、なにか
影響があるといやなので、(影響がないか調べるのもめんどうなので)上のコードに、
追加したメンバ変数を、コピーするコードを追加しておきました。
(こうやってどんどんダメなコードになっていくんだろうなぁ)