将电话号码从字母转换为数字形式

时间:2019-03-07 14:20:19

标签: c arrays character

我正在从KNKings的“ C编程:一种现代方法”一书中进行练习,该方法涉及将用户输入的字母形式的电话号码转换为数字形式。当程序遇到非字母字符(例如,数字或标点符号)时,应使它们保持不变。我可以假设用户只输入大写字母。

但是,至少可以说,我的程序似乎会产生垃圾。

#include <stdio.h>

#define MAX_SIZE 50

int main(void)
{
    char alphabetic[MAX_SIZE], ch;
    int num_elements = 0;

    printf("Enter phone number: ");
    int i;
    for (i = 0; i < MAX_SIZE && ((ch = getchar()) != '\n'); i++){
        alphabetic[i] = ch;
        num_elements++;
    }

    for (i = 0; i <= num_elements; i++){
        switch (alphabetic[i]){
            case 'A': case 'B': case 'C': alphabetic[i] = '2'; break;
            case 'D': case 'E': case 'F': alphabetic[i] = '3'; break;
            case 'G': case 'H': case 'I': alphabetic[i] = '4'; break;
            case 'J': case 'K': case 'L': alphabetic[i] = '5'; break;
            case 'M': case 'N': case 'O': alphabetic[i] = '6'; break;
            case 'P': case 'R': case 'S': alphabetic[i] = '7'; break;
            case 'T': case 'U': case 'V': alphabetic[i] = '8'; break;
            case 'W': case 'X': case 'Y': alphabetic[i] = '9'; break;
            default:                                           break;
        }
    }
    printf("%s\n", alphabetic);
    return 0;
}

我特别输入:COLLECT-800。 它输出如下内容:u░@■║k

我做错了什么?

2 个答案:

答案 0 :(得分:4)

您不会在任何地方放置null终止符,因此在读取字符串时,无论之后是否对其进行了修改,它都是未定义的行为。把这一行:

alphabetic[num_elements] = 0;

for (i = 0; i < MAX_SIZE &&...循环之后。


就我个人而言,我不会进行getchar循环,而是读入这样的字符串:

scanf("%49s", alphabetic); // reads in a string up to 50 characters
for (i = 0; alphabetic[i]; i++) { ...

答案 1 :(得分:4)

您的想法正确,但是程序中缺少两件事:

  1. 最重要的是,字符串末尾的空终止符。在for循环中读取数字后,添加以下行:

    alphabetic[i] = '\0';
    
  2. 如果用户输入小写字母,则在switch语句中将忽略它们。要解决此问题,请包括<ctype.h>并将switch的数量从alphabetic[i]更改为toupper(alphabetic[i])。在已经大写的字母上调用toupper是良性的。