ソートを文字列に含まれるカンマ区切りの数字で行いたい

Question

aaa, , ,59, ,
aaa, , ,17, ,
aaa, , ,02, ,
↓
aaa, , ,02, ,
aaa, , ,17, ,
aaa, , ,59, ,

上記の様に途中にあるカンマ区切りの数字でソートしたいです。

Collections.sortで辞書的にソートすることはできました。

カンマが一つだけなら

class MyComparator implements Comparator{
public int compare(Object obj1,Object obj2){
int num1;
int num2;

String str1; = obj1.toString();
String str1; = obj1.toString();

num1 = Integer.parseInt(str1.substring(str1.indexOf(’,’)+1).trim());

num2 = Integer.parseInt(str2).substring(str2.indexOf(’,’)+1).trim());

return num1 - num2
}
}

Collections.sort(リスト名,new MyComparator());

上記で出来たのですが、カンマが複数ある場合に応用できませんでした。

追記
上記のコードではint型の変数でソートしていますが、実際の環境ではカンマ区切りの文字列の中には1A,1Wなどのint型に変換できないものも含まれます。
string型でソートしたいです。

Comments

[m.ts10806]

えーっと・・
職場のコードをそのままでないとはいえ公開されている場に出していいのかという疑問は大いにあるのですが、
いずれにしても「コピペで確認できるコード」が大原則なので、コピペでコードが提示できないのであれば、できるような環境において質問されたほうが良いかと思います。
でないと、回答する側が考慮しなければならない点が多すぎて、的確な回答が得られない可能性もあります（インデントあるなしで質問からバックしてしまう回答者もいるので）

[LouiS0616]

なぜ職場で直接質問できないのか、その背景が分かりません。

補足：
当初の質問には、『職場の環境上コードをコピペできないのでiPhoneの手打ちでやっています。コード全文を載せられなくてすいません。』という文言がありました。https://teratail.com/questions/history-questions/200831

[tomopi3]

職場のpcがネットに繋がっておらず、コピペは不可です。

職場で質問できないため、こちらで質問させて頂きました。(上司のスケジュール等)

ご指摘ありがとうございました。

[swordone]

｢int型に変換できないもの｣の位置は各行で共通ですか？

[tomopi3]

共通です

Answer 1

カンマ区切りではなくsubstringで文字列を切り出してソートしました。
文字列でのソートなので数字以外のアルファベット混じりでもいけました。

Answer 2

num1 = Integer.parseInt(str1.substring(str1.indexOf(’,’)+1).trim());

この書き方だと、substringの引数が1つしかありませんから、「カンマがある最初の位置の次の文字」から最後までの文字列を切り取ることになります。カンマとカンマの間の文字列を取るためには、「次のカンマの位置」を調べて、引数2個のsubstringを使う必要があります。それを実現するためには1行では厳しいです。

int s1 = str1.indexOf(',') + 1;
int e1 = str1.indexOf(',', s1);
String a = str1.substring(s1, e1);

もっとも、substringを使うよりsplitで配列にしてしまったほうが処理は圧倒的に楽だと思います。

Comparator<String> c = new Comparator<>() {
    @Override
    public int compare(String o1, String o2) {
        String[] a1 = o1.split(",");
        String[] a2 = o2.split(",");
        int c = 0;
        for (int i = 0; i < a1.length; i++) {
            try {
                int num1 = Integer.parseInt(a1[i]);
                int num2 = Integer.parseInt(a2[i]);
                c = Integer.compare(num1, num2);
            } catch (NumberFormatException e) {
                c = a1[i].compareTo(a2[i]);
            }
            if (c != 0) return c;
        }
        return 0;
    }
}

※上記コードは率直に書いただけなので、非常に無駄が多いです。

Answer 3

カンマが複数あると言っても、１つ目のカンマにマッチさせて、２つ目のカンマにマッチさせて、３つ目のカンマの手前までを取り出せば良いのでしょ。

ターゲット文字列に繰り返しマッチ の記事が参考になると思います。

Comments

[tomopi3]

カンマ区切りの文字列を取り出す事自体はsubstringでできました。
取り出した文字列でのソートができません。

[coco_bauer]

質問に書かれた compare関数は、２つの文字列で最初のカンマの後を整数に変えて比較してますよね。このcompare関数を使う事で１つ目のカンマの後の数字(文字列)でソートできている。
同様に、「１つ目のカンマにマッチさせて、２つ目のカンマにマッチさせて、３つ目のカンマの手前までを取り出」して、それを整数に変えて比較するようにしたcompare関数を定義して、そのcompare関数を使たソートをすれば、希望するような動作が実現できると思うのですが。