PROSAGA码农传奇-C/C++-二进制文件打印键搜索c

<div class =“post-text”itemprop =“text”>
  
    该代码可疑：
  
  <BLOCKQUOTE>
     <pre>
 <code>
 for ( ctr = 0; ctr < NUM_ELEMENTS; ctr ++)
{
 fread (&elements[ctr], sizeof(struct elementTag), 1, fbin);
}

</code>
 </pre>
  </BLOCKQUOTE>
  
    在代码上方，您不检查是否
    
      FOPEN
    
     是否返回NULL，这里不检查结果
    
      FREAD
    
    ，所以你认为至少有
     <code>
 NUM_ELEMENTS
 </code>
     文件中的记录。
  
  
    但主要是你假设文件的内容对应于一系列的方式
    
      ElementTag的
    
     在内存中，因此该文件专用于给定编译器，在给定的体系结构上具有相关的编译选项，您确定吗？
  
  
    如果您的文件包含其外部表示，则您的阅读方式为false
  
   <pre>
 <code>
 1 Hydrogen H
2 Helium He
...

</code>
 </pre>
  <HR />
  <BLOCKQUOTE>
     <pre>
 <code>
 char ele[31];
...
scanf("%s", ele);

</code>
 </pre>
  </BLOCKQUOTE>
  
    您没有针对太长输入的保护，并且您没有检查EOF案例
  
  
    你可以做
  
   <pre>
 <code>
 if (scanf("%30s", ele) == 1) {
 for (...

</code>
 </pre>
  <HR />
  <BLOCKQUOTE>
    
      程序能够接受元素名称的任务（不应区分大小写）
    
  </BLOCKQUOTE>
  
    你的代码使用
  
  <BLOCKQUOTE>
     <pre>
 <code>
 if (strcmp(ele, elements[ctr].Name))

</code>
 </pre>
  </BLOCKQUOTE>
  
    
      STRCMP
    
     区分大小写，你使用错误的结果，使用
    
      strcasecmp
    
  
  <HR />
  
    在
  
  <BLOCKQUOTE>
     <pre>
 <code>
 for ( ctr = 0; ctr < NUM_ELEMENTS; ctr ++)
{
 if (strcmp(ele, elements[ctr].Name))
 {
 printf("Atomic Number: %d, Symbol: %s\n\n", elements[ctr].AtomicNumber, elements[ctr].Symbol);
 }
 else
 {
 printf("NOT FOUND!\n\n");
 break;
 }
}

</code>
 </pre>
  </BLOCKQUOTE>
  
    因为错误的使用
    
      STRCMP
    
     除了输入元素是第一个元素之外，你总是会说你找到了第一个元素。正确使用结果
    
      STRCMP
    
     你会经常说
     <code>
 NOT FOUND
 </code>
     除非输入是第一个元素，否则停止查看其他可能性。
  
  
    错误情况必须超出循环或知道循环结束，例如：
  
   <pre>
 <code>
 for ( ctr = 0; ; ctr ++)
 {
 if (ctr == NUM_ELEMENTS)
 {
 printf("NOT FOUND!\n\n");
 break;
 }

if (!strcasecmp(ele, elements[ctr].Name))
     {
         printf("Atomic Number: %d, Symbol: %s\n\n", elements[ctr].AtomicNumber, elements[ctr].Symbol);
          break;
     }
 }

</code>
 </pre>
</DIV>