sorting integers in a arraylist of objects

Question

I'm starting with Java, and my problem is that I have an Arraylist of objects("Articulos", means Articles), and this object has properties (like name, price, etc...). I want to sort the Arraylist by the price of each object. Here's what I've tried.

First, I fill the Arraylist manually:

public static void Introducir_Articulo(){
    Articulo a=new Articulo();

    System.out.println("Codigo del articulo: "+(art.size()+1));
    if(a.codArt==0){
        a.codArt++;
    } else {
        a.codArt+=art.size();
    }
    System.out.println("Nombre del articulo: ");
    a.NombreArt=sc.next();

    System.out.println("Precio del articulo: ");
    a.precio=sc.nextInt();

    System.out.println("IVA del articulo: ");
    a.iva=sc.nextInt();

    art.add(a);
}

Later, I tried this

//copying the arraylist, so I don't have to change the original
artOrdenado=new ArrayList<Articulo>(art);
System.out.println(artOrdenado);

Collections.sort(artOrdenado, new Comparator<Articulo>(){
    public int compare(Articulo uno, Articulo otro){
       return uno.getPrecio().compareTo(otro.getPrecio());
    }
});

but it throws an exception which says "int can not be deferenced".

Fahim Parkar · Accepted Answer · 2013-06-05 08:11:59Z

4

You need to do the comparision in your comparator like this:

Collections.sort(artOrdenado, new Comparator<Articulo>(){
    public int compare(Articulo uno, Articulo otro){
        return (uno.getPrecio() - otro.getPrecio());
    }

});

Or by using Integer.compare();

edited Jun 5, 2013 at 8:11

Fahim Parkar

31.8k46 gold badges169 silver badges285 bronze badges

answered Jun 5, 2013 at 8:09

CloudyMarble

37.7k70 gold badges103 silver badges133 bronze badges

Sign up to request clarification or add additional context in comments.

Comments

Brian Agnew · Accepted Answer · 2013-06-05 08:08:47Z

0

I'm guessing that getPrecio() returns int (a primitive). You can't call methods on primitives (they're not objects). You can call methods on objects, however, and if getPrecio() returns an Integer (the object counterpart to an int) then you should have more success (see here for more info on this aspect of Java, called boxing)

Alternatively you can compare the ints directly in your compare() method (e.g. if price1 < price2, return -1 etc.)

answered Jun 5, 2013 at 8:08

Brian Agnew

273k38 gold badges342 silver badges443 bronze badges

Comments

Sanjaya Liyanage · Accepted Answer · 2013-06-05 08:11:36Z

0

try using

return uno.getPrecio-otro.getPrecio();

I hope getPrecio returns the price value

answered Jun 5, 2013 at 8:11

Sanjaya Liyanage

4,75610 gold badges38 silver badges51 bronze badges

Comments

Simulant · Accepted Answer · 2013-06-05 08:12:57Z

0

What is the Type of the returning Value of getPrecio()? Have you defined the behavior of compareTo() in this Class? If you Type is int use the normal Operators >, < and == to compare the values and return the signum you want. Or simply use:

Collections.sort(artOrdenado, new Comparator<Articulo>(){
    public int compare(Articulo uno, Articulo otro){
        return (uno.getPrecio() - otro.getPrecio());
    }
});

answered Jun 5, 2013 at 8:12

Simulant

20.2k10 gold badges68 silver badges106 bronze badges

Comments

Vareks · Accepted Answer · 2013-06-05 08:14:54Z

0

Use this method to compare instead

public int compare(Articulo uno, Articulo otro)
{
    int comparation= (uno.getPrecio()> otro.getPrecio()) ? 1 : 0;
    if(comparation== 0)
    {
       comparation= (uno.getPrecio()== otro.getPrecio()) ? 0 : -1;
    }

return comparation;
}

answered Jun 5, 2013 at 8:14

Vareks

92 bronze badges

Collectives™ on Stack Overflow

sorting integers in a arraylist of objects

5 Answers 5

Comments

Comments

Comments

Comments

Comments

Your Answer

Linked

Hot Network Questions

Collectives™ on Stack Overflow

5 Answers 5

Comments

Comments

Comments

Comments

Comments

Your Answer

Sign up or log in

Post as a guest

Linked

Related