1、if语句对出错处理：
if ( ch >= ‘0‘ && ch <= ‘9‘ )
{
/* 正常处理代码 */
}
else
{
/* 输出错误信息 */
printf("error ......\n");
return ( FALSE );
}
这种结构很不好，特别是如果“正常处理代码”很长时，对于这种情况，最好不要用else。先判断错误，如：
if ( ch < ‘0‘ || ch > ‘9‘ )
{
/* 输出错误信息 */
printf("error ......\n");
return ( FALSE );
}
/* 正常处理代码 */
......
这样的结构，不是很清楚吗？突出了错误的条件，让别人在使用你的函数的时候，第一眼就能看到不合法的条件，于是就会更下意识的避免。
2、避免不必要的函数调用
考虑下面的2个函数：
void str_print( char *str )
{
int i;
for ( i = 0; i < strlen ( str ); i++ )
{
printf("%c",str[ i ] );
}
}
void str_print1 ( char *str )
{
int len;
len = strlen ( str );
for ( i = 0; i < len; i++ )
{
printf("%c",str[ i ] );
}
}
注：第1个多次调用函数，明显第2个比较好
3、
原文：http://www.cnblogs.com/vicinghua-study/p/6485985.html