在读fcitx源代码时想到的一个问题

bluespirit

以下代码在tools.c  void LoadProfile (void)函数中．

1. 
   
2. 
   
3. strcpy (strPath, (char *) getenv ("HOME"));
   
4.     strcat (strPath, "/.fcitx/profile");
   
5. 
   
6.     fp = fopen (strPath, "rt");
   
7. 
   
8.     if (fp) {
   
9.         for (;;) {
   
10.             if (!fgets (str, PATH_MAX, fp))
    
11.                 break;
    
12. 
    
13.             i = strlen (str) - 1;
    
14.             while (str[i] == ' ' || str[i] == '\n')
    
15.                 str[i--] = '\0';
    
16. 
    
17.             pstr = str;
    
18. 
    
19.             if (MyStrcmp (str, "版本=")) {
    
20.                 pstr += 5;
    
21. 
    
22.                 if (!strcasecmp (FCITX_VERSION, pstr))
    
23.                     bRetVal = True;
    
24.             }
    
25.             else if (MyStrcmp (str, "主窗口位置X=")) {
    
26.                 pstr += 12;
    
27. 
    

复制代码

一开始想到，getenv ()有时会返回ＮＵＬＬ的吧，代码中没有检查．但想一想Home变量是一定存在的，好像不检查也可以．　这儿到底需要不需要检查一下getenv的返回值？


fp = fopen (strPath, "rt");这个地方，我觉得，如果打不开文件的话，是不是有个提示更好？如果用
if(fp),当fp==NULL时，不是都没提示？

sharsma

就我的看法，第一个地方，稳健的代码应该检查返回值，以应对特殊的情况，比如某种大小写敏感的操作系统的用户主目录是“home”，当然fcitx跨平台的要求没那么强烈；
而第二个地方，确实错误输出到标准错误管道或者log会更好。

aerofox

还有一种没有HOME变量的情况：某个“贪玩”的用户把HOME变量unset掉了。