错误：'else'没有以前的'if'和fprintf [关闭]

Question

我想打印速度的屏​​幕索引。我不是代码的作者。

代码在这里http://pastebin.com/47CbB1vb

而get line也是编译所必需的

#include <stdio.h>
#include <stdlib.h>
#include <string.h>
/*
*   get_line:
*       Reads a line into string, returns length.
*       Kernighan & Ritchie p.69
*
*/
int
get_line(fp, line, lim)
   FILE           *fp;
   char           *line;
   int             lim;
{
   int             c, i;

   i = 0;
   while (--lim > 0 && (c = getc(fp)) != EOF && c != '\n')
      line[i++] = c;
   if (c == '\n')
   {
      line[i++] = c;
   }
   line[i] = '\0';
   if (c == EOF)
      return (-1);
   else
      return (i);
}
/*
 *  ignore_line:
 *      Gets the next line (up to and including the newline
 *      character) from the file pointed to by fptr and
 *      promptly loses it.  Taken from asc2ah.c.
 *
 *          Siggi   29.06.1990
 */
int
ignore_line(dat_fp)
   FILE           *dat_fp;
{
   char            string[256];
   char           *fgets();
   if (fgets(string, 250, dat_fp) == NULL)  /* nothing there     */
      return (-1);
   return (0);          /* there was something   */

当我尝试用 gcc 编译时

gcc gi_line.c  vel2d.c -lm -o vel2d
vel2d.c: In function ‘main’:
vel2d.c:205:19: warning: format ‘%f’ expects argument of type ‘double’, but argument 3 has type ‘int’ [-Wformat=]
                   fprintf(stdout,"%.1f",index);
                   ^
vel2d.c:206:9: error: ‘else’ without a previous ‘if’
         else
         ^

我只包含了 fprintf 行。没有那行我可以编译代码并且它可以完美运行。那么我应该改变什么？

Answer 1

这是有问题的代码：

       if (xflg)
              index = m * nxy + l*nx + k;
              fprintf(stdout,"%.1f",index);
       else
          index = m * nxy + k*ny + l;

if 语句的“if”和“else”部分最初各有一行，因此添加大括号并不是绝对必要的。但是，当您将呼叫添加到 fprintf 时，情况发生了变化。

由于if 后面没有大括号，因此index = m * nxy + l*nx + k; 行包含整个if 部分。后面的printf 不是if 的 部分，它是一个单独的语句。那么当编译器看到else时，就没有对应的if了，因为前面的if是完整的。

这可以通过添加大括号来解决：

       if (xflg) {
              index = m * nxy + l*nx + k;
              fprintf(stdout,"%.1f",index);
       } else {
          index = m * nxy + k*ny + l;
       }

您应该始终在任何if、else、for、while 或do 块中添加大括号，即使正文只有一行。这样可以防止发生此类错误。

在这种情况下，您很幸运，未能添加大括号会导致编译器错误。如果您在此示例中没有 else 部分，则代码将编译并且 printf 将始终运行。然后你会摸不着头脑想知道为什么。

至于printf 警告，index 的类型为int，因此您需要使用正确的格式说明符来打印它，即%d。