SQLiteに登録された情報をArrayAdapter(ArrayList)を用いて昇順にソートして表示

Question

データベース（Product）に保存した内容（食品名、量）をArrayAdapterを用いて表示させているのですが、
表示される順番が登録順になっているので、
これを昇順（食品名、量の二つとも）に表示させようとしています。

公式サイトよりArrayAdapter.sortを用いて試しているのですが、

return int.compare(i1.getfoodName(), i2.getfoodName());　　// .に赤線が出ます

という状況です。
ご教授お願い致します。

public class FoodActivity extends AppCompatActivity {
    private ArrayList<Product> products;

    public void onResume(){
        super.onResume();

         //データベースを読み込む
　　　　　products = new ArrayList<Product>();
        SQLiteDatabase db = helper.getReadableDatabase();

        String[] columns = {"id", "foodName", "foodgram"};
        Cursor cursor = db.query("Products", columns, null, null, null, null, null);

        while(cursor.moveToNext()) {
            int id = cursor.getInt(0) ;
            
            String foodName = cursor.getString(1);
            double foodgram = cursor.getDouble(2);

            products.add(
                    new Product(id, foodName, foodgram)
            );
        }

         String[] items = new String[products.size()];

         for(int i = 0; i < products.size(); i++) {
            Product product = products.get(i);
            items[i] = product.getfoodName() + "：" + product.getfoodgram();
         }

        ArrayAdapter<String> adapter = new ArrayAdapter<String>
                (this, android.R.layout.simple_list_item_single_choice, items);

        listView.setAdapter(adapter);
        listView.setItemChecked(0, true);

        //getfoodName(),getfoodgram()をソートさせたい
        //getfoodName()はString型、getfoodgram()はdouble型
        adapter.sort(new Comparator<Product>() {
            @Override
            public int compare(Product i1, Product i2) {
                return int.compare(i1.getfoodName(), i2.getfoodName());
            }
            public int compare(Product i1, Product i2) {
                return int.compare(i1.getfoodgram(), i2.getfoodgram());
            }
        });

        db.close();
    }
}

Comments

[dodox86]

とりあえず、ArrayAdapter<String> adapter は
ArrayAdapter<Product> adapter; でないとダメなのではないでしょうか。そうした上でのadapter.sort(new Comparator<Product>() では。

[karin10]

ArrayAdapter<Product> adapter;に変えると (this, android.R.layout.simple_list_item_single_choice, items);に赤線が入ってしまいます、、、

[dodox86]

エラーになるのはわかるのですが、エラーを取るために修正するのではなく、やりたいことを実現するためにそのようにコードを修正するのです。今のままだと単純に、
adapter.sort(new Comparator<Product>() {
@Override
public int compare(Product i1, Product i2) {
return 0;
}
});
としてもエラーになります。

[dodox86]

今のままのコードですとArrayAdapterに結びつけているのはStringの配列なので、ソートの対象としたいgramの値が取れないと思います。foodのNameだけならソート対象にできますが。

[karin10]

浅い考えですみませんでした。
foodのNameのソート方法についてご教授お願い致します。
foodgramのソートはそれを元に作成してみます。

Answer 1

ソートさせる内容がSQLiteに登録されているものだったので、
ソートさせる機能のコードを追加で書くのではなく、
SQLiteの登録情報を読み込むときに "order_by"を用いてデータベース情報をソートさせてから表示させるようにしました。

        products = new ArrayList<Product>();
        SQLiteDatabase db = helper.getReadableDatabase();

        String order_by = "foodName ASC";　//コード追加
        String[] columns = {"id", "foodName", "foodgram"};
        Cursor cursor = db.query("Products", columns, null, null, null, null, order_by);　//order_byの有効化

Answer 2

質問への追記・修正の依頼欄での[2021/12/08 22:05]のコメントより：

foodのNameのソート方法についてご教授お願い致します。
foodgramのソートはそれを元に作成してみます。

コメントで一部回答したように、質問者さんの現状のコードではArrayAdapterの生成時にArrayAdapter<String>...としているので、ArrayAdapter#sortでもStringの型同士でしか比較できません。その限りでコードを書くのであれば、String#compareToを使います。

adapter.sort(new Comparator<String>() {
    @Override
    public int compare(String s1, String s2) {
        return s1.compareTo(s2);
    }
});

foodNameとfoodGramの双方を考慮した昇順のソートとなると、まずfoodNameを比較し、同じであれば更にfoodGramを比較するような合せ技になるかと思います。参考として以下のサンプルコードを載せておきます。Androidはとりあえず関係ないので、ターミナル上で実行できます。

// Main4.java
import java.util.ArrayList;
import java.util.Comparator;

public class Main4 {
    public static void main(String[]args) {

        ArrayList<Product> prodList = new ArrayList<>();
        prodList.add(new Product("FoodC", 10.0));
        prodList.add(new Product("FoodA", 1.0));
        prodList.add(new Product("FoodB", 3.0));
        prodList.add(new Product("FoodB", 2.0));
        prodList.add(new Product("FoodB", 1.0));

        System.out.println("*** BEFORE:");
        for (Product prod: prodList) {
            System.out.println(prod);
        }

        // 昇順にソート
        prodList.sort(new Comparator<Product>() {
            @Override
            public int compare(Product p1, Product p2) {
                int result = p1.getFoodName().compareTo(p2.getFoodName());
                if (result == 0) {
                    // FoodNameが同じであれば更にFoodGramを比較
                    if (p1.getFoodGram() < p2.getFoodGram()) {
                        result = -1;
                    } else if (p1.getFoodGram() > p2.getFoodGram()) {
                        result = 1;
                    }
                }

                return result;
            }
        });

        System.out.println("*** AFTER:");
        for (Product prod: prodList) {
            System.out.println(prod);
        }
    }
}

class Product {
    private String foodName;
    private double foodGram;

    public Product(String foodName, double foodGram) {
        this.foodName = foodName;
        this.foodGram = foodGram;
    }

    public String getFoodName() {
        return this.foodName;
    }

    public double getFoodGram() {
        return this.foodGram;
    }

    @Override
    public String toString() {
        return foodName + ":" + foodGram;
    }
}

foodNameが違っていればfoodGramを比較する必要が無いのは、foodNameがいわば上位桁に相当するので、そこで大小関係が決まってしまえばfoodGramを比較する必要はないからです。

これを実行すると以下の出力結果となります。昇順にソートされているのが分かるかと思います。

$ java Main4
*** BEFORE:
FoodC:10.0
FoodA:1.0
FoodB:3.0
FoodB:2.0
FoodB:1.0
*** AFTER:
FoodA:1.0
FoodB:1.0
FoodB:2.0
FoodB:3.0
FoodC:10.0

Comments

[karin10]

無事ソートされて表示されました！
ただ、SQLiteに追加で登録した情報はソートされずに表示されてしまったり、ソート後に表示されている登録情報を選択すると別の登録情報が表示されたりなど何点かバグが発生しました。

しかし、dodox86様のコードのおかげで登録情報が表示されている画面のコードではなく、SQLiteでorder_byを有効にすることで登録情報をソートさせて表示できるのでは気付くことが出来ました。
質問で安直にSQLiteに関するコードを省いたこと申し訳ありません。（編集済み）

foodgramにも対応したコードも書いていただりして頂いたにもかかわらず、申し訳ありません。

[dodox86]

> ただ、SQLiteに追加で登録した情報はソートされずに表示されてしまったり、

そりゃ当たり前です。質問自体、前提とやりたいことがもはや変わっています。ちょっとマナー違反ですね。（こちらの質問に限らず、よくあることですが）

[karin10]

上手く説明できていなかったり、コードを省いてしまったりしていて申し訳ありませんでした。

[dodox86]

当初、ArrayAdapterを使っているのは、画面表示後に「FoodName」「FoodGram」「FoodName + FoodGram」のいずれかで動的にソートさせたいのだと思いました。それゆえの回答です。結局解決できたのかできていないのか分かりませんが、私の回答とは違う方法で解決したのであれば自己回答、解決で構わないと思いますので、回答を閉じてください。