5月 11日の評価

課題はこちら

全体

• 全体的にインデンテーションが滅茶苦茶である。 インデンテーションに関する約束事を読んでおくこと。

May11_1.java

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

      if (…){
          out.println("A");
          out.println("</body>");
          out.println("</html>");
          out.close();
      } else {
          out.println("B");
          out.println("</body>");
          out.println("</html>");
          out.close();
      }
  

  は

      if (…){
          out.println("A");
      } else {
          out.println("B");
      }
      out.println("</body>");
      out.println("</html>");
      out.close();
  

  と書くべきである。

• 出力されるHTMLファイルがおかしくなる解答がある。

      if (i%10==0){
          out.print("<font color='red'>");
      }
      out.print("ABCDE");
      out.println("</font>");
      out.println("</body>");
      out.println("</html>");
      out.close();
  

  では、 i%10==0でないとき、対応する <font …>タグがない </font>タグが出力されてしまう。

May11_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行で済む。

戻る