検索で該当するものがない場合、即文字列を反映させたいです。tu

Question

やっていること
登録商品の登録・編集・消去などができるコンソールアプリをつくっています。
今回質問させていただくのは編集部分となります。
Main.javaにて、変更したい登録商品名・変更後の名前を聞いて、Editクラスにて編集を行います。

聞きたいこと・困っていること
編集したい商品を打ち込む際に、該当するものがない場合result == ""で"編集する商品が見つかりませんでした”と表示されるようになっています。
しかし、「編集する商品が見つかりませんでした」が表示されるのが
すべての入力項目（編集したい登録名・新しい登録名・新しい価格などMainで聞いてることすべて）の入力終わった後に「編集する商品が見つかりませんでした」と表示される現状です。

これを、編集したい登録商品名を入力して、合致するものがないとわかった場合、新しい商品名などつぎの項目入力にすすむことなくすぐに「編集する商品が見つかりませんでした」と出したいです。どのように変更すればよいのでしょうか？

【現在】
編集したい登録商品名の文字列を入力してください
hh（登録されてない文字列を入力）
新しい商品名を入力してくださいとすべての入力項目終了後
編集する商品が見つかりませんでした

【このように変更したい】
編集したい登録商品名の文字列を入力してください
hh（登録されてない文字列を入力）
編集する商品が見つかりませんでした

**Main.java**

if(sentaku ==3) {
			Scanner editword = new  java.util.Scanner(System.in);
			String editname;
			do {
			System.out.println("編集したい登録商品名の文字列を入力してください");
			editname = editword.nextLine();

			}while(editname.isEmpty());



			String changename;
			do {
			System.out.println("新しい商品名を入力してください");
			changename = editword.nextLine();
			}while(changename.isEmpty());

			int changemoney;
			while(true) {
			try {
			System.out.println("新しい値段を入力してください");
			changemoney = editword.nextInt();
			break;
			}catch(InputMismatchException e) {
				System.out.println("文字列が入力されました。");
				editword.next();
			}
			}

			String changekari;
			do {
			System.out.println("仮の場合は１を本登録の場合は２を入力してください。");
				changekari = editword.next();
				if(changekari.equals("1")) {
					changekari="仮商品";
				}else {
					changekari="本登録商品";
				}

				}while(changekari.isEmpty());

			Edit.Changeproduct(editname,changename,changemoney,changekari,list);

public class Edit {
	public static void Changeproduct(String editname,String changename,int changemoney,String changekari,ArrayList<HandMadeGoods> list)
	{


		String result = "";
		File file;
		PrintWriter pw = null;

		for(int i=0; i<list.size(); i++) {

			if(editname.equals(list.get(i).getName())) {


				list.get(i).setName(changename);
				list.get(i).setMoney(changemoney);
				list.get(i).setKari(changekari);

				result +="商品名:"+list.get(i).getName()+"価格:"+list.get(i).getMoney()+"仮番号:"+list.get(i).getKari()+"で編集完了しました。";

				try{
					file = new File("src\HandMadeData.txt");
					pw = new PrintWriter(
							new BufferedWriter(new FileWriter(file))
					);
					for(int y=0; y<list.size(); y++) {

				    	pw.println("商品名:"+list.get(y).getName()+"/"+list.get(y).getMoney()+"円/"+list.get(y).getKari());

				}

				}catch(IOException e){
					System.out.println("該当ファイルがありません。");
				}catch(NullPointerException e){
					System.out.println("nullです。");
				}finally {
					pw.close();
				}


				}
				}	if(result == "") {

					result = "編集する商品がみつかりませんでした";
				}
				System.out.println(result);



	}
}

**追記 checkcheck**

		public static boolean checkcheck(String editname,ArrayList<HandMadeGoods> list,boolean checkgo) {
		for(int t=0; t<list.size(); t++) {
			if(!(editname.equals(list.get(t).getName()))){
				System.out.println("編集したい商品に合致するものはありません");
				checkgo=false;
			}
}
		return checkgo;
	}

**追記　main**
            Scanner editword = new  java.util.Scanner(System.in);
			String editname;
			boolean checkgo=true;

			do {
			System.out.println("編集したい登録商品名の文字列を入力してください");
			editname = editword.nextLine();

			}while(editname.isEmpty());

			Edit.checkcheck(editname,list,checkgo);

Answer 1

if文の修正レベルでなく、処理フローを変更する必要があるのですが・・・
そこは理解できてますか？
そもそも、Changeproductを呼ぶタイミングがもろもろの入力の後なので。

”編集したい登録商品名の文字列を入力してください”

で、文字が入力されたらチェックをする必要があります。

ですが、Changeproductは、名前以外の入力も必要なので、これとは別に
新しい検索処理を作成する必要があります。

---

追記に対応

戻り値を受けて処理を迂回すればよいです。Mainを完全に把握してるわけじゃないので
イメージレベルで

**Main.java**

if(sentaku ==3) {
            Scanner editword = new  java.util.Scanner(System.in);
            String editname;
            do {
            System.out.println("編集したい登録商品名の文字列を入力してください");
            editname = editword.nextLine();

            }while(editname.isEmpty());

            if(Edit.checkcheck(editname,list,checkgo)) {
                // 元の質問にあった、続きの処理をこのブロックに書けばよいです。
                String changename;
                do {
                    System.out.println("新しい商品名を入力してください");
                    changename = editword.nextLine();
                }while(changename.isEmpty());

                // 以降省略
            }

Mainの中身しだいですが以下のように書けるかもしれません。
ネストが深くならないので、こちらの方が好みです。
とはいえ、Mainの作りしだい。

       if(!Edit.checkcheck(editname,list,checkgo)) {
            return;
       }
       // 以降は元の質問にあった、続きの処理

あと今回のようなメソッドは戻り値を引数で受け取る必要はありません。

処理で気になったのは、見つからない場合っていうのは全てがequalsでfalseになった場合じゃないですかね？
listの中身がよくわかってないですが、１つでも違うと見つからないになるとしたら
ほぼほぼ合致しない(Listの中身が全て同じ？)ことになりますけど・・・

public static boolean checkcheck(String editname,ArrayList<HandMadeGoods> list) {
    for(int t=0; t<list.size(); t++) {
        if((editname.equals(list.get(t).getName()))){
            // 一致したらtrueで終了
            return true;
        }
    }
    // 全て合致しなかった場合
    System.out.println("編集したい商品に合致するものはありません");
    return false;
}

Comments

[chachachamaru]

回答ありがとうございます。なるほど...勉強不足な点、失礼いたしました。文字列が入力されたらチェックができるように、mainの”編集したい登録商品名の文字列を入力してください” のdowhileが終わったすぐ下にCheckedit.checkcheck(editname,list);というメゾット（中身はfor文で回しながら同意する名前があるかをさがす）を持ってきてみました。この処理が終わった後、ちゃんと合致しませんでした。と表示されまた続きの処理（mainのつぎの質問）に戻っています。メゾット内の処理が終わったので続きの処理に戻るのは承知の上で聞きたいのですが、このcheckcheckメゾットに入った場合(sentaku ==3)の処理を終わらすにはcheckcheckメゾットにどう記述すればよいのでしょうか？重ねての質問になりますがよろしくお願いいたします。

[momon-ga]

checkcheckに戻り値をつけてください。
戻り値によって、処理を続行するかどうかをifで判定すればよいです。

[chachachamaru]

回答いただいてから今までいろいろやってみたのですがreturnの使い方が間違っているのかうまくいかないので、どのようにして記述すればいいかご教授いただけますでしょうか？javaを学び始めてから１か月も経ってないため経験が少なく申し訳ないです。よろしくお願いいたします。

[momon-ga]

質問を最新化してください。（うまくいかないコードを載せてください）

[chachachamaru]

追記にて行ってみたコードを載せさせていただきました。falseになったら(sentaku ==3)を終了させると考えてみたのですが、mainでどう指示してよいかわからずに迷っています。またreturn文の使い方も間違っていましたらご指摘ください。

[chachachamaru]

本当に丁寧に解説いただきありがとうございました。無事、やりたいことにたどり着くことができました。今回ご教授頂いたことで、多くのことを学べましたこと感謝しております。また、何度も質問に答えて下さり助かりました。ありがとうございます！ここからまた励んでいきたいと思います。