Question

我正在从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

我做错了什么？

Answer 1

您不会在任何地方放置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++) { ...

Answer 2

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

最重要的是，字符串末尾的空终止符。在for循环中读取数字后，添加以下行：
```
alphabetic[i] = '\0';
```
如果用户输入小写字母，则在switch语句中将忽略它们。要解决此问题，请包括<ctype.h>并将switch的数量从alphabetic[i]更改为toupper(alphabetic[i])。在已经大写的字母上调用toupper是良性的。

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

2 个答案: