Question

void tr_str(char s[], char news[]){
int c;
size_t k =0;
    while ((c = getchar()) != EOF)
    {
        for(k=0; k < strlen(s);k++)
        {                   
            if(c == s[k])
            {
                c = news[k];
            }
        }
        putchar(c);
    }
}

这是我的翻译函数，当前从命令行接收两个参数，并使用这些字母将第一个集切换到第二个集。我刚刚发现一个问题，如果两个参数的长度不同，它就不起作用。必须发生的是，如果第一个参数是3个字符长度，第二个参数是2个字符长度，则第二个参数将添加第三个字符，它应该等于存储在其中的最后一个字符。例如：arg1（a，b，c）; ARG2（X，Y）;使用时，arg2变为arg2（x，y，y）;

感谢您的帮助。

Answer 1

如果您只想让代码工作，而不进行任何优化，则必须在选择新闻中的字符之前将k与strlen（新闻）进行比较。它假设新闻至少有一个字符。

void tr_str(char s[], char news[]){
int c;
size_t k =0;
    while ((c = getchar()) != EOF)
    {
        for(k=0; k < strlen(s);k++)
        {                   
            if(c == s[k])
            {
                if (k >= strlen(news))
                    c = news[strlen(news) - 1)];
                else
                    c = news[k];
                break; // Need a break here, else c could be found again in s and translated again.
            }
        }
        putchar(c);
    }
}

重复新闻中的最后一个字符以防止在循环中重复测试是一个好主意，但你不应该直接修改news []，除非你确定在每种情况下新闻都是可写的大足够的记忆力，即便如此，IMO仍然是不好的做法。制作新闻副本并使用它更好。

void tr_str(char s[], char news[]){
int c;
size_t k = 0;
size_t sSize = strlen(s);
size_t newsSize = strlen(news);
char *newsCopy;
int preventsMemoryLeak = 0;

    if (newsSize < sSize)
    {
        // Duplicates news in a large enough memory. No need a zero at end of this copy.
        newsCopy = malloc(sSize);
        memcpy(newsCopy, news, newsSize);
        // Fills the remaining with a copy of the last char
        memset(newsCopy + newsSize, news[newsSize - 1], sSize - newsSize);

        preventsMemoryLeak = 1;
    }
    else
    {
        newsCopy = news;
    }

    while ((c = getchar()) != EOF)
    {
        for(k = 0; k < sSize; k++)
        {                   
            if(c == s[k])
            {
                c = newsCopy[k];
                break;
            }
        }
        putchar(c);
    }

    if (preventsMemoryLeak == 1) free(newsCopy);
}

制作自定义tr.c函数

1 个答案: