2008-12-24

http://qune.cside.com/mt/mt-tb.cgi/1064

よりそいプログラミングによりそってみる。

>int

>count_comma(const char* str) {

別に comma に限定する必要はないから、count_char にして、char 引数もう一つもとうよ。

>int

>count_comma(const char* str) {

> int ret;

ret って?count?じゃあ count って変数名のほうがいいよ。あと = 0 ね。

> for (ptr = str; *ptr != '\0'; ptr++) {

>

> }

ヌル文字最後にない場合どうすんだよ、おめー

> static const char COMMA = ',';

だから、グローバルになんてするなよ。

再利用性を高めようとする意識はいい。けど、後で関数単体で再利用できるように書けって。

> うるさい!だまってて!

すいません・・・

記事への反応(ブックマークコメント)

ログイン ユーザー登録
ようこそ ゲスト さん