fgets（）入力から末尾の改行文字を削除する

ユーザーからデータを取得し、それをgccの別の関数に送信しようとしています。コードはこのようなものです。

printf("Enter your Name: ");
if (!(fgets(Name, sizeof Name, stdin) != NULL)) {
    fprintf(stderr, "Error reading Name.\n");
    exit(1);
}

ただし、\n最後に改行文字があることがわかりました。ですから、入力するJohnと送信されてしまいますJohn\n。それを削除し\nて適切な文字列を送信するにはどうすればよいですか。

少し醜い方法：

char *pos;
if ((pos=strchr(Name, '\n')) != NULL)
    *pos = '\0';
else
    /* input too long for buffer, flag error */

少し奇妙な方法：

strtok(Name, "\n");

strtokユーザーが空の文字列を入力した場合（つまり、Enterキーのみを押した場合）、関数は期待どおりに機能しないことに注意してください。それは\nキャラクターをそのまま残します。

もちろん他にもあります。

おそらく最も簡単な解決策は、私のお気に入りのあまり知られていない関数の1つを使用しますstrcspn()。

buffer[strcspn(buffer, "\n")] = 0;

それも処理したい場合'\r'（たとえば、ストリームがバイナリの場合）：

buffer[strcspn(buffer, "\r\n")] = 0; // works for LF, CR, CRLF, LFCR, ...

この関数は、a '\r'またはa に到達するまで'\n'（つまり、最初の'\r'またはが見つかるまで）、文字数をカウントし'\n'ます。何もヒットしない場合は、'\0'（文字列の長さを返す）で停止します。

これはので、何の改行がなくても正常に動作することを注意strcspnで停止'\0'。その場合、行全体が単にで置き換え'\0'られ'\0'ます。

size_t ln = strlen(name) - 1;
if (*name && name[ln] == '\n') 
    name[ln] = '\0';

以下は、'\n'によって保存された文字列から可能性を取り除くための迅速なアプローチfgets()です。2つのテストでを
使用strlen()します。

char buffer[100];
if (fgets(buffer, sizeof buffer, stdin) != NULL) {

  size_t len = strlen(buffer);
  if (len > 0 && buffer[len-1] == '\n') {
    buffer[--len] = '\0';
  }

bufferそしてlen、必要に応じて、を使用してください。

このメソッドには、len後続のコードの値という副次的な利点があります。それよりも簡単に速くなる可能性がありますstrchr(Name, '\n')。 YMMVを参照しますが、どちらの方法でも機能します。

buffer、いくつかの状況下ではオリジナルfgets()が含まれません"\n"：
A）行が長すぎるbufferため、のchar前のみ'\n'がに保存されbufferます。未読の文字はストリームに残ります。
B）ファイルの最後の行がで終わっていません'\n'。

入力の'\0'どこかにnull文字が埋め込まれている場合、によって報告さstrlen()れる長さには'\n'場所は含まれません。

他のいくつかの回答の問題：

1. strtok(buffer, "\n");削除に失敗した'\n'ときbufferです"\n"。この回答から-この制限を警告するためにこの回答の後に修正されました。

2. 最初のとき以下は、まれに失敗したcharことにより、読み取りがfgets()あります'\0'。これは、入力がembeddedで始まる場合に発生します'\0'。その後、確かにの正当な範囲外でメモリにアクセスするようにbuffer[len -1]なります。ハッカーがUTF16テキストファイルを愚かに読んでみたり見つけたりするかもしれない何か。これは、この答えが書かれたときの答えの状態でした。その後、非OPがこの回答のチェックのようなコードを含むように編集しました。buffer[SIZE_MAX]buffer""

   size_t len = strlen(buffer);
   if (buffer[len - 1] == '\n') {  // FAILS when len == 0
     buffer[len -1] = '\0';
   }

3. sprintf(buffer,"%s",buffer);未定義の動作です：Ref。さらに、先頭、区切り、または末尾の空白は保存されません。今すぐ削除しました。

4. 【後の良い答えで編集】アプローチにbuffer[strcspn(buffer, "\n")] = 0;比べて性能以外は1ライナーに問題ありませんstrlen()。コードがI / Oを実行している場合、トリミングでのパフォーマンスは通常問題ではありません-CPU時間のブラックホール。次のコードで文字列の長さが必要な場合、またはパフォーマンスを重視する場合は、このstrlen()アプローチを使用してください。それ以外の場合strcspn()は、優れた代替手段です。

すべての行に「\ n」がある場合、fgets出力から「\ n」を削除するように指示

line[strlen(line) - 1] = '\0';

さもないと：

void remove_newline_ch(char *line)
{
    int new_line = strlen(line) -1;
    if (line[new_line] == '\n')
        line[new_line] = '\0';
}

単一の「\ n」トリミングの場合、

void remove_new_line(char* string)
{
    size_t length = strlen(string);
    if((length > 0) && (string[length-1] == '\n'))
    {
        string[length-1] ='\0';
    }
}

複数の「\ n」トリミングの場合、

void remove_multi_new_line(char* string)
{
  size_t length = strlen(string);
  while((length>0) && (string[length-1] == '\n'))
  {
      --length;
      string[length] ='\0';
  }
}

私の初心者の方法;-)それが正しいかどうか私に知らせてください。それは私のすべてのケースで機能しているようです：

#define IPT_SIZE 5

int findNULL(char* arr)
{
    for (int i = 0; i < strlen(arr); i++)
    {
        if (*(arr+i) == '\n')
        {
            return i;
        }
    }
    return 0;
}

int main()
{
    char *input = malloc(IPT_SIZE + 1 * sizeof(char)), buff;
    int counter = 0;

    //prompt user for the input:
    printf("input string no longer than %i characters: ", IPT_SIZE);
    do
    {
        fgets(input, 1000, stdin);
        *(input + findNULL(input)) = '\0';
        if (strlen(input) > IPT_SIZE)
        {
            printf("error! the given string is too large. try again...\n");
            counter++;
        }
        //if the counter exceeds 3, exit the program (custom function):
        errorMsgExit(counter, 3); 
    }
    while (strlen(input) > IPT_SIZE);

//rest of the program follows

free(input)
return 0;
}

おそらく最も明白な方法で改行文字を削除する手順：

1. header NAMEを使用してstrlen()、内部の文字列の長さを判別しstring.hます。strlen()は終了をカウントしないことに注意してください\0。

size_t sl = strlen(NAME);

2. 文字列が1 \0文字（空の文字列）で始まるか、または1つだけ含まれるかどうかを確認します。この場合slになります0ので、strlen()私は数えるdoesn't上に述べたように\0、それの最初の発生時に停止します。

if(sl == 0)
{
   // Skip the newline replacement process.
}

3. 適切な文字列の最後の文字が改行文字かどうかを確認します'\n'。この場合は、に置き換え\nてください\0。インデックスカウントはで始まる0ため、次のようにする必要があることに注意してくださいNAME[sl - 1]。

if(NAME[sl - 1] == '\n')
{
   NAME[sl - 1] = '\0';
}

fgets()文字列要求でEnterキーのみを押した場合（文字列の内容は改行文字のみで構成されていました）、NAMEその後の文字列は空の文字列になります。

4. 私達はちょうど1のステップ2と3を一緒に組み合わせることができif、論理演算子を使って-statement &&：

if(sl > 0 && NAME[sl - 1] == '\n')
{
   NAME[sl - 1] = '\0';
}

5. 完成したコード：

size_t sl = strlen(NAME);
if(sl > 0 && NAME[sl - 1] == '\n')
{
   NAME[sl - 1] = '\0';
}

fgets毎回再入力せずに一般的に出力文字列を処理することにより、この手法を使用する関数が好きな場合は、fgets_newline_kill次のとおりです。

void fgets_newline_kill(char a[])
{
    size_t sl = strlen(a);

    if(sl > 0 && a[sl - 1] == '\n')
    {
       a[sl - 1] = '\0';
    }
}

提供された例では、次のようになります。

printf("Enter your Name: ");

if (fgets(Name, sizeof Name, stdin) == NULL) {
    fprintf(stderr, "Error reading Name.\n");
    exit(1);
}
else {
    fgets_newline_kill(NAME);
}

入力文字列にが埋め込まれている場合、このメソッドは機能しないことに注意してください\0。その場合strlen()、最初の文字数までの文字数のみが返され\0ます。しかし、これは一般的なアプローチではありません。ほとんどの文字列読み取り関数は通常、最初に停止するためです。\0そのnull文字まで文字列を取得するためです。

それ自体の質問は別として。コードが不明確になる二重否定を回避するようにしてください：if (!(fgets(Name, sizeof Name, stdin) != NULL) {}。簡単にできますif (fgets(Name, sizeof Name, stdin) == NULL) {}。

TimČ1つのライナーは、fgetsの呼び出しによって取得された文字列に最適です。これは、最後に改行が1つ含まれていることがわかっているためです。

別のコンテキストで、複数の改行を含む可能性のある文字列を処理する場合は、strrspnを探している可能性があります。これはPOSIXではありません。つまり、すべてのUnicesで見つかるとは限りません。自分のニーズに合わせて書きました。

/* 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;
}

使用getlineがオプションの場合-そのセキュリティ問題を無視せず、ポインタを中括弧にしたい場合- getline文字数を返すため、文字列関数を回避できます。以下のようなもの

#include<stdio.h>
#include<stdlib.h>
int main(){
char *fname,*lname;
size_t size=32,nchar; // Max size of strings and number of characters read
fname=malloc(size*sizeof *fname);
lname=malloc(size*sizeof *lname);
if(NULL == fname || NULL == lname){
 printf("Error in memory allocation.");
 exit(1);
}
printf("Enter first name ");
nchar=getline(&fname,&size,stdin);
if(nchar == -1){ // getline return -1 on failure to read a line.
 printf("Line couldn't be read.."); 
 // This if block could be repeated for next getline too
 exit(1);
}
printf("Number of characters read :%zu\n",nchar);
fname[nchar-1]='\0';
printf("Enter last name ");
nchar=getline(&lname,&size,stdin);
printf("Number of characters read :%zu\n",nchar);
lname[nchar-1]='\0';
printf("Name entered %s %s\n",fname,lname);
return 0;
}

^{注：[ セキュリティの問題 ]はgetline無視するべきではありません。}

以下の関数は、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';

二重引用符ではなく単一引用符を使用するためです。詳細については、シングルクォートとダブルクォートのリンクをご覧ください。

これを試してください：

        int remove_cr_lf(char *str)
        {
          int len =0;


          len = strlen(str);

          for(int i=0;i<5;i++)
          {
            if (len>0)
            if (str[len-1] == '\n')
            {
              str[len-1] = 0;
              len--;
            }

            if (len>0)
            if (str[len-1] == '\r')
            {
              str[len-1] = 0;
              len--;
            }
          }

          return 0;
        }