編集履歴

回答編集履歴

またコメント修正忘れ

2020/03/16 14:06

投稿

スコア13355

answer CHANGED Viewed

@@ -29,7 +29,7 @@
 /**
  * chars の各文字を循環出力する.
- * next() で次の文字に移動し, 最後の文字の後は先頭の文字に戻る.
+ * countUp() で次の文字に移動し, 最後の文字の後は先頭の文字に戻る.
  * この時, 登録されていれば CarryOverListener#carryOver() を呼び出す.
  */
 class CharCounter implements CarryOverListener {

出力方法を変更

2020/03/16 14:06

投稿

jimbe

スコア13355

answer CHANGED Viewed

@@ -13,14 +13,20 @@
       charsRow.check(moto);
       StringCounter stringCounter = new StringCounter(moto, charsRow);
-      stringCounter.outputAllPatternTo(System.out); //表示
-      stringCounter.outputAllPatternTo("ShinIrekae.txt"); //ファイル
+      try (PrintStream out = new PrintStream("ShinIrekae.txt");) {
+        do {
+          System.out.println(stringCounter);
+          out.println(stringCounter);
+        } while(stringCounter.countUp());
+      }
     }
   }
 }
 interface CarryOverListener {
   void carryOver();
 }
 /**
  * chars の各文字を循環出力する.
  * next() で次の文字に移動し, 最後の文字の後は先頭の文字に戻る.
@@ -34,7 +40,7 @@
     this.chars = chars;
     this.listener = listener;
   }
-  void next() {
+  void countUp() {
     if(++index >= chars.length) {
       index = 0;
       if(listener != null) listener.carryOver();
@@ -42,8 +48,9 @@
   }
   char get() { return chars[index]; }
   @Override
-  public void carryOver() { next(); }
+  public void carryOver() { countUp(); }
 }
 class StringCounter implements CarryOverListener {
   private List<CharCounter> ccList;
   private boolean carry;
@@ -56,21 +63,25 @@
       prev = cc;
     }
   }
+  /**
+   * 最下位桁をカウントアップ
-  void outputAllPatternTo(String filename) throws IOException {
+   * @return 処理後も有効値なら true (最上位桁でキャリーオーバーが発生したら false)
+   */
+  boolean countUp() {
-    try (PrintStream out = new PrintStream(filename);) {
+    if(!carry) ccList.get(ccList.size()-1).countUp();
-      outputAllPatternTo(out);
+    return !carry;
-    }
   }
-  void outputAllPatternTo(PrintStream out) {
+  boolean isCarryOver() { return carry; }
-    carry = false;
-    for (CharCounter last=ccList.get(ccList.size()-1); !carry; last.next()) {
-      for(CharCounter cc : ccList) out.print(cc.get());
-      out.println();
-    }
-  }
   @Override
   public void carryOver() { carry = true; }
+  @Override
+  public String toString() {
+    StringBuilder sb = new StringBuilder();
+    for(CharCounter cc : ccList) sb.append(cc.get());
+    return sb.toString();
+  }
 }
 class CharsRow {
   private static String[] ROWS = {
     "あぁかがさざただなはばぱまやゃらわ",

バグ修正

2020/03/16 13:56

投稿

jimbe

スコア13355

answer CHANGED Viewed

@@ -62,6 +62,7 @@
     }
   }
   void outputAllPatternTo(PrintStream out) {
+    carry = false;
     for (CharCounter last=ccList.get(ccList.size()-1); !carry; last.next()) {
       for(CharCounter cc : ccList) out.print(cc.get());
       out.println();

コード内コメント直し忘れ

2020/03/16 13:37

投稿

jimbe

スコア13355

answer CHANGED Viewed

@@ -22,7 +22,7 @@
   void carryOver();
 }
 /**
- * source の各文字を循環出力する.
+ * chars の各文字を循環出力する.
  * next() で次の文字に移動し, 最後の文字の後は先頭の文字に戻る.
  * この時, 登録されていれば CarryOverListener#carryOver() を呼び出す.
  */

コード改良

2020/03/16 13:33

投稿

jimbe

スコア13355

answer CHANGED Viewed

@@ -12,7 +12,7 @@
       CharsRow charsRow = new CharsRow();
       charsRow.check(moto);
-      StringCounter stringCounter = new StringCounter(moto);
+      StringCounter stringCounter = new StringCounter(moto, charsRow);
       stringCounter.outputAllPatternTo(System.out); //表示
       stringCounter.outputAllPatternTo("ShinIrekae.txt"); //ファイル
     }
@@ -47,9 +47,8 @@
 class StringCounter implements CarryOverListener {
   private List<CharCounter> ccList;
   private boolean carry;
-  StringCounter(String moto) {
+  StringCounter(String moto, CharsRow charsRow) {
     ccList = new ArrayList<>();
-    CharsRow charsRow = new CharsRow();
     CarryOverListener prev = this;
     for(char c : moto.toCharArray()){
       CharCounter cc = new CharCounter(charsRow.get(c), prev);

クラス分け

2020/03/16 13:06

投稿

jimbe

スコア13355

answer CHANGED Viewed

@@ -9,69 +9,18 @@
       System.out.println("どのような文の子音を変えますか？");
       String moto = buf.readLine();
+      CharsRow charsRow = new CharsRow();
-      hikaku(moto);
+      charsRow.check(moto);
-      mk(moto);
-    }
-  }
-  private static void hikaku(String moto){
+      StringCounter stringCounter = new StringCounter(moto);
-    for(char c : moto.toCharArray()){
-      String r = getCharsRow(c);
+      stringCounter.outputAllPatternTo(System.out); //表示
-      if(r != null) {
-        System.out.println(c + "は、" + r.charAt(0) + (r.length()>=2?"段にあります":"です"));
+      stringCounter.outputAllPatternTo("ShinIrekae.txt"); //ファイル
-      }
     }
   }
-  private static void mk(String moto) throws IOException {
-    List<CharCounter> ccList = new ArrayList<>();
-    CarryFlag carryFlag = new CarryFlag();
-    CarryOverListener prev = carryFlag;
-    for(char c : moto.toCharArray()){
-      String r = getCharsRow(c);
-      CharCounter cc = new CharCounter(r!=null?r:""+c, prev);
-      ccList.add(cc);
-      prev = cc;
-    }
-    try (BufferedWriter bw = new BufferedWriter(new FileWriter("ShinIrekae.txt"));) {
-      StringBuilder sb = new StringBuilder();
-      for (CharCounter last = ccList.get(ccList.size()-1); !carryFlag.isOn(); last.next()) {
-        sb.delete(0, sb.length());
-        for(CharCounter cc : ccList) sb.append(cc.get());
-        System.out.println(sb.toString());
-        bw.write(sb.toString());
-        bw.newLine();
-      }
-    }
-  }
-  private static String getCharsRow(char c) {
-    for(String r : ROWS) {
-      if(r.indexOf(c) >= 0) return r;
-    }
-    return null;
-  }
-  private static String A = "あぁかがさざただなはばぱまやゃらわ";
-  private static String I = "いぃきぎしじちぢにひびぴみりゐ";
-  private static String U = "うぅくぐすずつづぬふぶぷむゆゅる";
-  private static String E = "えぇけげせぜてでねへべぺめれゑ";
-  private static String O = "おぉこごそぞとどのほぼぽもよょろ";
-  private static String N = "ん";
-  private static String[] ROWS = {A,I,U,E,O,N};
 }
 interface CarryOverListener {
   void carryOver();
 }
-class CarryFlag implements CarryOverListener {
-  private boolean flag = false;
-  @Override
-  public void carryOver() {
-    flag = true;
-  }
-  boolean isOn() { return flag; }
-}
 /**
  * source の各文字を循環出力する.
  * next() で次の文字に移動し, 最後の文字の後は先頭の文字に戻る.
@@ -81,10 +30,9 @@
   private CarryOverListener listener;
   private char[] chars;
   private int index;
-  CharCounter(String source, CarryOverListener l) {
+  CharCounter(char[] chars, CarryOverListener listener) {
-    this.chars = source.toCharArray();
+    this.chars = chars;
-    this.listener = l;
+    this.listener = listener;
-    index = 0;
   }
   void next() {
     if(++index >= chars.length) {
@@ -96,6 +44,67 @@
   @Override
   public void carryOver() { next(); }
 }
+class StringCounter implements CarryOverListener {
+  private List<CharCounter> ccList;
+  private boolean carry;
+  StringCounter(String moto) {
+    ccList = new ArrayList<>();
+    CharsRow charsRow = new CharsRow();
+    CarryOverListener prev = this;
+    for(char c : moto.toCharArray()){
+      CharCounter cc = new CharCounter(charsRow.get(c), prev);
+      ccList.add(cc);
+      prev = cc;
+    }
+  }
+  void outputAllPatternTo(String filename) throws IOException {
+    try (PrintStream out = new PrintStream(filename);) {
+      outputAllPatternTo(out);
+    }
+  }
+  void outputAllPatternTo(PrintStream out) {
+    for (CharCounter last=ccList.get(ccList.size()-1); !carry; last.next()) {
+      for(CharCounter cc : ccList) out.print(cc.get());
+      out.println();
+    }
+  }
+  @Override
+  public void carryOver() { carry = true; }
+}
+class CharsRow {
+  private static String[] ROWS = {
+    "あぁかがさざただなはばぱまやゃらわ",
+    "いぃきぎしじちぢにひびぴみりゐ",
+    "うぅくぐすずつづぬふぶぷむゆゅる",
+    "えぇけげせぜてでねへべぺめれゑ",
+    "おぉこごそぞとどのほぼぽもよょろ",
+  };
+  private Map<Character,char[]> cache = new HashMap<>();
+  void check(String moto) {
+    for(char c : moto.toCharArray()){
+      char[] r = get(c);
+      System.out.println(c + "は、" + r[0] + (r.length>=2?"段にあります":"です"));
+    }
+  }
+  char[] get(char c) {
+    for(String r : ROWS) {
+      if(r.indexOf(c) >= 0) {
+        char[] a = cache.get(r.charAt(0));
+        if(a == null) {
+          a = r.toCharArray();
+          cache.put(r.charAt(0), a);
+        }
+        return a;
+      }
+    }
+    char[] a = cache.get(c);
+    if(a == null) {
+      a = new char[]{c};
+      cache.put(c, a);
+    }
+    return a;
+  }
+}
 ```
 ```plain text
 どのような文の子音を変えますか？

getCharsRow 追加

2020/03/16 12:40

投稿

jimbe

スコア13355

answer CHANGED Viewed

@@ -15,13 +15,10 @@
   }
   private static void hikaku(String moto){
-    final String[] rows = {A,I,U,E,O,N};
     for(char c : moto.toCharArray()){
-      for(String r : rows) {
+      String r = getCharsRow(c);
-        if(r.indexOf(c) >= 0) {
+      if(r != null) {
-          System.out.println(c + "は、" + r.charAt(0) + (r.length()>=2?"段にあります":"です"));
+        System.out.println(c + "は、" + r.charAt(0) + (r.length()>=2?"段にあります":"です"));
-          continue;
-        }
       }
     }
   }
@@ -30,16 +27,9 @@
     List<CharCounter> ccList = new ArrayList<>();
     CarryFlag carryFlag = new CarryFlag();
     CarryOverListener prev = carryFlag;
-    String[] rows = {A,I,U,E,O};
     for(char c : moto.toCharArray()){
-      CharCounter cc = null;
-      for(String r : rows) {
+      String r = getCharsRow(c);
-        if(r.indexOf(c) >= 0) {
-          cc = new CharCounter(r, prev);
-          break;
-        }
-      }
-      if(cc == null) cc = new CharCounter(""+c, prev);
+      CharCounter cc = new CharCounter(r!=null?r:""+c, prev);
       ccList.add(cc);
       prev = cc;
     }
@@ -56,12 +46,20 @@
     }
   }
+  private static String getCharsRow(char c) {
+    for(String r : ROWS) {
+      if(r.indexOf(c) >= 0) return r;
+    }
+    return null;
+  }
   private static String A = "あぁかがさざただなはばぱまやゃらわ";
   private static String I = "いぃきぎしじちぢにひびぴみりゐ";
   private static String U = "うぅくぐすずつづぬふぶぷむゆゅる";
   private static String E = "えぇけげせぜてでねへべぺめれゑ";
   private static String O = "おぉこごそぞとどのほぼぽもよょろ";
   private static String N = "ん";
+  private static String[] ROWS = {A,I,U,E,O,N};
 }
 interface CarryOverListener {
   void carryOver();