Comparator.nullsLast does not avoid NullPointerException












4














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
















4














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














4












4








4







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

1,976422




1,976422










asked Nov 15 '18 at 7:15









rellocs woodrellocs wood

252214




252214












  • @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


















6














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



















2














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









    6














    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
















    6














    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














    6












    6








    6






    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

    1,976422




    1,976422








    • 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













    2














    You can possibly use :



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





    share|improve this answer


























      2














      You can possibly use :



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





      share|improve this answer
























        2












        2








        2






        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









        nullpointernullpointer

        44.1k1095182




        44.1k1095182






























            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.





            Some of your past answers have not been well-received, and you're in danger of being blocked from answering.


            Please pay close attention to the following guidance:


            • 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

            Guess what letter conforming each word

            Port of Spain

            Run scheduled task as local user group (not BUILTIN)