0

我已经做了一些代码，但它没有正确运行

到目前为止我做了什么

void keyread1(void)
{
    char *string1= (char*) malloc (20);//char pointer pointing to allocated memory
    char *string2= (char*) malloc (20);
    char *string3= (char*) malloc (20);
    char *string4= (char*) malloc (20);
    string4[0]='\0';
    printf("Enter word to convert to morse code: \n");
    scanf("%s", string1);
    while (string1!='\0')
    {
        int z=0,a=0;
        strncpy(string2+a, string1+z,4 );
        string3=morse2english(string2);
        strncat(string4+a,string3,1);
        z=z+4;
    }
    printf("morse code string: %s\n",string4);
    free(string1);
    return;
}

char *morse2english(char *morsecode)
{
    int j=0;
    int a=0;
    char *azarray=(char*)malloc(26);
    strcpy(azarray, "abcdefghijklmnopqrstuvwxyz");
    char *morsearray=(char*)malloc(104*sizeof(char));
    strcpy(morsearray, ".-  -...-.-.-.. .   ..-.--. ......  .----.- .-..--  -.  --- .--.--.-.-. ... -   ..- ...-.-- -..--.----..");
        for (int i = 0; i < 104; ++i )
    for(int j=0;j<4;j++)
    if((morsecode[j++] == morsearray[i++]))
    a =((morsearray[i-4])/4);
    char *ch =(char*)malloc(1*sizeof(char));
    ch=azarray+a;
    return ch;
}

函数keyread应该将莫尔斯字符串分成4个字符并将其传递给morse2english应该找到莫尔斯电码的字母表示并将其返回给keyread函数并将其添加到的函数string4

4

1 回答 1

0

我将问题放在评论中，但是太多了！你真的需要回去学习更多——尤其是关于动态内存分配！

在 keyread1 中：

  • 不需要所有带有 malloced 缓冲区的 strcpy 东西。例如：你为 string3 分配了 20 个字节，但在你使用它之前你说 string3=...，所以这是一个即时泄漏！
  • while(string1 != '\0') 没有比较苹果和苹果：字符串 1 是一个指针，\0 是一个字符。您可能的意思是 *string1，但是您永远不会修改 string1，因此循环不会结束。
  • 您永远不会在循环中修改 a ，因此您不会附加到 string4
  • morse2english 的结果是动态分配的，但您永远不会释放它。

莫尔斯英语：

  • 你 malloc 一个 26 的数组，（它不允许空终止符，所以你有一个缓冲区溢出）
  • 你永远不会释放 azarray - 为什么 malloc 首先，你不修改它，所以字符串文字会起作用。
  • morsearray 同上（我没算，但你不改，所以不需要 malloc 和 strcpy）
  • for 循环似乎是乱码：对于所有莫尔斯电码中的每个单独的字符，查看输入（并修改 for 循环使用的索引）
于 2012-11-16T11:55:56.817 回答