C/C++の宿題を片付けます　115代目

C/C++の宿題を片付けます　115代目 at TECH

612:デフォルトの名無しさん
08/09/12 15:17:18
よーし>>610にイチャモンを付けまくるぞー

strdupはC言語標準の関数じゃないし。
いちいちC言語の規格を無視する理由になる程便利でもないし。
使わないほうがよくね？

あとreallocの結果がNULLの時にメモリリークしてるぜ。

printf(buffer[i])は文字列の中に%があると
破綻するからデータに%しかなくても使うのはよくないprintf("%s",...)にしなさい。

それに比較関数のポインタをキャストするのはよくない。
関数ポインタのキャストは動作するか保障されてない。
素直に比較関数の中でvoid*をキャストしろ。

比較関数で減算使ってるけど、
値がでかいとオーバーフローとかアンダーフローする可能性がある。
素直に比較しろ。

次ページ

続きを表示

1を表示