二次元リストの分岐処理について

Question

2つの可変(同数)なリストを二次元配列にし、処理を分岐させたいです。
やりたい事のイメージは下記の様なことです。
・クライアント端末で動画を見た際に、サーバーに再生時間と再生停止時間を送ります。(ここの部分は不要です。サーバの勉強をしたいので)
・サーバは2つの時間をそれぞれ、リストに格納し、その後2次元配列にします。
・for文でリストに格納されたデータの回数分、if文で再生中である場合「1」を停止中である場合「0」を出力を繰り返す(判定は1秒)
・出力された値を配列に格納

現状書いてみたソースは以下の通りです。

//二次元配列
ArrayList<ArrayList<Integer>> list = new ArrayList<ArrayList<Integer>>();
//停止時間
list.add(stop);
//再生時間
list.add(play);

//リスト数分繰り返し
		for(int i = 0; i < stop.size(); ++i ){
			//下記からどのように記載すればよいか分かりません。

			}
		}

どなたか分かる方教えてください。よろしくお願い致します。

Comments

[LouiS0616]

stopTimeって唐突に出てきましたけど、いったい何なのでしょうか？

[hide09090909]

言葉足らずですみません。stopTimeは停止時間のリスト playTimeは再生時間のリストです。

Answer 1

ご丁寧な回答とアドバイスありがとうございます。
ご指摘頂いた点に気をつけていきます。

サンプルのソースについての質問があります。
一つめのソースの下記の部分がなにをしているのかよくわかりません。
二つめのソースは0,1をコンマ区切りで表示させるための機能という認識で良いでしょうか？
また、下記処理内容がわかりません
buffer = buffer.substring(0, buffer.length()

Answer 2

質問への回答

動作のイメージが本当に仕様どおりかわかりませんが。こんな感じですかね。

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も、ネーミングとしてはアウトです。

---

• ネストをはっきりさせる

具体的には、次の部分です。

//リスト数分繰り返し
        for(int i = 0; i < stop.size(); ++i ){
            //下記からどのように記載すればよいか分かりません。

        }
    }

for文のインデントが深くなっているので、外側のループについての質問かと思いました。

---

• 具体的な実行例を書く

イメージする実行例を書きましょう。
書いていく中でイメージがはっきりしてきて、自己解決することも期待できます。

---

• 宣言部は略さない

変数の型が明確になります。

---

以上、余計なお世話かもしれませんが... 作文力は一朝一夕で身につくものではありません。
ですから、これらのように別のアプローチで伝えるようにするといいかもしれません。

もし出力の形式に拘るなら

例えば以下のようにして、出力の形式をコントロールできます。

// 任意の型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標準の出力形式とほぼ同じですので、次のようにも書けます。

public static <T> void printList(List<T> list) {
    String buffer = list.toString();
    System.out.println(buffer.substring(1, buffer.length() - 1));
}

今後拡張／保守しやすくするには

二次元配列は何かとバグを生みやすいです。
クラスをご自身で定義し、そのリストを用いた方が良いでしょう。
ただし下記の例は、私が勘違いした仕様に基づいています。直接的な参考にはなりません。

public class MovieInfo {
    private final int start_;
    private final int stop_;

    public MovieInfo(int start, int stop) {
        this.start_ = start;
        this.stop_ = stop;
    }

    public int getStart() { return this.start_; }
    public int getStop()  { return this.stop_;  }

    public boolean isBeingPlayed() {
        /*
        色んな処理
        */

        return result;
    }
}

public void mainProcess() {
    List<MovieInfo> movieInfoList = new ArrayList<>();

    /*
    movieInfoListに情報を詰めていく
    */
    
    for(MovieInfo movieInfo: movieInfoList) {
        if(movieInfo.isBeingPlayed()) {
            System.out.println(1);
        }
        else {
            System.out.println(0);
        }
    }
}

Comments

[hide09090909]

ご回答ありがとうございます。
こういうやり方もあるんですね。
二次元配列の勉強もかねていたので、質問文のようなやり方も教えていただけるとうれしいです。
あつかましくてすみません。

[LouiS0616]

変数『stop』と『stopTime』の違いは何なのでしょう？

[hide09090909]

同じです。間違えました。すぐに修正します。

[LouiS0616]

動画が再生中かどうかは、どのように判断するのでしょうか。

[hide09090909]

リストの中に停止時間、再生時間を格納させているので、それを分岐させて判定させるつもりなのですが、それのイメージがわかず困っております

[LouiS0616]

時間は、このように取得できますが...
int nowStop = stop.get(0).get(i);
int nowStart = start.get(1).get(i);

[LouiS0616]

ひょっとして、
再生開始時刻 <= 今の時刻 <= 再生終了時刻　だったら再生中と判断とか、そういう話ですか？

[hide09090909]

リストの中にクライアントから送られてきた再生時間(秒)と停止時間(秒)は格納されています。
この秒とは相対時間で考えており、停止してからの秒数
再生してからの秒数がそれぞれ入っております。

動画の視聴イメージとしては
停止 → 再生 → 停止 → 再生を繰り返すイメージです。
それを時系列で0,1で表示させたいということです。

後だしの情報すみません、説明がとても下手なのは理解してます。何度もすみません

[LouiS0616]

例えば、
10秒再生→20秒停止→5秒再生→10秒停止→... といった情報が格納されるのでしょうか？
そして、この場合
1, 0, 1, 0... と表示したいということですか？

[hide09090909]

はい、そうです
表示は秒数単位で行いたいので、
1,1,1,1,1…,0,0,0,0,0…1,1,1,1,1,0,0,0…のようにしたいです。
それの値を配列にいれたいということです

[LouiS0616]

for(int elem: stop) {
...System.out.print("1, 0, ");
}
で充分では。

[hide09090909]

すみません、コメントを修正しました。

[LouiS0616]

もう一個だけ確認します。
start, stop に格納されているのは、『開始／停止時刻』ですか？
それとも、『開始／停止継続時間』ですか？

[hide09090909]

『開始／停止継続時間』です

[hide09090909]

ご丁寧な回答とアドバイスありがとうございます。
ご指摘頂いた点に気をつけていきます。

サンプルのソースについての質問があります。
一つめのソースの下記の部分がなにをしているのかよくわかりません。
二つめのソースは0,1をコンマ区切りで表示させるための機能という認識で良いでしょうか？
また、下記処理内容がわかりません
buffer = buffer.substring(0, buffer.length()

[LouiS0616]

えと、実際に使ってみるとどうなります？
私自身が理解した仕様と、質問者様のイメージが本当に合っているのかまず確かめたいのです。

[hide09090909]

すみません、いまパソコンを置いて出掛けてしまっているので、実行できないです。

明日動かしてみますが、処理の内容を理解したいです。

[LouiS0616]

コードにコメントを追加しました。

[LouiS0616]

また、さりげなく今書き足しておいたんですが...
出力の形式に拘らないのであれば、System.out.println(playLog);で充分です。
すみません、私は細かいところまでカスタマイズしたくなってしまうタイプなのです...

[LouiS0616]

回答がとんでもなく煩雑になっていたので、整理しました。