从fgets中删除换行符

如果这个问题很明显，或者我犯了一个简单的逻辑错误，我很抱歉。 我已经搜索了各种方法来摆脱使用fgets产生的换行符，但是在构建时我仍然遇到问题。 我认为我不能正确理解某些内容并错误地应用我的“解决方案”。 我希望透明，并说这是学校的任务。 一切都运行良好，除了我的输出，它有不必要的新行。

此函数的唯一目的是将名称读入结构数组。

void initialize(FILE * ifp, Candidate * electionCandidates, int count) { for (int i = 0; i < count; i++) { fgets (electionCandidates[i].name , 20 , ifp); if (electionCandidates[i].name[strlen(electionCandidates[i].name) - 1] == '\n') { electionCandidates[i].name[strlen(electionCandidates[i].name) - 1] = '\0'; } } } 

当我尝试运行时显示以下内容：“隐式声明库函数”strlen“类型为unsigned long（constant char *）”

Answer 1

1）不，不一定是“明显的” – 好问题。

2）是的，你想使用“strlen（）”。

3）听起来你忘了#include 来定义“strlen（）”。

 #include  #include  ... char *trim (char *s) { int i = strlen(s)-1; if ((i > 0) && (s[i] == '\n')) s[i] = '\0'; return s; } 

Answer 2

这是我的看法。

 #include  #include  // to fix the warning ... // remove NL left by fgets, with protection void trimnl (char *s) { char *pnl = s + strlen(s); if (*s && *--pnl == '\n') *pnl = 0; } void initialize(FILE* ifp, Candidate* electionCandidates, int count) { for (int i = 0; i < count; i++) { fgets (electionCandidates[i].name, 20, ifp); trimnl(electionCandidates[i].name); } } 

---

在我的原始版本中，代码读取

  char *pnl = s + strlen(s) - 1; 

这是基于带有无符号值的带符号算法而受到批评的。 但是，批评不适用，因为在这种情况下，最终结果（如果strlen = 0）相当于：

  char *pnl = s - 1; 

没有无符号算术问题，但是存在未定义的行为问题。 这就是我改变代码的原因。

Answer 3

将来，使用-Wall编译以启用编译警告。

根据您的编译器，您将获得有关如何解决此问题和类似问题的有用建议（已在本线程的其他地方的一个答案中进行了讨论 – 与Clang进行编译将警告您有关缺少的include ，例如）。