質問をすることでしか得られない、回答やアドバイスがある。

15分調べてもわからないことは、質問しよう!

新規登録して質問してみよう
ただいま回答率
85.47%
Java

Javaは、1995年にサン・マイクロシステムズが開発したプログラミング言語です。表記法はC言語に似ていますが、既存のプログラミング言語の短所を踏まえていちから設計されており、最初からオブジェクト指向性を備えてデザインされています。セキュリティ面が強力であることや、ネットワーク環境での利用に向いていることが特徴です。Javaで作られたソフトウェアは基本的にいかなるプラットフォームでも作動します。

Android Studio

Android Studioは、 Google社によって開発された、 Androidのネイティブアプリケーション開発に特化した統合開発ツールです。

Q&A

解決済

1回答

583閲覧

List<Map>で条件によってMapを削除する方法

tasojiro

総合スコア16

Java

Javaは、1995年にサン・マイクロシステムズが開発したプログラミング言語です。表記法はC言語に似ていますが、既存のプログラミング言語の短所を踏まえていちから設計されており、最初からオブジェクト指向性を備えてデザインされています。セキュリティ面が強力であることや、ネットワーク環境での利用に向いていることが特徴です。Javaで作られたソフトウェアは基本的にいかなるプラットフォームでも作動します。

Android Studio

Android Studioは、 Google社によって開発された、 Androidのネイティブアプリケーション開発に特化した統合開発ツールです。

0グッド

0クリップ

投稿2021/05/21 07:54

前提・実現したいこと

現在、AndroidStudioでアプリを作成しています。

実現したい動作としては「タグ検索」のようなものです。
MainActivityにリストビュー(タグ一覧)があり、いずれかのリストを選択するとそのリスト名の属性を持った要素だけがSubActivityでリスト表示されるといったものを目指しています。

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

問題は、どのリストを選択しても同一のリストしか返ってきません。

下のコードから具体的な例を挙げると、「飲食店」を選択しても「暇つぶし」を選択しても「施設B」のリストしか返ってきません。

理想では、「飲食店」を選択した際には「施設B」を、「暇つぶし」を選択した際には「施設A」と「施設C」が返ってくる動作です。

該当のソースコード

Main Activity

Java

1public class MainActivity extends AppCompatActivity { 2 3 @Override 4 protected void onCreate(Bundle savedInstanceState) { 5 super.onCreate(savedInstanceState); 6 setContentView(R.layout.activity_main); 7 8 // タグをリスト表示にする。 9 ListView lvtag = findViewById(R.id.lvtag); 10 List<Map<String, String>> tagList = new ArrayList<>(); 11 // リストの追加 12 Map<String, String> tag = new HashMap<>(); 13 tag.put("name", "飲食店"); 14 tagList.add(tag); 15 // リストの追加 16 tag = new HashMap<>(); 17 tag.put("name", "暇つぶし"); 18 tagList.add(tag); 19 20 String[] from = {"name"}; 21 int[] to = {android.R.id.text1}; 22 SimpleAdapter adapter = new SimpleAdapter(MainActivity.this, tagList, android.R.layout.simple_list_item_1, from, to); 23 lvtag.setAdapter(adapter); 24 25 lvtag.setOnItemClickListener(new ListItemClickListener()); 26 } 27 28 private class ListItemClickListener implements AdapterView.OnItemClickListener{ 29 @Override 30 public void onItemClick(AdapterView<?> parent, View view, int position, long id) { 31 // リストデータの取得 32 Map<String, String> item = (Map<String, String>) parent.getItemAtPosition(position); 33 String tagname = item.get("name"); 34 // インテント生成 35 Intent intent = new Intent(MainActivity.this, SelectActivity.class); 36 // 次画面に送るデータを格納 37 intent.putExtra("tagname", tagname); 38 // 次画面を起動 39 startActivity(intent); 40 } 41 } 42}

Sub Activity

Java

1public class SelectActivity extends AppCompatActivity { 2 3 @Override 4 protected void onCreate(Bundle savedInstanceState) { 5 super.onCreate(savedInstanceState); 6 setContentView(R.layout.activity_select); 7 8 // インテントオブジェクトを取得 9 Intent intent = getIntent(); 10 String tagname = intent.getStringExtra("tagname"); 11 12 //ListViewに表示する項目 13 ListView lvplace = findViewById(R.id.lvplace); 14 List<Map<String, String>> placeList = new ArrayList<>(); 15 //追加1 16 Map<String, String> place = new HashMap<>(); 17 place.put("name", "施設A"); 18 place.put("tag", "暇つぶし"); 19 placeList.add(place); 20 //追加2 21 place = new HashMap<>(); 22 place.put("name", "施設B"); 23 place.put("tag", "飲食店"); 24 placeList.add(place); 25 //追加3 26 place = new HashMap<>(); 27 place.put("name", "施設C"); 28 place.put("tag", "暇つぶし"); 29 placeList.add(place); 30 31 // 選択したタグと同じタグを集めたリスト生成 32 for (int i = 0; i< placeList.size(); i++) { 33 if (tagname != placeList.get(i).get("tag")) { 34 placeList.remove(i); 35 } 36 } 37 38 String[] from = {"name", "tag"}; 39 int[] to = {android.R.id.text1, android.R.id.text2}; 40 SimpleAdapter adapter = new SimpleAdapter(SelectActivity.this, placeList, android.R.layout.simple_list_item_2, from, to); 41 lvplace.setAdapter(adapter); 42 43 }

考察

MainActivityでの「tagname」が飲食店に固定されているのだと思いますが、解決策がわかりません。

よろしくお願いします。

気になる質問をクリップする

クリップした質問は、後からいつでもMYページで確認できます。

またクリップした質問に回答があった際、通知やメールを受け取ることができます。

バッドをするには、ログインかつ

こちらの条件を満たす必要があります。

hoshi-takanori

2021/05/21 15:44

Java では、文字列比較に == や != ではなく .equals メソッドを使う必要があります。 また、ループ中で remove(i) するとインデックス i がずれます。
tasojiro

2021/05/22 00:21 編集

返信ありがとうございます。 Javaへの基礎的な知識が足りてませんでした。 「.equalsメソッド」や「for文でリストを触るということ」についての理解が深まりました。 ありがとうございました。問題は無事解決いたしました。
guest

回答1

0

ベストアンサー

2点間違いがあります。

  1. 文字列の比較に != を使っている

Java での文字列の比較は String.equals() を使います。
== は「同じオブジェクトかどうか」をチェックするため、この場合は必ず一致しません。

  1. placeList のループ内で placeList から要素を削除している

このようなループ内でループ要素を変更すると正常に動作しません。
たとえば i = 0 で 0 番目の要素を削除すると、placeList には 施設B, 施設C が残ります。
次のループで i = 1 となりますが、施設B を飛ばして施設 C が参照されることになります。

投稿2021/05/21 17:12

yh1224

総合スコア653

バッドをするには、ログインかつ

こちらの条件を満たす必要があります。

tasojiro

2021/05/22 00:22

返信ありがとうございます。 Javaへの基礎的な知識が足りてませんでした。 「.equalsメソッド」や「for文でリストを触るということ」についての理解が深まりました。 ありがとうございました。問題は無事解決いたしました。
guest

あなたの回答

tips

太字

斜体

打ち消し線

見出し

引用テキストの挿入

コードの挿入

リンクの挿入

リストの挿入

番号リストの挿入

表の挿入

水平線の挿入

プレビュー

15分調べてもわからないことは
teratailで質問しよう!

ただいまの回答率
85.47%

質問をまとめることで
思考を整理して素早く解決

テンプレート機能で
簡単に質問をまとめる

質問する

関連した質問