１、代碼邏輯不清晰，if嵌套過多。
２、代碼邏輯不清晰，邏輯代碼書寫較為分散
rc = sqlite3_open(DB_NAME, &db); //邏輯分散
if (rc) {
fprintf(stderr, "Can't open database: %s\n", sqlite3_errmsg(db));
sqlite3_close(db);
return cJSON_CreateObject();
}
cJSON * p;
p = cJSON_GetObjectItem(params, "action");
if (p == NULL)
return cJSON_CreateObject(); //存在資源泄漏問題
else
PowerPolicy = p->valueint;//邏輯不清晰導致if過多
string sql = "";
建議寫法：
cJSON * p = cJSON_GetObjectItem(params, "action");
if(p == NULL)
return cJSON_CreateObject();
int powerpolicy = p->valueint;
int rc= sqlite3_open(DB_NAME, &db); //邏輯分散
if (rc) {
fprintf(stderr, "Can't open database");
return cJSON_CreateObject();
}
...
3、調試日志不規范。
建議：
調試時候 打印時間戳，線程號，當前文件和行號，調試信息，可如下設計API
c實現
#include <stdio.h>
#include <stdlib.h>
#include <stdarg.h>
#include <time.h>
#include <pthread.h>
void _debug(const char *filename,int line,const char *fmt,...)
{
char buf[1024];
va_list ap;
va_start(ap,fmt);
vsnprintf(p,size,fmt,ap);
va_end(ap);
time_t t = time(NULL);
printf("%u-%s:%d-%p %s\n",t,filename,line,pthread_self(),buf);
}
.h頭文件
#define debug(fmt,...) _debug(__FILE__,__LINE__,fmt,__VA_ARGS__)
4、API接口設計不合理，沒有遵循誰申請，誰釋放原則。
cJSON* send_request(char const *ip,uint16_t port,char const *method,int id, cJSON* params)
1、不應該在send_request中釋放參數申請的內存。
2、返回參數最好傳入指針。（副作用，不釋放）
5、代碼中沒有處理好定時和異步任務。
１、能不用信號的地方，就不要用信號。
２、框架上有定時器，必要的時候可以增加API
6、調試技巧
1 查看資源占用情況
1. top
2 lsof
3 進入/proc/pid/ 查看線程數量。
有異常處理異常。
gdb 抓到信息發現比較難分析的話，就放棄gdb，利用日志解決。
