Comparator.nullsLast does not avoid NullPointerException



.everyoneloves__top-leaderboard:empty,.everyoneloves__mid-leaderboard:empty,.everyoneloves__bot-mid-leaderboard:empty height:90px;width:728px;box-sizing:border-box;








5















I want to sort a list of objects by one of nullable fields.



In order to avoid NullPointerexception I use Comparator.nullsLast. But the exception still occurs:



public class Test 

public static void main(String args)
List<Bean> l = new ArrayList<>();
for (int i=0;i<5;i++)
Bean b = new Bean("name_"+i,i);
l.add(b);

l.get(2).setVal(null);
System.out.println(l);
Collections.sort(l, Comparator.nullsLast(Comparator.comparing(Bean::getVal)));
System.out.println(l);


static class Bean
String name;
Integer val;
// omit getters & setters & constructor





How can I sort this kind of list?










share|improve this question
























  • @rellocswood Please, post your solution as an answer instead of editing the question.

    – Tomasz Linkowski
    Nov 15 '18 at 7:39

















5















I want to sort a list of objects by one of nullable fields.



In order to avoid NullPointerexception I use Comparator.nullsLast. But the exception still occurs:



public class Test 

public static void main(String args)
List<Bean> l = new ArrayList<>();
for (int i=0;i<5;i++)
Bean b = new Bean("name_"+i,i);
l.add(b);

l.get(2).setVal(null);
System.out.println(l);
Collections.sort(l, Comparator.nullsLast(Comparator.comparing(Bean::getVal)));
System.out.println(l);


static class Bean
String name;
Integer val;
// omit getters & setters & constructor





How can I sort this kind of list?










share|improve this question
























  • @rellocswood Please, post your solution as an answer instead of editing the question.

    – Tomasz Linkowski
    Nov 15 '18 at 7:39













5












5








5








I want to sort a list of objects by one of nullable fields.



In order to avoid NullPointerexception I use Comparator.nullsLast. But the exception still occurs:



public class Test 

public static void main(String args)
List<Bean> l = new ArrayList<>();
for (int i=0;i<5;i++)
Bean b = new Bean("name_"+i,i);
l.add(b);

l.get(2).setVal(null);
System.out.println(l);
Collections.sort(l, Comparator.nullsLast(Comparator.comparing(Bean::getVal)));
System.out.println(l);


static class Bean
String name;
Integer val;
// omit getters & setters & constructor





How can I sort this kind of list?










share|improve this question
















I want to sort a list of objects by one of nullable fields.



In order to avoid NullPointerexception I use Comparator.nullsLast. But the exception still occurs:



public class Test 

public static void main(String args)
List<Bean> l = new ArrayList<>();
for (int i=0;i<5;i++)
Bean b = new Bean("name_"+i,i);
l.add(b);

l.get(2).setVal(null);
System.out.println(l);
Collections.sort(l, Comparator.nullsLast(Comparator.comparing(Bean::getVal)));
System.out.println(l);


static class Bean
String name;
Integer val;
// omit getters & setters & constructor





How can I sort this kind of list?







java collections java-8 java-stream comparator






share|improve this question















share|improve this question













share|improve this question




share|improve this question








edited Nov 15 '18 at 10:37









ETO

2,7261629




2,7261629










asked Nov 15 '18 at 7:15









rellocs woodrellocs wood

274216




274216












  • @rellocswood Please, post your solution as an answer instead of editing the question.

    – Tomasz Linkowski
    Nov 15 '18 at 7:39

















  • @rellocswood Please, post your solution as an answer instead of editing the question.

    – Tomasz Linkowski
    Nov 15 '18 at 7:39
















@rellocswood Please, post your solution as an answer instead of editing the question.

– Tomasz Linkowski
Nov 15 '18 at 7:39





@rellocswood Please, post your solution as an answer instead of editing the question.

– Tomasz Linkowski
Nov 15 '18 at 7:39












2 Answers
2






active

oldest

votes


















5














You should use Comparator.nullsLast twice:



list.sort(nullsLast(comparing(Bean::getVal, nullsLast(naturalOrder()))));


  • First nullsLast will handle the cases when the Bean objects are null.

  • Second nullsLast will handle the cases when the return value of Bean::getVal is null.

In case you're sure there aren't any null values in your list then you can omit the first nullsLast (as noted by @Holger) :



list.sort(comparing(Bean::getVal, nullsLast(naturalOrder())));





share|improve this answer




















  • 1





    Or omit the outer nullsLast if the beans are never null, i.e. only the property values might be null.

    – Holger
    Nov 15 '18 at 8:40


















1














You can possibly use :



Collections.sort(l, Comparator.comparing(Bean::getVal,
Comparator.nullsLast(Comparator.naturalOrder())));





share|improve this answer























    Your Answer






    StackExchange.ifUsing("editor", function ()
    StackExchange.using("externalEditor", function ()
    StackExchange.using("snippets", function ()
    StackExchange.snippets.init();
    );
    );
    , "code-snippets");

    StackExchange.ready(function()
    var channelOptions =
    tags: "".split(" "),
    id: "1"
    ;
    initTagRenderer("".split(" "), "".split(" "), channelOptions);

    StackExchange.using("externalEditor", function()
    // Have to fire editor after snippets, if snippets enabled
    if (StackExchange.settings.snippets.snippetsEnabled)
    StackExchange.using("snippets", function()
    createEditor();
    );

    else
    createEditor();

    );

    function createEditor()
    StackExchange.prepareEditor(
    heartbeatType: 'answer',
    autoActivateHeartbeat: false,
    convertImagesToLinks: true,
    noModals: true,
    showLowRepImageUploadWarning: true,
    reputationToPostImages: 10,
    bindNavPrevention: true,
    postfix: "",
    imageUploader:
    brandingHtml: "Powered by u003ca class="icon-imgur-white" href="https://imgur.com/"u003eu003c/au003e",
    contentPolicyHtml: "User contributions licensed under u003ca href="https://creativecommons.org/licenses/by-sa/3.0/"u003ecc by-sa 3.0 with attribution requiredu003c/au003e u003ca href="https://stackoverflow.com/legal/content-policy"u003e(content policy)u003c/au003e",
    allowUrls: true
    ,
    onDemand: true,
    discardSelector: ".discard-answer"
    ,immediatelyShowMarkdownHelp:true
    );



    );













    draft saved

    draft discarded


















    StackExchange.ready(
    function ()
    StackExchange.openid.initPostLogin('.new-post-login', 'https%3a%2f%2fstackoverflow.com%2fquestions%2f53314193%2fcomparator-nullslast-does-not-avoid-nullpointerexception%23new-answer', 'question_page');

    );

    Post as a guest















    Required, but never shown

























    2 Answers
    2






    active

    oldest

    votes








    2 Answers
    2






    active

    oldest

    votes









    active

    oldest

    votes






    active

    oldest

    votes









    5














    You should use Comparator.nullsLast twice:



    list.sort(nullsLast(comparing(Bean::getVal, nullsLast(naturalOrder()))));


    • First nullsLast will handle the cases when the Bean objects are null.

    • Second nullsLast will handle the cases when the return value of Bean::getVal is null.

    In case you're sure there aren't any null values in your list then you can omit the first nullsLast (as noted by @Holger) :



    list.sort(comparing(Bean::getVal, nullsLast(naturalOrder())));





    share|improve this answer




















    • 1





      Or omit the outer nullsLast if the beans are never null, i.e. only the property values might be null.

      – Holger
      Nov 15 '18 at 8:40















    5














    You should use Comparator.nullsLast twice:



    list.sort(nullsLast(comparing(Bean::getVal, nullsLast(naturalOrder()))));


    • First nullsLast will handle the cases when the Bean objects are null.

    • Second nullsLast will handle the cases when the return value of Bean::getVal is null.

    In case you're sure there aren't any null values in your list then you can omit the first nullsLast (as noted by @Holger) :



    list.sort(comparing(Bean::getVal, nullsLast(naturalOrder())));





    share|improve this answer




















    • 1





      Or omit the outer nullsLast if the beans are never null, i.e. only the property values might be null.

      – Holger
      Nov 15 '18 at 8:40













    5












    5








    5







    You should use Comparator.nullsLast twice:



    list.sort(nullsLast(comparing(Bean::getVal, nullsLast(naturalOrder()))));


    • First nullsLast will handle the cases when the Bean objects are null.

    • Second nullsLast will handle the cases when the return value of Bean::getVal is null.

    In case you're sure there aren't any null values in your list then you can omit the first nullsLast (as noted by @Holger) :



    list.sort(comparing(Bean::getVal, nullsLast(naturalOrder())));





    share|improve this answer















    You should use Comparator.nullsLast twice:



    list.sort(nullsLast(comparing(Bean::getVal, nullsLast(naturalOrder()))));


    • First nullsLast will handle the cases when the Bean objects are null.

    • Second nullsLast will handle the cases when the return value of Bean::getVal is null.

    In case you're sure there aren't any null values in your list then you can omit the first nullsLast (as noted by @Holger) :



    list.sort(comparing(Bean::getVal, nullsLast(naturalOrder())));






    share|improve this answer














    share|improve this answer



    share|improve this answer








    edited Nov 15 '18 at 16:52

























    answered Nov 15 '18 at 8:31









    ETOETO

    2,7261629




    2,7261629







    • 1





      Or omit the outer nullsLast if the beans are never null, i.e. only the property values might be null.

      – Holger
      Nov 15 '18 at 8:40












    • 1





      Or omit the outer nullsLast if the beans are never null, i.e. only the property values might be null.

      – Holger
      Nov 15 '18 at 8:40







    1




    1





    Or omit the outer nullsLast if the beans are never null, i.e. only the property values might be null.

    – Holger
    Nov 15 '18 at 8:40





    Or omit the outer nullsLast if the beans are never null, i.e. only the property values might be null.

    – Holger
    Nov 15 '18 at 8:40













    1














    You can possibly use :



    Collections.sort(l, Comparator.comparing(Bean::getVal,
    Comparator.nullsLast(Comparator.naturalOrder())));





    share|improve this answer



























      1














      You can possibly use :



      Collections.sort(l, Comparator.comparing(Bean::getVal,
      Comparator.nullsLast(Comparator.naturalOrder())));





      share|improve this answer

























        1












        1








        1







        You can possibly use :



        Collections.sort(l, Comparator.comparing(Bean::getVal,
        Comparator.nullsLast(Comparator.naturalOrder())));





        share|improve this answer













        You can possibly use :



        Collections.sort(l, Comparator.comparing(Bean::getVal,
        Comparator.nullsLast(Comparator.naturalOrder())));






        share|improve this answer












        share|improve this answer



        share|improve this answer










        answered Nov 15 '18 at 7:28









        NamanNaman

        45.7k11102204




        45.7k11102204



























            draft saved

            draft discarded
















































            Thanks for contributing an answer to Stack Overflow!


            • Please be sure to answer the question. Provide details and share your research!

            But avoid


            • Asking for help, clarification, or responding to other answers.

            • Making statements based on opinion; back them up with references or personal experience.

            To learn more, see our tips on writing great answers.




            draft saved


            draft discarded














            StackExchange.ready(
            function ()
            StackExchange.openid.initPostLogin('.new-post-login', 'https%3a%2f%2fstackoverflow.com%2fquestions%2f53314193%2fcomparator-nullslast-does-not-avoid-nullpointerexception%23new-answer', 'question_page');

            );

            Post as a guest















            Required, but never shown





















































            Required, but never shown














            Required, but never shown












            Required, but never shown







            Required, but never shown

































            Required, but never shown














            Required, but never shown












            Required, but never shown







            Required, but never shown







            Popular posts from this blog

            Use pre created SQLite database for Android project in kotlin

            Darth Vader #20

            Ondo