10月 26日の評価

pairsum.c

未だにインデントが不適切な解答が少なからずある。

この問題でif文を使うのは良くない。

    for (i=0; i<=n; i++) {
        for (j=0; j<=n; j++) {
	    if (i+j<=n) {  
                printf("i=%d, j=%d \n", i, j);
            }		
        }
    }

では無駄に繰り返しをするので、

    for (i=0; i<=n; i++) {
        for (j=0; i+j<=n; j++) {
	    printf("i=%d, j=%d \n", i, j);
        }
    }

と書くべきである。

n=5にしか対応していない解答がたくさんあった…。

lattice.c

未提出者が 35人もいる。

四重ループにしている解答が何故か多かったが、基本的な部分は二重ループでできるはず。

    for (i=0; i<n; i++) {
        for (j=0; j<n; j++) {
            printf("%3d %3d %3d %3d \n", i*UNIT, j*UNIT, (i+1)*UNIT, j*UNIT);
            printf("%3d %3d %3d %3d \n", i*UNIT, j*UNIT, i*UNIT, (j+1)*UNIT);
        }
    }

あとは端の部分をどうにか工夫する。例えば、

    for (i=0; i<n; i++) {
        printf("%3d %3d %3d %3d \n", i*UNIT, n*UNIT, (i+1)*UNIT, n*UNIT);
    }
    
    for (j=0; j<n; j++) {
        printf("%3d %3d %3d %3d \n", n*UNIT, j*UNIT, n*UNIT, (j+1)*UNIT);
    }

縦方向と横方向の格子の数が同じであることを利用すれば、次のようなやり方も（技巧的だが）OK。

    for (i=0; i<n; i++) {
        for (j=0; j<=n; j++) {
            printf("%3d %3d %3d %3d \n", i*UNIT, j*UNIT, (i+1)*UNIT, j*UNIT);
            printf("%3d %3d %3d %3d \n", j*UNIT, i*UNIT, j*UNIT, (i+1)*UNIT);
        }
    }

10という magic numberはプログラム中に埋め込んではいけない。必ず定数マクロを使うこと。

min3.c

インデントはプログラミングⅡでの約束に従えば、次のようにしなければいけない。

int min(int x, int y, int z) {
    int min = x;

    if (y < min) {
        min = y; 
    }
    if (z < min) {
        min = z;
    }
    return min;
}

latice3D.c

出していた人（5人）は全員 OK。

latice3Dproj.c

提出したのは1人だけだった。

mid3.c

比較は3回だけで良いはず。提出されていた解答のほとんどが複雑なことをやり過ぎている。模範解答は示さないので、再挑戦してほしい。
ときどき、テストしたのか？と疑問に思う解答がある。

戻る

Koji Kagawa(kagawa@eng.～)