customised comparator in not working

Question

I am using Comparator to sort my ListView, but it doesn't work.

My code:

Collections.sort(orgi, new Comparator<Loc>() {

                @Override
                public int compare(Loc lhs, Loc rhs) {

                    if( lhs.getDist() < rhs.getDist() )
                         return 1;
                      else
                         return 0;
                }
            });

Can anyone suggest a solution?

Please post the code of `Loc` class, and you can't sort `ListView`, I guess you meant `ArrayList` or `List`. — M-Wajeeh, Nov 14 '13 at 11:13
BTW you must return a value < 0 as well because according to docs compare should return "an integer < 0 if lhs is less than rhs, 0 if they are equal, and > 0 if lhs is greater than rhs" — M-Wajeeh, Nov 14 '13 at 11:15
@M-WaJeEh its arraylist.ArrayList orgi; orgi = new ArrayList(); — gIrL, Nov 14 '13 at 11:36

score 3 · Accepted Answer · answered Nov 14 '13 at 11:17

3

Try this:

Collections.sort(orgi, new Comparator<Loc>() {

            @Override
            public int compare(Loc lhs, Loc rhs) {
                if(lhs.getDist() < rhs.getDist()){
                     return -1;
                 } else if(lhs.getDist() > rhs.getDist()){
                     return 1;
                 } else {
                     return 0;
                 }
            }
        });

answered Nov 14 '13 at 11:17

M-Wajeeh

17,204
10
66
103

Thanks a lot, its working now :).. i was returning the wrong values. am such a dumb ;P. thanks. – gIrL Nov 14 '13 at 11:38
@user2978175 I am glad that your problem was solved but simply thanking is not the way to go on Stackoverflow. You must accept one of the answers if it helped you. – M-Wajeeh Nov 14 '13 at 12:53

customised comparator in not working

1 Answers1