1月 18日の評価

QUIZ

• 全問完答  12人

• 問1: for文で3回繰り返すことを見落としている人があまりにも多い…orz
• 問3: for文のあとに変数の値がどうなっているかを理解できていない人が多い。for文の意味を復習すること。

jan18_1.c

• 書込みモードでオープンしたファイルから読み込んでいる。

    …
    fp = fopen(file_name, "w");
    fscanf(fp, "%d", &n) /* 間違い */
  

  → 正しく動作することを保証できない。
• 2回読込みモードでオープンしている。 → 非効率すぎる。
• 初回と2回目以降で同じような処理を2回書いている

    /* 良くない */
    if ((fp=fopen(file_name, "r")) {
        AAAA
        CCCC
        DDDD
        EEEE
        FFFF
    } else {
        BBBB
        CCCC
        DDDD
        EEEE
        FFFF
    }
  

  → まとめるべきである。。

    if ((fp=fopen(file_name, "r")) {
        AAAA
    } else {
        BBBB
    }      
    CCCC
    DDDD
    EEEE
    FFFF 
  

• 読込みと書込みを別の関数で行っていて、データを渡すことができない。
  → 関数の引数と戻り値を使えばよい。

    int get_data(void) {
        int n;
        
        …
        fscanf(fp, "%d", &n);
        …    
        return n;
    }
  
    void put_data(int n) {
          int m;
  
          …
          scanf("%d", &m);
          n -= m;
          …
    }
  
    int main(void) {
        int n;
  
        n = get_data();
        put_data(n);
    }
  

  また、必要がないのにポインタを使っている人もいたが、 これは感心できない。

jan18_2.c

• 未提出者 49名
• RAND_MAXマクロを使わずに自分で

  #define N 32767
  

  のように定義している人がいたが、間違いの元になるので良くない。