for文の継続条件を変更したいです。

Question

実現したいこと

ここに実現したいことを箇条書きで書いてください。

• 通常のfor文を使用して記述しなければなりません。
• Mapの大きさ分for文を回すような記述に変更しなければなりません。
• beafMap1について通常のfor文のループカウンタ初期化の部分と継続条件をわかりやすくする必要があります。
• Mapの値を出力する処理を記述したい部分は、for文のループカウンタの継続条件がListのサイズ分にならないように記述する必要があります。

前提

出力結果は以下の結果となるようにしないといけません。

----- for文: 焼き肉の部位牛リスト -----
カルビ
ロース
サブトン
----- 拡張for文: 焼き肉の部位リスト -----
タン
ハラミ

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

エラーメッセージは表示されていません。
ソースコードの内容を修正する必要があります。

該当のソースコード

package check;

import java.util.ArrayList;
import java.util.HashMap;
import java.util.List;
import java.util.Map;

public class Check {

    /* 定数（焼き肉の部位） */
    public static final String GYU_100_CALVI = "カルビ";
    public static final String GYU_101_TONGUE = "タン";
    public static final String GYU_102_LOIN = "ロース"; 
    public static final String GYU_103_HARAMI = "ハラミ";
    public static final String GYU_104_ZABUTON = "サブトン"; 

    public static void main(String[] args) {
        
        int startNumber = 100;
        
        List<String> beafList = new ArrayList<>();
        beafList.add(GYU_100_CALVI);
        beafList.add(GYU_101_TONGUE);
        beafList.add(GYU_102_LOIN);
        beafList.add(GYU_103_HARAMI);
        beafList.add(GYU_104_ZABUTON);
        
        HashMap<Integer, String> beafMap1 = new HashMap<>();
        
        // 問：ループ文を利用して、「beafList」より「beafMap1」に値を設定しなさい。
        // また、値を設定する際のキー値には「startNumber」を利用しなさい。
       
        // 問: beafMap1
        // ルール:【for文】を使用して値を設定しなさい。
        // ここへ記述
        for (int i = 0; i < beafList.size(); i++) {
            beafMap1.put(startNumber+ i ,beafList.get(i));
        }
        System.out.println("----- for文: 焼き肉の部位牛リスト -----");
        
        // 問: beafMap1
        // ルール:【for文】を使用して値を出力しなさい。
        for (int key :beafMap1.keySet()) {
            if (key % 2 ==0 ) {
                System.out.println(beafMap1.get(key));
            }
        }
       
        startNumber = 100;

        HashMap<Integer, String> beafMap2 = new HashMap<>();
                
        for (String beaf : beafList) {
            beafMap2.put(startNumber++, beaf);
        }
        
        System.out.println("----- 拡張for文: 焼き肉の部位リスト -----");
        
        for (Map.Entry<Integer , String> entry :beafMap2.entrySet()) {
            if (entry.getKey() % 2 != 0) { 
                System.out.println(entry.getValue());
            }
        }
    }
}

試したこと

for文の継続条件が「i < beafList.size();」とListの大きさ分for文を回すような記述になっているから、
他に考えられる方法を考えて、コンパイルを実行しました。
全て不適切でした。
以下のコードなどはダメみたいです。

for (int i = 0; i < beafList.size(); i++) {
beafMap1.put(startNumber + i, beafList.get(i));
}

    System.out.println("----- for文: 焼き肉の部位牛リスト -----");
    
    for (int i = 0; i < beafMap1.size(); i++) {
        int key = startNumber + i;
        if (i % 2 == 0) { // Even iteration (i = 0, 2, 4, ...)
            System.out.println(beafMap1.get(key));
        }
    }

あと、これもダメみたいです。

for (int i = 0; i < beafList.size(); i++) {
beafMap1.put(startNumber + i, beafList.get(i));
}

    System.out.println("----- for文: 焼き肉の部位牛リスト -----");

    for (int key = startNumber; key < startNumber + beafList.size(); key++) {
        if (key % 2 == 0) {
            System.out.println(beafMap1.get(key));
        }
    }

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

ここにより詳細な情報を記載してください。
・Javaを学び始めて2週間の初心者です。
・ツールはeclipseを使用しています。
・独学中です。
・踏まえておくべき基礎知識や実務経験はありません。
・答えが分からず、つまずいています。
・何が間違っているのか、分からないまま困っており、手が止まっています。
・chatGPTやGeminiを使いながら、学習しています。
・困ったことを聞いたり、わからないことを聞ける人がいません。

Comments

[TakaiY]

独学とありますが、問題を解いているように見えます。この問題はどこから出たものですか？ また、問題そのものを載せることはできますか？

コードの どの部分を修正するのかよくわらなず、出題の意図と内容がよくわからないので、アドバイスしにくいです。

[meg_]

> ・独学中です。
問題は参考書や学習サービスからの抜粋でしょうか？どこからの引用か明記した方が良いかと思います。また、サービスによっては転載不可のものもあるかと思いますがその辺りは大丈夫ですか？

[jimbe]

36～38行目の部分は for 文になってますから完了していると見て良いのでしょうか。
43～47行目が拡張 for 文になっていますので、質問はこの部分ということでよろしいですか？

試したといわれる2つが『不適切』でダメみたいということですが、2つ目は条件に beafList.size() があるからとしても1つ目にはありませんし、具体的に何がダメなのでしょう。
それがはっきりしなければあれは？これは？とただ試しまくるしか無く、回答もはっきり書けないのでは無いでしょうか。

Answer 1

どこかのオンライン学習サービスかプログラミングスクールの教材みたいですね。
この問題の意図を推測すると、

普通のfor文だと大変だけど、拡張for文だと同じことが簡単に書けるよ!

ということなのかなと思いますので、それが正しいと仮定して回答しますね。

簡単な回答

この問題でしか動かない回答になりますが、これが一番シンプルでおそらく出題者の意図にそったものな気がします。

beafListのkeyはstartNumberから始まる1ずつ増える(連番になっている)値になっている

という前提を利用したものです。

    for (int i = 0; i < beafMap1.size(); i++) {
      int key = startNumber + i;
      if (key % 2 == 0) {
        System.out.println(beafMap1.get(key));
      }
    }

これならば

for文のループカウンタの継続条件がListのサイズ分にならないように記述する

を満たしているので大丈夫なのではないかと思いますがどうでしょうか？

すこし汎用的な回答

上の方法は「keyの最初の値と、keyが連番である」ことを前提としたコードなので、もうすこし汎用的に動くようにしてみます。
import文を見るかぎりIteratorを使う想定はしていないように見えるので、一度keyを配列に変換して1つ1つアクセスするようにしてみます。

    Integer[] keys = beafMap1.keySet().toArray(new Integer[0]);
    for (int i = 0; i < keys.length; i++) {
      int key = keys[i];
      if (key % 2 == 0) {
        System.out.println(beafMap1.get(key));
      }  
    }

拡張for文を使ってMapを簡単にアクセスするための問題だと思いますので、「SetをArrayに変換してkeyを取りだす」はちょっとやりすぎな気がしますので、おそらく前の処理が意図したものなんじゃないかいかと思っています。
ご参考までに。