- 2008-10-01 (水) 23:25
- C言語
下のようなのはバグに近いと思うんです。
void hoge (char *text) {
if (text == NULL) return;
free(text);
text = NULL;
}
いや、別にちゃんと動くし、間違ってないし、そういう意味ではバグではないんですが。最後の「text = NULL」というのが意味がない。こう書いてしまう人は結構いるんじゃないかと思うんです。(自分も含めて)
上記では冒頭でNULLチェックをしているし、最後にNULLを代入するので連続で呼ばれても2重解放はしないぜ!って感があふれています。でも連続で呼ばれたら2重解放になります。なぜなら呼び出し元のtextはNULLにできないから。
void hoge (char *text) {
if (text == NULL) return;
free(text);
text = NULL;
}
int main() {
char *text = NULL;
text = (char *)malloc(10);
hoge(text);
if (text != NULL) {
printf("いやん\n");
}
return 0;
}
「いやん」となるわけです。hogeに渡されるのはtextのコピーなので、hogeの中で値を変えても無意味なわけだけども、textの指している先のデータをhogeの中で書き換えることはできるので勘違いしやすいんじゃないかなと。
あるいは下記のようなことはできません。
void fuga(char *text) {
text = (char *)malloc(10);
}
いや、やってもいいけど期待したとおりには動きません。上のようなことをやりたい場合は
void fugaShinuchi(char **text) {
(*text) = (char *)malloc(10);
}
int main() {
char *text = NULL;
fugaShinuchi(&text);
free(text);
return 0;
}
初心忘るべからずということで。
以上
関連記事
- Newer: 世界撲滅始めました
- Older: google map ムズカシス
Comments:5
- 匿名 08-10-09 (木) 0:00
-
塩です。
お久しぶり。hogeに渡されるのは*textのコピーじゃないかな?
- コスミー 08-10-09 (木) 21:48
-
あ、お久しぶりです。
あーこれ、失敗しましたね。hogeとmainの変数が両方textだったのでややこしいですね。
mainから見たときのtextのコピーという意味だったんです。いや、それも違うのかな?text自体はポインタ変数なのでポインタ変数の値がコピーされたポインタ変数が渡されるという、そういう理解は共有できていると思うんですけど・・・。なんて書くのが正しいのかわかりません!P.S. mixiの方のぞいてみます。
- legiaquangcao.com 17-07-13 (木) 17:54
-
Having read this I thought it was rather informative.
I appreciate you taking the time and effort to put this
article together. I once again find myself personally
spending a significant amount of time both reading and commenting.But so what, it was still worthwhile!
- Rebekah 19-03-09 (土) 12:18
-
Great post! We will be linking to this great article on our website.
Keep up the great writing.
- Dong 19-06-07 (金) 7:58
-
Fantastic post however I was wanting to know if you could write a litte more on this subject?
I’d be very thankful if you could elaborate a little bit more.
Thank you!
Trackbacks:0
- Trackback URL for this entry
- http://red-treasure.com/report/wp-trackback.php?p=279
- Listed below are links to weblogs that reference
- Cでポインタを引数に渡すときに勘違いするなよ from コスミー報告書[社外秘]