5月 18日の評価

課題はこちら

全体

May18_1.java

• 冗長な if文の書き方をしている人が多い。

     if (sec < 20)  {
       …
     } else if (20 <= sec && sec < 40) {
       …
     } else if (40 <= sec && sec < 60) {
       …
     }
  

  赤色部分は不要である。

May18_2.java

• ものすごくばかばかしい書き方

      if (i==0) {
          out.println("<img src='0.png'>");
      } else if (i==1) {
          out.println("<img src='1.png'>");
      } else if (i==2) {
          out.println("<img src='2.png'>");
      } else if (i==3) {
          out.println("<img src='3.png'>");
      } else if (i==4) {
          out.println("<img src='4.png'>");
      } else if (i==5) {
          out.println("<img src='5.png'>");
      } else if (i==6) {
          out.println("<img src='6.png'>");
      } else if (i==7) {
          out.println("<img src='7.png'>");
      } else if (i==8) {
          out.println("<img src='8.png'>");
      } else if (i==9) {
          out.println("<img src='9.png'>");
      }
  

  をしている人が少なからずいる。

      out.println("<img src='"+i+".png'>");
  

  あるいは

      out.printf("<img src='%d.png'>%n", i);
  

  の1行で済む。

• 謎の書き方:

      for (j = 0;j < 10;j++){
          if (a == j){
               out.printf("<img src='image/%d.gif'>",j);
          }
      }
  

  をしている人がいる。

      out.printf("<img src='image/%d.gif'>", a);
  

  の1行で済む。

戻る