このコードがセグメンテーションフォールトを与えているのはなぜですか？

Question

私は、1文字のテキスト文字列の引数を128文字以下の任意の数だけ使用するプログラムを作成しようとしています。プログラムはstdinからstdoutにテキストをコピーしますが、入力に表示されている単語のいずれかが "CENSORED"という単語に置き換えられます。

例：

Said Hamlet to Ophelia, 
I'll draw a sketch of thee, 
What kind of pencil shall I use? 
2B or not 2B? 

プログラムは、この実行する必要があります：

./censor Ophelia < poem.txt 
Said Hamlet to CENSORED, 
I'll draw a sketch of thee, 
What kind of pencil shall I use? 
2B or not 2B? 

をここに私のコードです：

#include <stdio.h> 
#include <string.h> 
#include <stdlib.h> 
int main(int argc, char *argv[]) 
{ 
    char lines[200][200]; 
    int numLines=0,i,j; 
    int nbytes = 128; 
    int bytes_read=0; 
    char *my_string; 
    char * pch; 
    //reading from stdin 
    while(stdin) 
    { 
    my_string=(char *) malloc (nbytes + 1); 
    bytes_read = getline (&my_string, &nbytes, stdin); 
    strcpy(lines[numLines++],my_string); 
    } 

    //scanning and replacing specified words by "CENSORED" 
    for(i=0;i<argc;i++) 
    { 
     for(j=0;j<numLines;j++) 
     { 
      pch = strstr (lines[j],argv[i]); 
      strncpy (pch,"CENSORED",8); 
     } 
    } 
    //display the result in output screen 
    for(j=0;j<numLines;j++) 
     { 
      printf("\n%s",lines[i]); 
     } 

} 

問題がある 私はpoem.txtと呼ばれるこのファイルを持っていますこれがセグメンテーション違反を与えているが、私は間違いを特定できません。

Answer 1

ヒットしたヒットが長くても短くても適切に上書きされているわけではありません。端末に上書きされる可能性があります（セグメンテーションフォールトにつながる可能性があります。\0）。また、各行に対してそれぞれのコマンドラインワードを1回だけチェックするので、ダブルヒットを逃したように見えます。最後に、すべての行を格納することでこれをより複雑にしました。行は他の行に影響を与えません。そうそう

#include <stdio.h> 
#include <string.h> 
#include <stdlib.h> 

#define REPLACEMENT "CENSORED" 
#define BUFFER_SIZE (1024) 

int main(int argc, char *argv[]) 
{ 
    ssize_t bytes_read; 
    char *s, *line = malloc(BUFFER_SIZE); 
    size_t nbytes = BUFFER_SIZE, replacement_length = strlen(REPLACEMENT); 

    // read from stdin 
    while ((bytes_read = getline(&line, &nbytes, stdin)) != -1) 
    { 
     // scanning and replacing specified words 
     for (int i = 1; i < argc; i++) 
     { 
      while ((s = strstr(line, argv[i])) != NULL) 
      { 
       size_t search_length = strlen(argv[i]); 
       size_t tail_length = strlen(s + search_length); 

       (void) memmove(s + replacement_length, s + search_length, tail_length + 1); 
       (void) memcpy(s, REPLACEMENT, replacement_length); 
      } 
     } 

     // display the result in output screen 
     (void) fputs(line, stdout); 
    } 

    free(line); 
} 

、あなたはあなたがをmallocで確保するもの無料に忘れてしまった：

はここで、より詳細な置換コードと全体的な単純化されたアプローチです。そして、あなたは

例...あなたの目標の一つとして、プログラムの名前を探している

> ./a.out pencil 2B < poem.txt 
Said Hamlet to Ophelia, 
I'll draw a sketch of thee, 
What kind of CENSORED shall I use? 
CENSORED or not CENSORED?