間違ってもキャストでごまかすことなかれ。
これはワーニングではないですが、const int の配列の配列へのポインタを宣言できなくて、intの配列へのポインタにキャストされてるのをみて目が点になったことがあります。
ほかによくあるのが関数の引数にキャストをつかっているもの、プロトタイプ宣言の意味が。。

シンタックスシュガーとはその言語でよく使う構文だから言語設計者が親切に設定してくれているものなので、これを利用しない手はない。Ｃ言語の関数名、配列名の式での扱いを理解してないせいか、

int foo(int *p, int (*f)(int))
{
    return (*f)(*(p+100));
}

とかわざわざ読みくい書き方をしているコードを見ることがある。素直に

int foo(int *p, int (*f)(int))
{
    return f(p[100]));
}

と書けばいいのにね。他にも(*p).hoge とか、->演算子の立場がない...

良いコメントはプログラムの理解を助けるが、悪いコメントは理解を妨げる。
悪いコメントの典型例はコードを翻訳したコメントだ。情報量としてまったく増えていないし、
バグフィックスやメンテナンスでコードとずれが起きたときには嘘が書いてあるコメントになってしまう。
変更履歴をコメントに残すのは構成管理ツールを使っていないことを宣伝するようなものだ。
処理の塊に名前をつけたいときにはコメントではなく関数にしよう。

foo()
{
　　/* goo処理 */
    ....
　　/* foo処理 */
    ....
}