代码一直在给我创建的字符串末尾

时间:2012-11-22 02:35:07

标签: c ansi

此代码基本上取用户放置的内容(a-z)并将其转换为莫尔斯代码。但是返回字符串总是有' m'最后试图用数以百万计的东西来解决它你能看出我做错了什么,谢谢。

//Functions
#include "stdafx.h"
#include <ctype.h>

#include <stdio.h>
#include <string.h>
#include <malloc.h> 
void morse(void);   //prototype
int _tmain(int argc, _TCHAR* argv[])
{
    for(;;){
    morse(); // function call
    }
}

void morse(void){

    char *ss= (char*)malloc(110); // allocating dynamic memory
    strcpy(ss, ".-  -...-.-.-.. .   ..-.--. ......  .----.- .-..--  -.  --- .--.--.-.-. ... -   ..- ...-.-- -..--.----..");
    char *pp =(char*)malloc(110);
    int i = 0;
    int n = 0;
    printf("Enter text to convert to morse code: ");
    scanf("%s", pp);
    char *q =(char*)malloc(110);
    while (*pp != '\0'){//intiate while loop
        *pp = *(pp + n); 
        int f = (*pp - 97)*4; //find letters postion in morse code string
        int a = 0;

        while (a != 4) //copy morse code for the letter into new string
        {
            *(q + i) = *(ss + f);
            i++;
            a++;
            f++;
        }
        n++;

    }
    q[i] = '\0';    
    printf("%s", q); //return morse code
    printf("\n");
    free(ss); //free the allocated memory
    free(pp);
    free(q);
    return;
} 

2 个答案:

答案 0 :(得分:2)

你的循环绕过一个额外的角色。外部循环查找* pp为\ 0,但* pp仍然是前一个字符的值。

将其更改为:

while (*(pp+n) != '\0') {  // test next character
    char c = *(pp + n);    // fetch next character
    int f = (c - 97)*4;    // find character's index into morse array

我发现这种方式是在调试器下运行并观察发生了什么。即使我只输入了一个字符,它很快就会变得很明显。使用调试器,它是一个很棒的工具。即使您认为它正在运行,也值得通过调试器逐步执行代码。有时你会感到惊讶!

答案 1 :(得分:2)

调试并检查“n”的值。

你可以这样做:

   while (*pp != '\0'){//intiate while loop
        //*pp = *(pp + n); 
        int f = (*pp - 97)*4; //find letters postion in morse code string
        int a = 0;

        while (a != 4) //copy morse code for the letter into new string
        {
            *(q + i) = *(ss + f);
            i++;
            a++;
            f++;
        }
        //n++;
        pp++;

    }