2

我正在制作一个程序，该程序从用逗号分隔的用户中获取名称。该程序允许用户在逗号之间放置任意数量的空格。例如：

如果我要输入类似的东西

Smith, John

或者

Smith,John

我想打印出来

John, Smith

问题是我的程序没有正确处理上面的以下示例；如果输入类似于

Smith , John

或者

Smith ,John.

这是我的代码：

#include <stdio.h>
#include <string.h>
#include <ctype.h>
#define LINESIZE 128

int get_last_first(FILE *fp);

int main (void)
{
    get_last_first(stdin);
}

/*takes names in the format [LASTNAME],[FIRSTNAME]*/
int get_last_first(FILE *fp)
{
    char first[LINESIZE];
    char last[LINESIZE];
    char line[LINESIZE];
    size_t i;

    while(1)
    {
        printf("Enter your last name followed by a comma and your first name\n");

        /*if we cant read a line from stdin*/
        if(!fgets(line, LINESIZE, fp)) 
        {
            clearerr(stdin);
            break;   /*stop the loop*/
        }

        /*goes through the line array and checks for non-alphabetic characters*/        
        for(i = 0; i < strlen(line); i++)
        {
            if(!isalpha(line[i]))
            {
                /*if it sees a space hyphen or comma, it continues the program*/
                if((isspace(line[i]) || line[i] == ',') || line[i] == '-' )
                {
                    continue;
                }
                else
                {
                    return -1;
                }
            }

        }

        if(sscanf(line, "%s , %s", last, first))
        {
            printf("%s, %s", first, last);
            return 1;
        }

        return 0;   

    }
}

是因为我没有正确使用 sscanf 吗？

4

2 回答 2

2

sscanf()不进行模式匹配；逗号是一个完全有效的非空白字符串组件，因此会被%s规范所吞噬。您可以使用类似的东西%[^ \t,]来匹配一系列字符，直到空格或逗号。

于 2011-06-23T03:49:25.453 回答
1

geekosaur的回答中概述了基本问题-是的，您在滥用sscanf().

您的代码中还有其他问题。

  • 不要strlen()在循环测试中使用；它每次都会被调用并产生相同的答案（除非你正在破解字符串）。在循环之前调用它一次并使用保存的值。

  • 从风格上讲，条件中有不必要的括号和空格（并错过了必要的空格）：

    if((isspace(line[i]) || line[i] == ',') || line[i] == '-' )

if (isspace(line[i]) || line[i] == ',' || line[i] == '-')

    关键字后面应该有一个空格（见标准）。没有必要用额外的括号破坏三向条件的对称性；最后的右括号之前不需要空格。（或者，如果你坚持，那么你需要在条件的左括号后有一个平衡的开放空间。但请不要坚持。）

  • 您需要sscanf()成功转换两个值的测试；您只需检查它是否没有转换 0 值。它可能返回 1；它可能会返回 EOF（以及 0 或 2）。

  • 您的“无限循环”被 a 打破break，但随后该函数不返回值。
  • 您的main()程序会忽略函数的返回值 - 所以您根本不需要返回值。
  • 在风格上，虽然 C99 标准允许您省略return末尾的main()，但我更喜欢在那里看到它。（你没有犯有main()回报的罪void；谢谢。）
  • 您可能已经注意到，当我格式化问题时，我删除了一些嵌入在注释中的杂散代码。不要保留死代码——也不要在 SO 上炫耀它。
  • 在风格上，我在#include和之间放置了空格<stdio.h>。再次，看看 C 标准并注意他们是如何编写它的：#include <stdio.h>你也这样做。

标记为“风格”的评论是建议 - 编译器在很大程度上忽略了空格或缺少空格。然而，C 标准显示了所有在行中带有空格的示例，#include并且在关键字之后有空格，for并且ifKernighan 和 Ritchie 对 C 的原始描述也是如此，因此您可以遵循可靠的先例。通常，如果您遵循标准约定，其他人会发现您的代码更易于阅读。牙套的定位更具争议性——但如果你在那里遵循 K&R 就不会出错（尽管我更喜欢Allman风格）。

于 2011-06-23T04:12:38.950 回答