0

我正在制作一个程序，它将打印出元素周期表[ ]，以及[*]您正在搜索的元素......

 void printmap(int x, int y)
{
    int a, b, table[9][18] = {
        1, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 1, 
        1, 1, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 1, 1, 1, 1, 1, 
        1, 1, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 1, 1, 1, 1, 1, 
        1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 
        1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 
        1, 1, 0, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 
        1, 1, 0, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 
        0, 0, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 0, 
        0, 0, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 0 
    };   // 0 - Empty, 1 - [ ], 2 - [*]

    table[x][y] = 2;
        b = 0;
    printf("\n*******************\n\n");
    for( a = 1, printf("  \t"); a < 19; a++)
    {
        printf("%3d", (b + a));
    }
    printf("\n");
    for(a = 0; a < 9; a++)
    {
        printf("\n%2d\t", a + 1);
        for(b = 0; b < 18; b++)
        {
            switch(table[a][b])
            {
                case 0 :
                printf("   ");

                case 1 : 
                printf("[ ]");
                break;

                case 2 : 
                printf("[*]");
                break;

                default : 
                printf("");
            }
        }
    }
}

出于某种原因，输出搞砸了......（如果我通过氦的参数，应该是0，18这就是我得到的：http: //pastebin.com/YjhjzSi8

我究竟做错了什么？谢谢

4

3 回答 3

2

正如其他受访者所说，您需要正确初始化表数组。

此外，您在 switch/case statemet 中错过了一个“中断”，并且您还错过了在每行末尾打印一个换行符。我冒昧地将您的变量 a,b 重命名为 row,col 并修复了您的代码，这应该可以满足您的要求，

void printmap(int x, int y)
{
    int row=0, col=0;
    int was = table[x][y];
    table[x][y] = 2;

    col = 0;
    printf("\n*******************\n\n");
    printf("  \t");
    row=0; //you were not initializing row
        for( col = 0; col < 18; col++)
        {
            printf("%3d", (col+row+1));
        }
    printf("\n");
    for(row = 0; row < 9; row++)
    {
        printf("\n%2d\t", row + 1);
        char* rc = "   ";
        for(col = 0; col < 18; col++)
        {
            switch(table[row][col])
            {
            case 0 : rc=("   "); break; //was missing break
            case 1 : rc=("[ ]"); break;
            case 2 : rc=("[*]"); break;
            default: rc=("   "); break; //was missing break
            }
            printf("%s",rc);
        }
        printf("\n"); //missing print newline
    }
    table[x][y] = was;
}

这是这张桌子，正如其他人所建议的，

int table[9][18] = {
    { 1, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 1 },
    { 1, 1, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 1, 1, 1, 1, 1 },
    { 1, 1, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 1, 1, 1, 1, 1 },
    { 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1 },
    { 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1 },
    { 1, 1, 0, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1 },
    { 1, 1, 0, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1 },
    { 0, 0, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 0 },
    { 0, 0, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 0 }
};  // 0=Empty, 1=[ ], 2=[*]
于 2013-11-13T03:25:25.940 回答
2

这不是一个二维数组，仅仅因为你让它看起来像一个二维数组，并不意味着它就是一个。

这就是你所拥有的

 table[9][18] = {
    1, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 1, 
    1, 1, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 1, 1, 1, 1, 1,
    ...
    0, 0, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 0
 };

您应该相应地添加更多括号

 table[9][18] = {
    {1, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 1}, 
    {1, 1, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 1, 1, 1, 1, 1),
    ....
    {0, 0, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 0}};
于 2013-11-13T02:54:11.297 回答
2

自从我使用 C++ 以来已经有一段时间了，但是，这不是你初始化 2D 数组的方式。事实上，我很惊讶它甚至可以编译。正确的方法是这样的：

table[0] = {1, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 1};
table[1] = {1, 1, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 1, 1, 1, 1, 1};
table[2] = {1, 1, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 1, 1, 1, 1, 1};
...

另一个例子：二维数组值 C++

于 2013-11-13T02:56:28.827 回答