从 fgets() 输入中移除尾部换行符

对@Jerry Coffin和@Tim Čas的回答进行补充：

strchr版本的设计比strcspn（以及strlen版本可能是最快的）要快得多。

strcspn的内部必须迭代通过"\n"字符串，如果合理实现，它只需要执行一次并将字符串长度存储在某个地方。然后在搜索时，它还必须使用一个嵌套的for循环来遍历"\n"字符串。

忽略像库质量实现这些函数所考虑的字长之类的东西，朴素的实现可能看起来像这样：

char* my_strchr (const char *s, int c)
{
  while(*s != '\0')
  {
    if(*s == c)
      return (char*)s;
    s++;
  }
  return NULL;
}

size_t my_strcspn (const char *s1, const char *s2)
{
  size_t s2_length = strlen(s2);
  size_t i;
  for(i=0; s1[i] != '\0'; i++)
  {
    for(size_t j=0; j<s2_length; j++)
    {
      if(s1[i] == s2[j])
      {
        return i;
      }
    }
  }
  return i;
}

• 对于strchr函数，每个字符有两个分支。一个搜索空终止符，另一个比较当前字符与所搜索的字符。

• 对于strcspn函数，它必须像我的例子一样预先计算s2的大小，或者在查找null以及搜索键时迭代它。后者本质上就是strchr函数所做的事情，因此内部循环可以用strchr替换。无论我们如何实现，都会有很多额外的分支。

  细心的语言专家还可能注意到strcspn标准库定义中缺少restrict关键字。这意味着编译器不允许假设s1和s2是不同的字符串。这也阻止了一些优化。

strlen版本将比两者都要快，因为strlen只需要检查null终止，并且不需要其他操作。虽然正如@chux-Reinstate Monica的答案中提到的，有一些情况它不适用，因此它比其他版本稍微脆弱一些。

问题的根源在于fgets函数的糟糕API - 如果它在早期实现得更好，它将返回与实际读取的字符数相对应的大小，这将是很好的。或者像strchr一样返回指向最后一个读取字符的指针。而标准库浪费了返回值，只返回传递字符串中第一个字符的指针，这有点有用。

对于通过调用fgets获得的字符串，Tim Čas的一行代码非常出色，因为您知道它们在末尾包含一个单独的换行符。

如果您处于不同的上下文环境中，并且想要处理可能包含多个换行符的字符串，则可能正在寻找strrspn。它不是POSIX，这意味着您在所有Unix系统上都找不到它。我为自己的需要编写了一个。

/* Returns the length of the segment leading to the last 
   characters of s in accept. */
size_t strrspn (const char *s, const char *accept)
{
  const char *ch;
  size_t len = strlen(s);

more: 
  if (len > 0) {
    for (ch = accept ; *ch != 0 ; ch++) {
      if (s[len - 1] == *ch) {
        len--;
        goto more;
      }
    }
  }
  return len;
}

对于那些在C语言中寻找Perl chomp等效函数的人，我认为这就是它（chomp只会删除尾部的换行符）。

line[strrspn(string, "\r\n")] = 0;

strrcspn函数：

/* Returns the length of the segment leading to the last 
   character of reject in s. */
size_t strrcspn (const char *s, const char *reject)
{
  const char *ch;
  size_t len = strlen(s);
  size_t origlen = len;

  while (len > 0) {
    for (ch = reject ; *ch != 0 ; ch++) {
      if (s[len - 1] == *ch) {
        return len;
      }
    }
    len--;
  }
  return origlen;
}

下面的函数是我在Github上维护的字符串处理库的一部分。它可以从字符串中删除不需要的字符，正好符合您的要求。

int zstring_search_chr(const char *token,char s){
    if (!token || s=='\0')
        return 0;

    for (;*token; token++)
        if (*token == s)
            return 1;

    return 0;
}

char *zstring_remove_chr(char *str,const char *bad) {
    char *src = str , *dst = str;
    while(*src)
        if(zstring_search_chr(bad,*src))
            src++;
        else
            *dst++ = *src++;  /* assign first, then incement */

    *dst='\0';
        return str;
}

一个例子用法可以是：

Example Usage
      char s[]="this is a trial string to test the function.";
      char const *d=" .";
      printf("%s\n",zstring_remove_chr(s,d));

  Example Output
      thisisatrialstringtotestthefunction

你可能想要检查其他可用的函数，甚至为项目做出贡献 :) https://github.com/fnoyanisi/zString

 for(int i = 0; i < strlen(Name); i++ )
{
    if(Name[i] == '\n') Name[i] = '\0';
}

你应该试一试。这段代码基本上是循环遍历字符串，直到找到 '\n' 为止。当找到 '\n' 时，它将被空字符终止符 '\0' 替换。

请注意，在此行中，您正在比较字符而不是字符串，因此无需使用 strcmp()：

if(Name[i] == '\n') Name[i] = '\0';

由于您将使用单引号而不是双引号。如果您想了解更多关于单引号和双引号的区别，这里有一个链接。

这是我的解决方案。非常简单。

// Delete new line
// char preDelete[256]  include "\n" as newline after fgets

char deletedWords[256];
int iLeng = strlen(preDelete);
int iFinal = 0;
for (int i = 0; i < iLeng; i++) {
    if (preDelete[i] == '\n') {

    }
    else {
        deletedWords[iFinal]  = preDelete[i];
        iFinal++;
    }
    if (i == iLeng -1 ) {
        deletedWords[iFinal] = '\0';
    }
}