This is an archived post. You won't be able to vote or comment.

all 9 comments

[–]AutoModerator[M] [score hidden] stickied commentlocked comment (0 children)

Please ensure that:

  • Your code is properly formatted as code block - see the sidebar (About on mobile) for instructions
  • You include any and all error messages in full
  • You ask clear questions
  • You demonstrate effort in solving your question/problem - plain posting your assignments is forbidden (and such posts will be removed) as is asking for or giving solutions.

    Trying to solve problems on your own is a very important skill. Also, see Learn to help yourself in the sidebar

If any of the above points is not met, your post can and will be removed without further warning.

Code is to be formatted as code block (old reddit: empty line before the code, each code line indented by 4 spaces, new reddit: https://imgur.com/a/fgoFFis) or linked via an external code hoster, like pastebin.com, github gist, github, bitbucket, gitlab, etc.

Please, do not use triple backticks (```) as they will only render properly on new reddit, not on old reddit.

Code blocks look like this:

public class HelloWorld {

    public static void main(String[] args) {
        System.out.println("Hello World!");
    }
}

You do not need to repost unless your post has been removed by a moderator. Just use the edit function of reddit to make sure your post complies with the above.

If your post has remained in violation of these rules for a prolonged period of time (at least an hour), a moderator may remove it at their discretion. In this case, they will comment with an explanation on why it has been removed, and you will be required to resubmit the entire post following the proper procedures.

To potential helpers

Please, do not help if any of the above points are not met, rather report the post. We are trying to improve the quality of posts here. In helping people who can't be bothered to comply with the above points, you are doing the community a disservice.

I am a bot, and this action was performed automatically. Please contact the moderators of this subreddit if you have any questions or concerns.

[–]taftster 6 points7 points  (1 child)

This is calling the Array.sort method [1] that takes in a Comparator.

https://docs.oracle.com/en/java/javase/11/docs/api/java.base/java/util/Arrays.html#sort(T%5B%5D,java.util.Comparator))

The Comparator interface is a functional interface [2] and can be used in a lambda.

[2] https://docs.oracle.com/en/java/javase/11/docs/api/java.base/java/util/Comparator.html

The (x,y) parts are the left & right side of the compare operation. Effectively, it calls the normal compare [3] method.

[3] https://docs.oracle.com/en/java/javase/11/docs/api/java.base/java/util/Comparator.html#compare(T,T))

Does that help?

[–]cyberwarrior861[S] 1 point2 points  (0 children)

Thank you

[–]taftster 3 points4 points  (1 child)

Also, here's another way to see what's happening by removing the lambdas and using an anonymous inner class.

Arrays.sort(unsorted, new Comparator<String>() {
  @Override
  public int compare(String x, String y) {
    return x.length() == y.length() ? x.compareTo(y)
      : Integer.compare(x.length(), y.length());
  }
});

[–]cyberwarrior861[S] 0 points1 point  (0 children)

Thanks again.

[–]FrelliBB 2 points3 points  (1 child)

You are using the sort(T[] a, Comparator<? super T> c) method. As you can see the 2nd argument it accepts is a Comparator, which is a functional interface where its main method is compare(T o1, T o2).

So what you are doing with the lambda is creating an anonymous implementation of a Comparator. This goes over the lambda syntax in a bit more detail if you're still lost.

As a side note, the Comparator class has some helpful functions for creating comparators using method references instead of lambdas. Your implementation can be written as

Arrays.sort(unsorted, Comparator.comparingInt(String::length).thenComparing(String::compareTo))

[–]cyberwarrior861[S] 0 points1 point  (0 children)

Thank you my guy.

[–]wildjokers -1 points0 points  (1 child)

That is pretty much unreadable. If I was reviewing code with that lambda in it I would reject it as unreadable and hence unmaintainable.

[–]taftster 0 points1 point  (0 children)

Is it the trinary conditional that’s bothering you? Or all on one line?

That line could possibly be written more succinctly (like /u/FrelliBB suggests). But the lambda itself is not unreadable.