乱码字符串输出

Question

我目前正在做一项任务，要求我审查argv的单词和输入重定向。

我的问题在我的输出中很明显，可以在下面找到。我有很多跟踪打印语句，可能会或可能不会帮助您指出我的问题。

我的本意是：

1. 获取file.txt
2. 的file.txt内容复制到buffer[x][y]，其中[x]是单词的串并[y]是[x]一个字符。
3. 将argv[]参数与buffer[][]进行比较。
4. 创建newstr[size2]。对于buffer[][]中发现的每个argv[]争议，请将其替换为replace[9] = "CENSORED"。
5. 打印字符串newstr[0 to (size2-1)]。

这里是我的代码：

// censored.c 
#include <stdio.h> 
#include <string.h> 
#include <stdlib.h> 
#include <ctype.h> 
#define SIZE 128 

int main (int argc, char* argv[]) 
{ 
    int i = 0; 
    int j = 0; 
    int k = 0; 
    char buffer[SIZE][SIZE]; 
    char x; 
    int count = 0; 
    int size1 = sizeof(buffer); 
    int size2 = 0; 
    char replace[9] = "CENSORED"; 
    int buffersize=0; 
    printf("tracing: size1: %d.\n", size1); 
    printf("tracing: argc: %d.\n", argc); 
    while((fscanf(stdin, "%c", &x)!=EOF)) 
    { 
     if(isalpha(x)) 
     { 
      buffer[i][j]=x; 
      printf("tracing: buffer[%d][%d]: %c\n", i,j, buffer[i][j]); 
      j++; 
     } 
     else if(isspace(x)) // if whitespace 
     { 
      j = 0; 
      i++; 
      buffer[i][j]=x;//this should be buffer[i][j] 
      printf("tracing: buffer[%d][%d]: %c\n", i,j, buffer[i][j]); 
      j = 0; 
      i++; 
     } 
     else if(ispunct(x)) // if x is a punctuation 
     { 
      j = 0; 
      i++; 
      buffer[i][j]=x;//this should be buffer[i][j] 
      printf("tracing: buffer[%d][%d]: %c\n", i,j, buffer[i][j]); 
     } 
     else if(iscntrl(x)) // if control key (\n, \r etc...) 
     { 
      j = 0; 
      i++; 
      buffer[i][j]=x;//this should be buffer[i][j] 
      printf("tracing: buffer[%d][%d]: %c", i,j, buffer[i][j]); 
      j = 0; 
      i++; 
     } 
     else if(isdigit(x)) 
     { 
      buffer[i][j]=x;//this should be buffer[i][j] 
      printf("tracing: buffer[%d][%d]: %c\n", i,j, buffer[i][j]); 
      j++; 
     } 
     else 
     { 
      break; 
     } 
    } 
    size2 = i; 
    printf("tracing: buffer[8][0]:%s\n",buffer[8]); 
    char newstr[size2][SIZE]; 
    i = 0; 
    j = 0; 
// tracing: 
    printf("tracing: line 72\n"); 
    printf("tracing: size2: %d.\n", size2); 
    while(i < size2) //print buffer[] 
    { 
     printf("%s", buffer[i]); 
     i++; 
    } 

    printf("tracing: line 80\n"); 
    for(k=1; k < argc; k++) 
    { 
     printf("%s\n", argv[k]); 
    } 
// end tracing 
    i = 0; //reinitialize i 
    j = 0; //reinitialize j 
// creating newstr[SIZE] and censoring words 
    printf("tracing: line 89\n"); 
    for(i = 0; i < size2; i++) 
    { 
     for(j=1; j < argc; j++) 
     { 
      if(strcmp(buffer[i], argv[j])==0) 
      { 
       strcpy(newstr[i], &replace[0]); 
       printf("tracing: replaced at [%d]\n", i); 
       break; 
      } 
      else 
      { 
       strcpy(newstr[i], buffer[i]); 
       printf("tracing: copied at [%d]\n", i); 
      } 
     } 
    } 
    i = 0; //reinitialize i 
    while(i < size2) 
    { 
     printf("%s", newstr[i]); 
     i++; 
    } 
    return 0; 
} 

假设我已经输入重定向文件名为file.txt及其内容：

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

我输入的是：

./censored Ophelia thee 2B < file.txt 

这是我得到的奇怪的输出：

Said Hamlet to CENSORED, Ill draw a sketch ???)ofoQ? ?"h?CENSORED,2oQ? 
What? /oQ?kind? of 6oQ?pencil +oQ?shall ?"h?I-oQ? ???)use? ???2BoQ? 
7oQoroQ Qnot 1oQ?CENSORED?4oQ? 

任何帮助表示赞赏，我知道我的代码是凌乱的，这是我的第一个学期的学习C.

Answer 1

我有一个好消息和坏消息，坏的一个是：我发现了一些错误和失误。 好的一个是：所有这些对初学者来说都很常见！

第一个：buffer[]中的字符串没有以'\0'结尾，这是字符串指示符的结尾。这是你问题的原因之一。 这里是其他的：

• char buffer [SIZE] [SIZE]; 在这里，您正在考虑单词和单词的长度不能超过128个，只要这个条件错误就会导致分段错误。我建议你学习动态分配（malloc（））。
• for（j = 1; j < argc; j ++）如果argc == 1，则不会进入此循环，这将导致newstr为空。通常我们用一个NULL指针结束数组，然后在读取数组时检查数组中的每个非空的字符串，以便我们不尝试使用未定义的内容。
• 正如您所指出的那样，您的代码非常混乱，下次尝试在代码中分隔代码，而不是在main()中编码。
• 尝试减少这一点，更少的行你写，将更方便您的代码是阅读和调试：

j = 0; i++; buffer[i][j]=x;//this should be buffer[i][j] printf("tracing: buffer[%d][%d]: %c\n", i,j, buffer[i][j]); j = 0; i++;

• 这个建议不是那种计划旨在真正相关的阅读大量数据但记住：尽可能避免存储数据。

我觉得现在已经足够了，祝你好运，并为我可怜的英语道歉。