java 配列代入したものを倍数計算する

Question

前提・実現したいこと

配列代入したものを倍数確認をする。

発生している問題・エラーメッセージ

for文やif文を用いて計算したところ代入した要素数ではなく要素番号を計算してしまっていると考えています。

エラーメッセージ

要素番号を計算してしまっている為　※0番目から4番目
4の倍数です
48
55
68
4の倍数です
と表示されています。

該当のソースコード

ソースコード

public class sample {
public static void main(String[] args) {
//5つの要素を持ったListを宣言し、代入する
int[]list = {35,48,55,68,75};
for (int i=0; i<list.length; i++) {
if ( i % 4 == 0) {
System.out.println("4の倍数です");
}else if(i % 5 == 0) {
System.out.println("5の倍数です");
}else{
System.out.println(list[i]);
}
}
}
}

試したこと

要素番号が計算されているので　for (int i=0; i<list.length; i++) {　
for文が間違っていると予想できます。※計算したい場所が参照できていない為
沢山調べたりしましたが手付かずになってしまった為、助けていただける方お願い致します。

補足情報（FW/ツールのバージョンなど）

ここにより詳細な情報を記載してください。

Answer 1

i % 4 == 0

などとなっているところを

list[i] % 4 == 0

という風にしたらどうでしょうか。

public class sample {
  public static void main(String[] args) {
  //5つの要素を持ったListを宣言し、代入する
    int[]list = {35,48,55,68,75};
    for (int i=0; i<list.length; i++) {
      int val = list[i];
      if (val % 4 == 0) {
        System.out.println("4の倍数です");
      }else if(val % 5 == 0) {
        System.out.println("5の倍数です");
      }else{
        System.out.println(val);
      }
    }
  }
}

Comments

[domimimimimimi]

iだけだと要素番号を参照してしまったんですね…理解しました！(´;ω;｀)
解決致しました。ありがとうございました！

[PC_breakman]

蛇足ですがこの書き方だとifの度にメモリにいちいちアクセスするのでよくないですね。値をどっか別の変数に保持しておいた方が良いかもです。(詳細は追記)

[domimimimimimi]

これからお仕事にしたいと思っているので、こういうのはとても有難いです。
ありがとうございます！！
他の文によってもif文などは他の変数に変えて負担を減らす事を意識した方が良いですか？

[PC_breakman]

if文というかlist[i]を何度も書くということはそれだけlist[i]がある場所に何度もアクセスするということなのでプログラムの効率が悪くなるということです。これくらいのプログラムなら大した問題にはならないですがもっと大きなプログラムを書くときに問題になるかもしれません。詳しくはメモリ、レジスタ、キャッシュなどについて調べてみるといいと思います。

[jimbe]

>お仕事にしたい

ということですので、効率に関しまして別の意見も１つ。

このレベルのコードから効率を求めることを考えるのはお勧め出来ません。

手法を知っていることはもちろん無駄にはなりませんが、それを行うのは、要求性能を満たせ無かった場合にプロファイリング等で問題箇所を特定してからでなければ、無駄にコードを弄り、読みにくいコードになるだけになります。

適切なデータ構造・アルゴリズムを用い、分かり易いクラス・メソッド分割を行うほうを先ずは意識して戴きたいです。

[domimimimimimi]

申し訳ございません、只今拝見しました。
ありがとうございます。
見返した時にわかりやすいコードを書くのを意識してみます！