条件分岐がうまくいかない

Question

###実現したいこと
下記のようにLinesという配列の中身をif文を使って判定し、
resultを出力したいのですが、
Line[0]>=1,Lines[1]<=2,Lines[2]<=3の時に

Lines[0]は1、Lines[1]は2、Lines[2]は3に届かない分だけresultに加算して出力しようと考えてプログラミングしたはずなのですが、

例)
入力：Lines={3,2,1}
期待する出力：2
実際の出力：6

と出力されてしまいます。
if文のどこが間違っているのか教えていただけないでしょうか。

int result=0;
    for (int i=0; i<Lines.length; i++) {
      if(Lines[0]<1) { result += 1-Lines[0]; }
      if(Lines[1]<2) { result += 2-Lines[1]; }
      if(Lines[2]<3) { result += 3-Lines[2]; }
      else { result += 0; }
    }
	  System.out.println(result);

Answer 1

result に0が加算されないのですが、なぜでしょうか。

仮に加算されてもされなくても、値は変わりません。

Comments

[azuapricot]

（確かにかわらないのに何がしたいんでしょうね）

[d_96a]

ご回答ありがとうございます。
質問に少し追記させていただきますと、
例えばLines={3,2,1}となっている際に、
2と出力されるようにプログラミングをしているつもりなのですが、
6と出力されてしまいます。
こちらはなぜこうなってしまっているのでしょうか。

[maisumakun]

> 2と出力されるようにプログラミングをしているつもりなのですが、

まず根本的な問題として、ループ内でiを使っていないため、同じことを3回繰り返すコードとなっています。

（ということで、どのようなロジックで「2」と計算したいのかがまずわかりません）

[azuapricot]

if(Lines[2]<3) {
　result += 3-Lines[2];
}

1 < 3 なので この3番目のif文の処理が Lines.length分 (3回)　繰り返されますよね
3-1 = 2
2 + 2 + 2 = 6ですよね

[d_96a]

なるほどfor文を消すと解決できました。
ありがとうございました。

Answer 2

        int result=0;
        for (int i=0; i<Lines.length; i++) {
          if(Lines[0]<1) {
            result += 1-Lines[0]; 
          }
          
          if(Lines[1]<2) {
            result += 2-Lines[1]; 
          }
          
          if(Lines[2]<3) { // こいつが falseの時しかelseは通りませんよ
            result += 3-Lines[2];
              
          }　else { 
            result += 0; 
          }
        }

インデントをきちんとそろえればすぐわかると思いますが、
このelse文は Lines[2] < 3 の結果が falseの時にしか通りません