c - これはCの多次元配列ナビゲーションを処理する正しい方法ですか

Question

私の懸念は、td の終わりを計算する方法が正しいことです。これは最適ですか？この種のことを行うためのより良い C イディオムはありますか?

#include <stdio.h>

int main() {

    char td[][4] = { {0,1,2,3}, {4,5,6,7}, {8,9,10,11}, 
                      {12,13,14,15}, {16,17,18,19}, {20,21,22,23}  };
    char* p = *td;  

    int rows = sizeof(td) / sizeof(td[0]);
    int cols = sizeof(td[0]) / sizeof(td[0][0]);

    char* end = p + (rows * cols);

    /* Print first element in each 'row' */
    while(p != end) {
        printf("first element: %u\n", p[0]);
        p += 4;
    }
    return 0;
}

Answer 1

通常、C では for ループを使用します。

int main() {
    char td[][4] = { {0,1,2,3}, {4,5,6,7}, {8,9,10,11}, 
                      {12,13,14,15}, {16,17,18,19}, {20,21,22,23}  };

    int rows = sizeof(td) / sizeof(td[0]);
    int cols = sizeof(td[0]) / sizeof(td[0][0]);

    /* Print first element in each 'row' */
    for (int row = 0; row < rows; row++)
        printf("first element: %u\n", td[row][0]);

    return 0;
}

ただし、作成したコードは機能します。

Answer 2

#define COLS 4
int main() {
    char td[][COLS] = { {0,1,2,3}, {4,5,6,7}, {8,9,10,11}, 
                      {12,13,14,15}, {16,17,18,19}, {20,21,22,23}  };

    int rows = sizeof(td) / sizeof(td[0]);

    /* print the matrix */
    for (int row = 0; row < rows; row++) {
        for (int col = 0; col < COLS; col++) {
            printf("first element: %u\n", td[row][col]);
        }
    }
    return 0;
}

各行の最初の列:

    for (int row = 0; row < rows; row++) {
        printf("first element: %u\n", td[row][0]);
    }