編集履歴

回答編集履歴

回答が尋常じゃなく長くなってしまったので、整理

2017/09/07 15:59

投稿

スコア35660

test CHANGED Viewed

@@ -1,10 +1,214 @@
-元の回答
+質問への回答
----
+---
+動作のイメージが本当に仕様どおりかわかりませんが。こんな感じですかね。
+```Java
+List<Integer> playLog = new ArrayList<>();
+for(int i = 0; i < stop.size(); i++) {
+    int nowStart = list.get(1).get(i);
+    int nowStop  = list.get(0).get(i);
+    for(int j = 0; j < nowStart; j++) {
+        playLog.add(1);
+    }
+    for(int j = 0; j < nowStop; j++) {
+        playLog.add(0);
+    }
+}
+System.out.println(playLog);
+```
+質問自体への回答は以上です。以降の回答はちょっと脱線気味です。
+仕様を正しく伝えるために
+---
+口下手なことをしきりに気にしておられるようですが...
+なかなか上手く説明できないのなら、『語るコード』を書くべきなのです。
+- **適切な変数名を付ける**
+基本的に変数名は横文字になるかと思いますが、可能な限り、その性質を表す言葉にしましょう。
+ex) `stop` → `pauseDurationSecs`
+0. stopより、pauseの方が一時停止の意味合いが強い(多分)。
+0. 継続時間であることを明示するために、Durationを付ける。
-すみません、ちょっと外した回答かもしれません。
+0. 単位を明確にするために、Secと付ける。
+0. リストであることを明示するために、Listと付けるか複数形にする。
+0. 変数であることを明示するために、camelCaseを用いる。
+`play`も`list`も、ネーミングとしてはアウトです。
+---
+- **ネストをはっきりさせる**
-次のように設計すればよいのでは？
+具体的には、次の部分です。
+> ```Java
+> //リスト数分繰り返し
+>         for(int i = 0; i < stop.size(); ++i ){
+>             //下記からどのように記載すればよいか分かりません。
+>             }
+>         }
+> ```
+for文のインデントが深くなっているので、外側のループについての質問かと思いました。
+---
+- **具体的な実行例を書く**
+イメージする実行例を書きましょう。
+書いていく中でイメージがはっきりしてきて、自己解決することも期待できます。
+---
+- **宣言部は略さない**
+変数の型が明確になります。
+---
+以上、余計なお世話かもしれませんが... 作文力は一朝一夕で身につくものではありません。
+ですから、これらのように別のアプローチで伝えるようにするといいかもしれません。
+もし出力の形式に拘るなら
+---
+例えば以下のようにして、出力の形式をコントロールできます。
+```Java
+// 任意の型Tに関する、List<T>を引数に取る
+public static <T> void printList(List<T> list) {
+    // 返り値用の文字列を宣言・初期化
+    String buffer = "";
+    // 引数listに属するすべての要素を、Tという型のelemとして扱う
+    for(T elem: list) {
+        // 返り値用の文字列に継ぎ足していく
+        buffer += elem + ", ";
+    }
+    // このままだと最後に無駄な", "が付いてしまうので、二文字分尻から削る
+    buffer = buffer.substring(0, buffer.length() - 2);
+    System.out.println(buffer);
+}
+```
+**『Tってなに？』**
+→ ジェネリクスについて調べてみてください。
+**『for文なんか変じゃない？』**
+→ 拡張for文について調べてみてください。
+---
+ただし、上記の例ではList標準の出力形式とほぼ同じですので、次のようにも書けます。
+```Java
+public static <T> void printList(List<T> list) {
+    String buffer = list.toString();
+    System.out.println(buffer.substring(1, buffer.length() - 1));
+}
+```
+今後拡張／保守しやすくするには
+---
+二次元配列は何かとバグを生みやすいです。
+クラスをご自身で定義し、そのリストを用いた方が良いでしょう。
+**ただし下記の例は、私が勘違いした仕様に基づいています。直接的な参考にはなりません。**
 ```Java
@@ -89,207 +293,3 @@
 ```
-追記
----
-仕様がようやくわかってきたので追記します。こんな感じですかね。
-```Java
-List<Integer> playLog = new ArrayList<>();
-for(int i = 0; i < stop.size(); i++) {
-    int nowStart = list.get(1).get(i);
-    int nowStop  = list.get(0).get(i);
-    for(int j = 0; j < nowStart; j++) {
-        playLog.add(1);
-    }
-    for(int j = 0; j < nowStop; j++) {
-        playLog.add(0);
-    }
-}
-if(出力をカスタマイズしたい) {
-    printList(playLog);
-}
-else {
-    System.out.println(playLog);
-}
-```
-```Java
-// 任意の型Tに関する、List<T>を引数に取る
-public static <T> void printList(List<T> list) {
-    // 返り値用の文字列を宣言・初期化
-    String buffer = "";
-    // 引数listに属するすべての要素を、Tという型のelemとして扱う
-    for(T elem: list) {
-        // 返り値用の文字列に継ぎ足していく
-        buffer += elem + ", ";
-    }
-    // このままだと最後に無駄な", "が付いてしまうので、二文字分尻から削る
-    buffer = buffer.substring(0, buffer.length() - 2);
-    System.out.println(buffer);
-}
-```
-**『Tってなに？』**
-→ ジェネリクスについて調べてみてください。
-**『for文なんか変じゃない？』**
-→ 拡張for文について調べてみてください。
----
-**よく考えたら、次のコードで充分でした。**
-```Java
-public static <T> void printList(List<T> list) {
-    String buffer = list.toString();
-    System.out.println(buffer.substring(1, buffer.length() - 1));
-}
-```
-難しく考えすぎた。
-蛇足
----
-口下手なことをしきりに気にしておられるようですが...
-なかなか上手く説明できないのなら、『語るコード』を書くべきなのです。
-- **適切な変数名を付ける**
-基本的に変数名は横文字になるかと思いますが、可能な限り、その性質を表す言葉にしましょう。
-ex) `stop` → `pauseDurationSecs`
-0. stopより、pauseの方が一時停止の意味合いが強い(多分)。
-0. 継続時間であることを明示するために、Durationを付ける。
-0. 単位を明確にするために、Secと付ける。
-0. リストであることを明示するために、Listと付けるか複数形にする。
-0. 変数であることを明示するために、camelCaseを用いる。
-`play`も`list`も、ネーミングとしてはアウトです。
----
-- **ネストをはっきりさせる**
-具体的には、次の部分です。
-> ```Java
-> //リスト数分繰り返し
->         for(int i = 0; i < stop.size(); ++i ){
->             //下記からどのように記載すればよいか分かりません。
->             }
->         }
-> ```
-for文のインデントが深くなっているので、外側のループについての質問かと思いました。
----
-- **具体的な実行例を書く**
-イメージする実行例を書きましょう。
-書いていく中でイメージがはっきりしてきて、自己解決することも期待できます。
----
-- **宣言部は略さない**
-変数の型が明確になります。
----
-以上、余計なお世話かもしれませんが... 作文力は一朝一夕で身につくものではありません。
-ですから、これらのように別のアプローチで伝えるようにするといいかもしれません。

無駄なコードの修正

2017/09/07 15:59

投稿

LouiS0616

スコア35660

test CHANGED Viewed

@@ -184,6 +184,28 @@
+---
+**よく考えたら、次のコードで充分でした。**
+```Java
+public static <T> void printList(List<T> list) {
+    String buffer = list.toString();
+    System.out.println(buffer.substring(1, buffer.length() - 1));
+}
+```
+難しく考えすぎた。
 蛇足
 ---

追記

2017/09/07 15:30

投稿

LouiS0616

スコア35660

test CHANGED Viewed

@@ -124,7 +124,17 @@
+if(出力をカスタマイズしたい) {
-printList(playLog);
+    printList(playLog);
+}
+else {
+    System.out.println(playLog);
+}
 ```

追記

2017/09/07 15:25

投稿

LouiS0616

スコア35660

test CHANGED Viewed

@@ -132,16 +132,28 @@
 ```Java
+// 任意の型Tに関する、List<T>を引数に取る
 public static <T> void printList(List<T> list) {
+    // 返り値用の文字列を宣言・初期化
     String buffer = "";
+    // 引数listに属するすべての要素を、Tという型のelemとして扱う
     for(T elem: list) {
+        // 返り値用の文字列に継ぎ足していく
         buffer += elem + ", ";
     }
+    // このままだと最後に無駄な", "が付いてしまうので、二文字分尻から削る
     buffer = buffer.substring(0, buffer.length() - 2);
     System.out.println(buffer);
@@ -152,6 +164,16 @@
+**『Tってなに？』**
+→ ジェネリクスについて調べてみてください。
+**『for文なんか変じゃない？』**
+→ 拡張for文について調べてみてください。
 蛇足
 ---

追記

2017/09/07 15:19

投稿

LouiS0616

スコア35660

test CHANGED Viewed

@@ -1,3 +1,7 @@
+元の回答
+---
 すみません、ちょっと外した回答かもしれません。
 次のように設計すればよいのでは？
@@ -83,3 +87,155 @@
 }
 ```
+追記
+---
+仕様がようやくわかってきたので追記します。こんな感じですかね。
+```Java
+List<Integer> playLog = new ArrayList<>();
+for(int i = 0; i < stop.size(); i++) {
+    int nowStart = list.get(1).get(i);
+    int nowStop  = list.get(0).get(i);
+    for(int j = 0; j < nowStart; j++) {
+        playLog.add(1);
+    }
+    for(int j = 0; j < nowStop; j++) {
+        playLog.add(0);
+    }
+}
+printList(playLog);
+```
+```Java
+public static <T> void printList(List<T> list) {
+    String buffer = "";
+    for(T elem: list) {
+        buffer += elem + ", ";
+    }
+    buffer = buffer.substring(0, buffer.length() - 2);
+    System.out.println(buffer);
+}
+```
+蛇足
+---
+口下手なことをしきりに気にしておられるようですが...
+なかなか上手く説明できないのなら、『語るコード』を書くべきなのです。
+- **適切な変数名を付ける**
+基本的に変数名は横文字になるかと思いますが、可能な限り、その性質を表す言葉にしましょう。
+ex) `stop` → `pauseDurationSecs`
+0. stopより、pauseの方が一時停止の意味合いが強い(多分)。
+0. 継続時間であることを明示するために、Durationを付ける。
+0. 単位を明確にするために、Secと付ける。
+0. リストであることを明示するために、Listと付けるか複数形にする。
+0. 変数であることを明示するために、camelCaseを用いる。
+`play`も`list`も、ネーミングとしてはアウトです。
+---
+- **ネストをはっきりさせる**
+具体的には、次の部分です。
+> ```Java
+> //リスト数分繰り返し
+>         for(int i = 0; i < stop.size(); ++i ){
+>             //下記からどのように記載すればよいか分かりません。
+>             }
+>         }
+> ```
+for文のインデントが深くなっているので、外側のループについての質問かと思いました。
+---
+- **具体的な実行例を書く**
+イメージする実行例を書きましょう。
+書いていく中でイメージがはっきりしてきて、自己解決することも期待できます。
+---
+- **宣言部は略さない**
+変数の型が明確になります。
+---
+以上、余計なお世話かもしれませんが... 作文力は一朝一夕で身につくものではありません。
+ですから、これらのように別のアプローチで伝えるようにするといいかもしれません。