間違ってもキャストでごまかすことなかれ。
これはワーニングではないですが、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処理 */
    ....
}

ローカルな定数宣言や変数宣言が書いてあるヘッダファイルを時々見かける。
ヘッダファイルを字義通りプログラムの最初の方に書くものを置くファイルと考えてるのだろうか。
余計な情報が漏れると、それに依存した利用をされる可能性があり、
マーフィーの法則にしたがって、利用されてしまう。
一度利用されてしまえば、ローカルであったはずの定数はグローバルになってしまい、
ローカルな都合では変更ができなくなってしまうか、変更したときに予期せぬバグをもたらす。
表に出さなくてよいものは徹底的に隠そう。