Identity for BinaryOperator

The name of the pictureThe name of the pictureThe name of the pictureClash Royale CLAN TAG#URR8PPP











up vote
7
down vote

favorite
2












I see in Java8 in UnaryOperator Interface following piece of code which does nothing on parameter and returns same value.



static <T> UnaryOperator<T> identity() 
 return t -> t;



Is there anything for BinaryOperator which accepts two parameters of samekind and returns one value



static <T> BinaryOperator<T> identity() 
 return (t,t) -> t;



why I am asking this question is for below requirement,



List<String> list = Arrays.asList("Abcd","Abcd");
Map<String,Integer> map = list.stream().collect(Collectors.toMap(str->str, 
str->(Integer)str.length(),(t1,t2)->t1));
System.out.println(map.size());


in above code I don't want to do anything for two values of same key, I just wanted return one value, because in my case for sure values will be same.
As am not using t2 value Sonar throwing error, So I am finding out is there any thing like UnaryOperator.identity() for BinaryOpertor also in java8






java lambda java-8 functional-programming







share|improve this question



















  • 5




    An identity, by definition, returns its input without any modification. If the input is two values, its output must also be two values. What you want is not an identity, but some sort of distinct or group operation before processing the values. (I don't speak modern Java, but in C# that would literally be .Distinct and .GroupBy.)
    – Jeroen Mostert
    yesterday







  • 2




    The problem is not in your code, but in the Sonar alert instead. Just turn it off.
    – Federico Peralta Schaffner
    yesterday






  • 1




    Some functional languages only have unary functions. In these languages, a function accepting a tuple is equivalent to a binary function in Java. Identity for such a function is easy when you think of it in this way: if a tuple is input, it must also be the output.
    – user633183
    yesterday















up vote
7
down vote

favorite
2












I see in Java8 in UnaryOperator Interface following piece of code which does nothing on parameter and returns same value.



static <T> UnaryOperator<T> identity() 
 return t -> t;



Is there anything for BinaryOperator which accepts two parameters of samekind and returns one value



static <T> BinaryOperator<T> identity() 
 return (t,t) -> t;



why I am asking this question is for below requirement,



List<String> list = Arrays.asList("Abcd","Abcd");
Map<String,Integer> map = list.stream().collect(Collectors.toMap(str->str, 
str->(Integer)str.length(),(t1,t2)->t1));
System.out.println(map.size());


in above code I don't want to do anything for two values of same key, I just wanted return one value, because in my case for sure values will be same.
As am not using t2 value Sonar throwing error, So I am finding out is there any thing like UnaryOperator.identity() for BinaryOpertor also in java8






java lambda java-8 functional-programming







share|improve this question



















  • 5




    An identity, by definition, returns its input without any modification. If the input is two values, its output must also be two values. What you want is not an identity, but some sort of distinct or group operation before processing the values. (I don't speak modern Java, but in C# that would literally be .Distinct and .GroupBy.)
    – Jeroen Mostert
    yesterday







  • 2




    The problem is not in your code, but in the Sonar alert instead. Just turn it off.
    – Federico Peralta Schaffner
    yesterday






  • 1




    Some functional languages only have unary functions. In these languages, a function accepting a tuple is equivalent to a binary function in Java. Identity for such a function is easy when you think of it in this way: if a tuple is input, it must also be the output.
    – user633183
    yesterday













up vote
7
down vote

favorite
2









up vote
7
down vote

favorite
2






2





I see in Java8 in UnaryOperator Interface following piece of code which does nothing on parameter and returns same value.



static <T> UnaryOperator<T> identity() 
 return t -> t;



Is there anything for BinaryOperator which accepts two parameters of samekind and returns one value



static <T> BinaryOperator<T> identity() 
 return (t,t) -> t;



why I am asking this question is for below requirement,



List<String> list = Arrays.asList("Abcd","Abcd");
Map<String,Integer> map = list.stream().collect(Collectors.toMap(str->str, 
str->(Integer)str.length(),(t1,t2)->t1));
System.out.println(map.size());


in above code I don't want to do anything for two values of same key, I just wanted return one value, because in my case for sure values will be same.
As am not using t2 value Sonar throwing error, So I am finding out is there any thing like UnaryOperator.identity() for BinaryOpertor also in java8






java lambda java-8 functional-programming







share|improve this question















I see in Java8 in UnaryOperator Interface following piece of code which does nothing on parameter and returns same value.



static <T> UnaryOperator<T> identity() 
 return t -> t;



Is there anything for BinaryOperator which accepts two parameters of samekind and returns one value



static <T> BinaryOperator<T> identity() 
 return (t,t) -> t;



why I am asking this question is for below requirement,



List<String> list = Arrays.asList("Abcd","Abcd");
Map<String,Integer> map = list.stream().collect(Collectors.toMap(str->str, 
str->(Integer)str.length(),(t1,t2)->t1));
System.out.println(map.size());


in above code I don't want to do anything for two values of same key, I just wanted return one value, because in my case for sure values will be same.
As am not using t2 value Sonar throwing error, So I am finding out is there any thing like UnaryOperator.identity() for BinaryOpertor also in java8






java lambda java-8 functional-programming




java lambda java-8 functional-programming






share|improve this question















share|improve this question













share|improve this question




share|improve this question








edited 23 hours ago









nullpointer

30.4k1054127




30.4k1054127










asked yesterday









Karunakar Reddy L

666




666







  • 5




    An identity, by definition, returns its input without any modification. If the input is two values, its output must also be two values. What you want is not an identity, but some sort of distinct or group operation before processing the values. (I don't speak modern Java, but in C# that would literally be .Distinct and .GroupBy.)
    – Jeroen Mostert
    yesterday







  • 2




    The problem is not in your code, but in the Sonar alert instead. Just turn it off.
    – Federico Peralta Schaffner
    yesterday






  • 1




    Some functional languages only have unary functions. In these languages, a function accepting a tuple is equivalent to a binary function in Java. Identity for such a function is easy when you think of it in this way: if a tuple is input, it must also be the output.
    – user633183
    yesterday













  • 5




    An identity, by definition, returns its input without any modification. If the input is two values, its output must also be two values. What you want is not an identity, but some sort of distinct or group operation before processing the values. (I don't speak modern Java, but in C# that would literally be .Distinct and .GroupBy.)
    – Jeroen Mostert
    yesterday







  • 2




    The problem is not in your code, but in the Sonar alert instead. Just turn it off.
    – Federico Peralta Schaffner
    yesterday






  • 1




    Some functional languages only have unary functions. In these languages, a function accepting a tuple is equivalent to a binary function in Java. Identity for such a function is easy when you think of it in this way: if a tuple is input, it must also be the output.
    – user633183
    yesterday








5




5




An identity, by definition, returns its input without any modification. If the input is two values, its output must also be two values. What you want is not an identity, but some sort of distinct or group operation before processing the values. (I don't speak modern Java, but in C# that would literally be .Distinct and .GroupBy.)
– Jeroen Mostert
yesterday





An identity, by definition, returns its input without any modification. If the input is two values, its output must also be two values. What you want is not an identity, but some sort of distinct or group operation before processing the values. (I don't speak modern Java, but in C# that would literally be .Distinct and .GroupBy.)
– Jeroen Mostert
yesterday





2




2




The problem is not in your code, but in the Sonar alert instead. Just turn it off.
– Federico Peralta Schaffner
yesterday




The problem is not in your code, but in the Sonar alert instead. Just turn it off.
– Federico Peralta Schaffner
yesterday




1




1




Some functional languages only have unary functions. In these languages, a function accepting a tuple is equivalent to a binary function in Java. Identity for such a function is easy when you think of it in this way: if a tuple is input, it must also be the output.
– user633183
yesterday





Some functional languages only have unary functions. In these languages, a function accepting a tuple is equivalent to a binary function in Java. Identity for such a function is easy when you think of it in this way: if a tuple is input, it must also be the output.
– user633183
yesterday













3 Answers
3






active

oldest

votes

















up vote
6
down vote













Your question doesn't really make sense. If you were to paste your proposed BinaryOperator.identity method into an IDE, you would immediately see that it would complain that the identifier t is declared twice.



To fix this, we need a different identifier for each parameter:



return (t, u) -> t;


Now we can clearly see that this is not an identity function. It's a method which takes two arguments and returns the first one. Therefore the best name for this would be something like getFirst.



To answer your question about whether there's anything like this in the JDK: no. Using an identity function is a common use case, so defining a method for that is useful. Arbitrarily returning the first argument of two is not a common use case, and it's not useful to have a method to do that.






share|improve this answer




















  • Well, I agree to what you've stated here, except for the last part: Arbitrarily returning the first argument of two is not a common use case, and it's not useful to have a method to do that. In fact it is quite useful, as OP is showing here. Another use case is when implementing collectors and you need to provide a combiner function that you know will never be used (because i.e. the stream is sequential). Besides, BinaryOperator does provide maxBy(Comparator) static method (and also minBy), so it's valid to wonder why there's no i.e. first or second methods...
    – Federico Peralta Schaffner
    22 hours ago











  • @FedericoPeraltaSchaffner "common" is kind of subjective. I've never needed that, and I work with the JDK on a daily basis. By contrast, I've needed Function.identity probably 100s of times. The JDK is supposed to be a collection of broadly useful tools. It's not designed to contain absolutely everything a developer might ever need. If you add a getFirst, people will wonder where getSecond is. If you add both of those for BinaryOperator, why not add similar ones for the other 42 functional interfaces? Boom - you've now got ~100 basically useless methods cluttering the API.
    – Michael
    22 hours ago











  • I absolutely agree with you, I was playing the role of the devil's advocate a little bit... The problem with the question is not that there doesn't exist a method to choose the first or second element, but that Sonar is throwing an alert, when it shouldn't...
    – Federico Peralta Schaffner
    22 hours ago

















up vote
5
down vote













Your code seemingly can be improved as 



List<String> list = Arrays.asList("Abcd", "Abcd");
Map<String, Integer> map = list.stream()
 .collect(Collectors.toMap(Function.identity(), String::length, (a, b) -> a));
System.out.println(map.size());


Or possibly for your use case I don't want to do anything for two values of same key, I just wanted return one value, you may just choose to randomly return any value in using an implementation as following:



private static <T> BinaryOperator<T> any() 
 return Math.random() < 0.5 ? ((x, y) -> x) : ((x, y) -> y);



and then in your code use it as



Map<String, Integer> map = list.stream()
 .collect(Collectors.toMap(Function.identity(), String::length, any()));


Thanks to the suggestions from Holger, Eugene, and Federico, there are other efficient implementations of the any method that can actually involve using :



private static <T> BinaryOperator<T> any() 
 // suggested by Holger
 return ThreadLocalRandom.current().nextBoolean() ? ((x, y) -> x) : ((x, y) -> y);
 // suggested by Eugene
 long nt = System.nanoTime(); 
 ((nt >>> 32) ^ nt) > 0 ? ((x, y) -> x) : ((x, y) -> y);






share|improve this answer






















  • that btw is a very interesting merger :)
    – Eugene
    yesterday






  • 1




    I guess you are right, it just looks weird to me.
    – Eugene
    yesterday






  • 3




    You know that your random implementation won't pick a random argument each time, but will choose one implementation when called, and then will return the same argument all times. btw I didn't downvote
    – Federico Peralta Schaffner
    yesterday







  • 1




    I remember I asked something about that here and Stuart Marks kindly answered... :D
    – Federico Peralta Schaffner
    23 hours ago






  • 2




    I’d use ThreadLocalRandom.current().nextBoolean() instead of Math.random() < 0.5. An alternative would be (a,b) -> assert Objects.equals(a, b); return a;
    – Holger
    20 hours ago

















up vote
4
down vote













T means they have the same types, not the same values, that is not an identity per-se.



It just means that BinaryOperator will be used for the same types, but providing an identity for different values... this somehow sounds like foldLeft or foldRight or foldLeftIdentity/foldRightIdentity, which java does not have.






share|improve this answer






















  • what's your thought on the implementation of any for returning any value as in the merger?
    – nullpointer
    yesterday






  • 1




    @nullpointer :) that is all I can say. probably Math.random is too expensive here, I would go for something cheaper like long nt = System.nanoTime(); ((nt >>> 32) ^ nt) > 0 ? ... : ... but I've seen this somewhere else already :) (of course the cheapest and fastest random would have to be taken here)
    – Eugene
    23 hours ago










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',
convertImagesToLinks: true,
noModals: false,
showLowRepImageUploadWarning: true,
reputationToPostImages: 10,
bindNavPrevention: true,
postfix: "",
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%2f52296075%2fidentity-for-binaryoperator%23new-answer', 'question_page');

);

Post as a guest

















3 Answers
3






active

oldest

votes








3 Answers
3






active

oldest

votes









active

oldest

votes






active

oldest

votes








up vote
6
down vote













Your question doesn't really make sense. If you were to paste your proposed BinaryOperator.identity method into an IDE, you would immediately see that it would complain that the identifier t is declared twice.



To fix this, we need a different identifier for each parameter:



return (t, u) -> t;


Now we can clearly see that this is not an identity function. It's a method which takes two arguments and returns the first one. Therefore the best name for this would be something like getFirst.



To answer your question about whether there's anything like this in the JDK: no. Using an identity function is a common use case, so defining a method for that is useful. Arbitrarily returning the first argument of two is not a common use case, and it's not useful to have a method to do that.






share|improve this answer




















  • Well, I agree to what you've stated here, except for the last part: Arbitrarily returning the first argument of two is not a common use case, and it's not useful to have a method to do that. In fact it is quite useful, as OP is showing here. Another use case is when implementing collectors and you need to provide a combiner function that you know will never be used (because i.e. the stream is sequential). Besides, BinaryOperator does provide maxBy(Comparator) static method (and also minBy), so it's valid to wonder why there's no i.e. first or second methods...
    – Federico Peralta Schaffner
    22 hours ago











  • @FedericoPeraltaSchaffner "common" is kind of subjective. I've never needed that, and I work with the JDK on a daily basis. By contrast, I've needed Function.identity probably 100s of times. The JDK is supposed to be a collection of broadly useful tools. It's not designed to contain absolutely everything a developer might ever need. If you add a getFirst, people will wonder where getSecond is. If you add both of those for BinaryOperator, why not add similar ones for the other 42 functional interfaces? Boom - you've now got ~100 basically useless methods cluttering the API.
    – Michael
    22 hours ago











  • I absolutely agree with you, I was playing the role of the devil's advocate a little bit... The problem with the question is not that there doesn't exist a method to choose the first or second element, but that Sonar is throwing an alert, when it shouldn't...
    – Federico Peralta Schaffner
    22 hours ago














up vote
6
down vote













Your question doesn't really make sense. If you were to paste your proposed BinaryOperator.identity method into an IDE, you would immediately see that it would complain that the identifier t is declared twice.



To fix this, we need a different identifier for each parameter:



return (t, u) -> t;


Now we can clearly see that this is not an identity function. It's a method which takes two arguments and returns the first one. Therefore the best name for this would be something like getFirst.



To answer your question about whether there's anything like this in the JDK: no. Using an identity function is a common use case, so defining a method for that is useful. Arbitrarily returning the first argument of two is not a common use case, and it's not useful to have a method to do that.






share|improve this answer




















  • Well, I agree to what you've stated here, except for the last part: Arbitrarily returning the first argument of two is not a common use case, and it's not useful to have a method to do that. In fact it is quite useful, as OP is showing here. Another use case is when implementing collectors and you need to provide a combiner function that you know will never be used (because i.e. the stream is sequential). Besides, BinaryOperator does provide maxBy(Comparator) static method (and also minBy), so it's valid to wonder why there's no i.e. first or second methods...
    – Federico Peralta Schaffner
    22 hours ago











  • @FedericoPeraltaSchaffner "common" is kind of subjective. I've never needed that, and I work with the JDK on a daily basis. By contrast, I've needed Function.identity probably 100s of times. The JDK is supposed to be a collection of broadly useful tools. It's not designed to contain absolutely everything a developer might ever need. If you add a getFirst, people will wonder where getSecond is. If you add both of those for BinaryOperator, why not add similar ones for the other 42 functional interfaces? Boom - you've now got ~100 basically useless methods cluttering the API.
    – Michael
    22 hours ago











  • I absolutely agree with you, I was playing the role of the devil's advocate a little bit... The problem with the question is not that there doesn't exist a method to choose the first or second element, but that Sonar is throwing an alert, when it shouldn't...
    – Federico Peralta Schaffner
    22 hours ago












up vote
6
down vote










up vote
6
down vote









Your question doesn't really make sense. If you were to paste your proposed BinaryOperator.identity method into an IDE, you would immediately see that it would complain that the identifier t is declared twice.



To fix this, we need a different identifier for each parameter:



return (t, u) -> t;


Now we can clearly see that this is not an identity function. It's a method which takes two arguments and returns the first one. Therefore the best name for this would be something like getFirst.



To answer your question about whether there's anything like this in the JDK: no. Using an identity function is a common use case, so defining a method for that is useful. Arbitrarily returning the first argument of two is not a common use case, and it's not useful to have a method to do that.






share|improve this answer












Your question doesn't really make sense. If you were to paste your proposed BinaryOperator.identity method into an IDE, you would immediately see that it would complain that the identifier t is declared twice.



To fix this, we need a different identifier for each parameter:



return (t, u) -> t;


Now we can clearly see that this is not an identity function. It's a method which takes two arguments and returns the first one. Therefore the best name for this would be something like getFirst.



To answer your question about whether there's anything like this in the JDK: no. Using an identity function is a common use case, so defining a method for that is useful. Arbitrarily returning the first argument of two is not a common use case, and it's not useful to have a method to do that.







share|improve this answer












share|improve this answer



share|improve this answer










answered 23 hours ago









Michael

15.9k62964




15.9k62964











  • Well, I agree to what you've stated here, except for the last part: Arbitrarily returning the first argument of two is not a common use case, and it's not useful to have a method to do that. In fact it is quite useful, as OP is showing here. Another use case is when implementing collectors and you need to provide a combiner function that you know will never be used (because i.e. the stream is sequential). Besides, BinaryOperator does provide maxBy(Comparator) static method (and also minBy), so it's valid to wonder why there's no i.e. first or second methods...
    – Federico Peralta Schaffner
    22 hours ago











  • @FedericoPeraltaSchaffner "common" is kind of subjective. I've never needed that, and I work with the JDK on a daily basis. By contrast, I've needed Function.identity probably 100s of times. The JDK is supposed to be a collection of broadly useful tools. It's not designed to contain absolutely everything a developer might ever need. If you add a getFirst, people will wonder where getSecond is. If you add both of those for BinaryOperator, why not add similar ones for the other 42 functional interfaces? Boom - you've now got ~100 basically useless methods cluttering the API.
    – Michael
    22 hours ago











  • I absolutely agree with you, I was playing the role of the devil's advocate a little bit... The problem with the question is not that there doesn't exist a method to choose the first or second element, but that Sonar is throwing an alert, when it shouldn't...
    – Federico Peralta Schaffner
    22 hours ago
















  • Well, I agree to what you've stated here, except for the last part: Arbitrarily returning the first argument of two is not a common use case, and it's not useful to have a method to do that. In fact it is quite useful, as OP is showing here. Another use case is when implementing collectors and you need to provide a combiner function that you know will never be used (because i.e. the stream is sequential). Besides, BinaryOperator does provide maxBy(Comparator) static method (and also minBy), so it's valid to wonder why there's no i.e. first or second methods...
    – Federico Peralta Schaffner
    22 hours ago











  • @FedericoPeraltaSchaffner "common" is kind of subjective. I've never needed that, and I work with the JDK on a daily basis. By contrast, I've needed Function.identity probably 100s of times. The JDK is supposed to be a collection of broadly useful tools. It's not designed to contain absolutely everything a developer might ever need. If you add a getFirst, people will wonder where getSecond is. If you add both of those for BinaryOperator, why not add similar ones for the other 42 functional interfaces? Boom - you've now got ~100 basically useless methods cluttering the API.
    – Michael
    22 hours ago











  • I absolutely agree with you, I was playing the role of the devil's advocate a little bit... The problem with the question is not that there doesn't exist a method to choose the first or second element, but that Sonar is throwing an alert, when it shouldn't...
    – Federico Peralta Schaffner
    22 hours ago















Well, I agree to what you've stated here, except for the last part: Arbitrarily returning the first argument of two is not a common use case, and it's not useful to have a method to do that. In fact it is quite useful, as OP is showing here. Another use case is when implementing collectors and you need to provide a combiner function that you know will never be used (because i.e. the stream is sequential). Besides, BinaryOperator does provide maxBy(Comparator) static method (and also minBy), so it's valid to wonder why there's no i.e. first or second methods...
– Federico Peralta Schaffner
22 hours ago





Well, I agree to what you've stated here, except for the last part: Arbitrarily returning the first argument of two is not a common use case, and it's not useful to have a method to do that. In fact it is quite useful, as OP is showing here. Another use case is when implementing collectors and you need to provide a combiner function that you know will never be used (because i.e. the stream is sequential). Besides, BinaryOperator does provide maxBy(Comparator) static method (and also minBy), so it's valid to wonder why there's no i.e. first or second methods...
– Federico Peralta Schaffner
22 hours ago













@FedericoPeraltaSchaffner "common" is kind of subjective. I've never needed that, and I work with the JDK on a daily basis. By contrast, I've needed Function.identity probably 100s of times. The JDK is supposed to be a collection of broadly useful tools. It's not designed to contain absolutely everything a developer might ever need. If you add a getFirst, people will wonder where getSecond is. If you add both of those for BinaryOperator, why not add similar ones for the other 42 functional interfaces? Boom - you've now got ~100 basically useless methods cluttering the API.
– Michael
22 hours ago





@FedericoPeraltaSchaffner "common" is kind of subjective. I've never needed that, and I work with the JDK on a daily basis. By contrast, I've needed Function.identity probably 100s of times. The JDK is supposed to be a collection of broadly useful tools. It's not designed to contain absolutely everything a developer might ever need. If you add a getFirst, people will wonder where getSecond is. If you add both of those for BinaryOperator, why not add similar ones for the other 42 functional interfaces? Boom - you've now got ~100 basically useless methods cluttering the API.
– Michael
22 hours ago













I absolutely agree with you, I was playing the role of the devil's advocate a little bit... The problem with the question is not that there doesn't exist a method to choose the first or second element, but that Sonar is throwing an alert, when it shouldn't...
– Federico Peralta Schaffner
22 hours ago




I absolutely agree with you, I was playing the role of the devil's advocate a little bit... The problem with the question is not that there doesn't exist a method to choose the first or second element, but that Sonar is throwing an alert, when it shouldn't...
– Federico Peralta Schaffner
22 hours ago












up vote
5
down vote













Your code seemingly can be improved as 



List<String> list = Arrays.asList("Abcd", "Abcd");
Map<String, Integer> map = list.stream()
 .collect(Collectors.toMap(Function.identity(), String::length, (a, b) -> a));
System.out.println(map.size());


Or possibly for your use case I don't want to do anything for two values of same key, I just wanted return one value, you may just choose to randomly return any value in using an implementation as following:



private static <T> BinaryOperator<T> any() 
 return Math.random() < 0.5 ? ((x, y) -> x) : ((x, y) -> y);



and then in your code use it as



Map<String, Integer> map = list.stream()
 .collect(Collectors.toMap(Function.identity(), String::length, any()));


Thanks to the suggestions from Holger, Eugene, and Federico, there are other efficient implementations of the any method that can actually involve using :



private static <T> BinaryOperator<T> any() 
 // suggested by Holger
 return ThreadLocalRandom.current().nextBoolean() ? ((x, y) -> x) : ((x, y) -> y);
 // suggested by Eugene
 long nt = System.nanoTime(); 
 ((nt >>> 32) ^ nt) > 0 ? ((x, y) -> x) : ((x, y) -> y);






share|improve this answer






















  • that btw is a very interesting merger :)
    – Eugene
    yesterday






  • 1




    I guess you are right, it just looks weird to me.
    – Eugene
    yesterday






  • 3




    You know that your random implementation won't pick a random argument each time, but will choose one implementation when called, and then will return the same argument all times. btw I didn't downvote
    – Federico Peralta Schaffner
    yesterday







  • 1




    I remember I asked something about that here and Stuart Marks kindly answered... :D
    – Federico Peralta Schaffner
    23 hours ago






  • 2




    I’d use ThreadLocalRandom.current().nextBoolean() instead of Math.random() < 0.5. An alternative would be (a,b) -> assert Objects.equals(a, b); return a;
    – Holger
    20 hours ago














up vote
5
down vote













Your code seemingly can be improved as 



List<String> list = Arrays.asList("Abcd", "Abcd");
Map<String, Integer> map = list.stream()
 .collect(Collectors.toMap(Function.identity(), String::length, (a, b) -> a));
System.out.println(map.size());


Or possibly for your use case I don't want to do anything for two values of same key, I just wanted return one value, you may just choose to randomly return any value in using an implementation as following:



private static <T> BinaryOperator<T> any() 
 return Math.random() < 0.5 ? ((x, y) -> x) : ((x, y) -> y);



and then in your code use it as



Map<String, Integer> map = list.stream()
 .collect(Collectors.toMap(Function.identity(), String::length, any()));


Thanks to the suggestions from Holger, Eugene, and Federico, there are other efficient implementations of the any method that can actually involve using :



private static <T> BinaryOperator<T> any() 
 // suggested by Holger
 return ThreadLocalRandom.current().nextBoolean() ? ((x, y) -> x) : ((x, y) -> y);
 // suggested by Eugene
 long nt = System.nanoTime(); 
 ((nt >>> 32) ^ nt) > 0 ? ((x, y) -> x) : ((x, y) -> y);






share|improve this answer






















  • that btw is a very interesting merger :)
    – Eugene
    yesterday






  • 1




    I guess you are right, it just looks weird to me.
    – Eugene
    yesterday






  • 3




    You know that your random implementation won't pick a random argument each time, but will choose one implementation when called, and then will return the same argument all times. btw I didn't downvote
    – Federico Peralta Schaffner
    yesterday







  • 1




    I remember I asked something about that here and Stuart Marks kindly answered... :D
    – Federico Peralta Schaffner
    23 hours ago






  • 2




    I’d use ThreadLocalRandom.current().nextBoolean() instead of Math.random() < 0.5. An alternative would be (a,b) -> assert Objects.equals(a, b); return a;
    – Holger
    20 hours ago












up vote
5
down vote










up vote
5
down vote









Your code seemingly can be improved as 



List<String> list = Arrays.asList("Abcd", "Abcd");
Map<String, Integer> map = list.stream()
 .collect(Collectors.toMap(Function.identity(), String::length, (a, b) -> a));
System.out.println(map.size());


Or possibly for your use case I don't want to do anything for two values of same key, I just wanted return one value, you may just choose to randomly return any value in using an implementation as following:



private static <T> BinaryOperator<T> any() 
 return Math.random() < 0.5 ? ((x, y) -> x) : ((x, y) -> y);



and then in your code use it as



Map<String, Integer> map = list.stream()
 .collect(Collectors.toMap(Function.identity(), String::length, any()));


Thanks to the suggestions from Holger, Eugene, and Federico, there are other efficient implementations of the any method that can actually involve using :



private static <T> BinaryOperator<T> any() 
 // suggested by Holger
 return ThreadLocalRandom.current().nextBoolean() ? ((x, y) -> x) : ((x, y) -> y);
 // suggested by Eugene
 long nt = System.nanoTime(); 
 ((nt >>> 32) ^ nt) > 0 ? ((x, y) -> x) : ((x, y) -> y);






share|improve this answer














Your code seemingly can be improved as 



List<String> list = Arrays.asList("Abcd", "Abcd");
Map<String, Integer> map = list.stream()
 .collect(Collectors.toMap(Function.identity(), String::length, (a, b) -> a));
System.out.println(map.size());


Or possibly for your use case I don't want to do anything for two values of same key, I just wanted return one value, you may just choose to randomly return any value in using an implementation as following:



private static <T> BinaryOperator<T> any() 
 return Math.random() < 0.5 ? ((x, y) -> x) : ((x, y) -> y);



and then in your code use it as



Map<String, Integer> map = list.stream()
 .collect(Collectors.toMap(Function.identity(), String::length, any()));


Thanks to the suggestions from Holger, Eugene, and Federico, there are other efficient implementations of the any method that can actually involve using :



private static <T> BinaryOperator<T> any() 
 // suggested by Holger
 return ThreadLocalRandom.current().nextBoolean() ? ((x, y) -> x) : ((x, y) -> y);
 // suggested by Eugene
 long nt = System.nanoTime(); 
 ((nt >>> 32) ^ nt) > 0 ? ((x, y) -> x) : ((x, y) -> y);







share|improve this answer














share|improve this answer



share|improve this answer








edited 6 hours ago

























answered yesterday









nullpointer

30.4k1054127




30.4k1054127











  • that btw is a very interesting merger :)
    – Eugene
    yesterday






  • 1




    I guess you are right, it just looks weird to me.
    – Eugene
    yesterday






  • 3




    You know that your random implementation won't pick a random argument each time, but will choose one implementation when called, and then will return the same argument all times. btw I didn't downvote
    – Federico Peralta Schaffner
    yesterday







  • 1




    I remember I asked something about that here and Stuart Marks kindly answered... :D
    – Federico Peralta Schaffner
    23 hours ago






  • 2




    I’d use ThreadLocalRandom.current().nextBoolean() instead of Math.random() < 0.5. An alternative would be (a,b) -> assert Objects.equals(a, b); return a;
    – Holger
    20 hours ago
















  • that btw is a very interesting merger :)
    – Eugene
    yesterday






  • 1




    I guess you are right, it just looks weird to me.
    – Eugene
    yesterday






  • 3




    You know that your random implementation won't pick a random argument each time, but will choose one implementation when called, and then will return the same argument all times. btw I didn't downvote
    – Federico Peralta Schaffner
    yesterday







  • 1




    I remember I asked something about that here and Stuart Marks kindly answered... :D
    – Federico Peralta Schaffner
    23 hours ago






  • 2




    I’d use ThreadLocalRandom.current().nextBoolean() instead of Math.random() < 0.5. An alternative would be (a,b) -> assert Objects.equals(a, b); return a;
    – Holger
    20 hours ago















that btw is a very interesting merger :)
– Eugene
yesterday




that btw is a very interesting merger :)
– Eugene
yesterday




1




1




I guess you are right, it just looks weird to me.
– Eugene
yesterday




I guess you are right, it just looks weird to me.
– Eugene
yesterday




3




3




You know that your random implementation won't pick a random argument each time, but will choose one implementation when called, and then will return the same argument all times. btw I didn't downvote
– Federico Peralta Schaffner
yesterday





You know that your random implementation won't pick a random argument each time, but will choose one implementation when called, and then will return the same argument all times. btw I didn't downvote
– Federico Peralta Schaffner
yesterday





1




1




I remember I asked something about that here and Stuart Marks kindly answered... :D
– Federico Peralta Schaffner
23 hours ago




I remember I asked something about that here and Stuart Marks kindly answered... :D
– Federico Peralta Schaffner
23 hours ago




2




2




I’d use ThreadLocalRandom.current().nextBoolean() instead of Math.random() < 0.5. An alternative would be (a,b) -> assert Objects.equals(a, b); return a;
– Holger
20 hours ago




I’d use ThreadLocalRandom.current().nextBoolean() instead of Math.random() < 0.5. An alternative would be (a,b) -> assert Objects.equals(a, b); return a;
– Holger
20 hours ago










up vote
4
down vote













T means they have the same types, not the same values, that is not an identity per-se.



It just means that BinaryOperator will be used for the same types, but providing an identity for different values... this somehow sounds like foldLeft or foldRight or foldLeftIdentity/foldRightIdentity, which java does not have.






share|improve this answer






















  • what's your thought on the implementation of any for returning any value as in the merger?
    – nullpointer
    yesterday






  • 1




    @nullpointer :) that is all I can say. probably Math.random is too expensive here, I would go for something cheaper like long nt = System.nanoTime(); ((nt >>> 32) ^ nt) > 0 ? ... : ... but I've seen this somewhere else already :) (of course the cheapest and fastest random would have to be taken here)
    – Eugene
    23 hours ago














up vote
4
down vote













T means they have the same types, not the same values, that is not an identity per-se.



It just means that BinaryOperator will be used for the same types, but providing an identity for different values... this somehow sounds like foldLeft or foldRight or foldLeftIdentity/foldRightIdentity, which java does not have.






share|improve this answer






















  • what's your thought on the implementation of any for returning any value as in the merger?
    – nullpointer
    yesterday






  • 1




    @nullpointer :) that is all I can say. probably Math.random is too expensive here, I would go for something cheaper like long nt = System.nanoTime(); ((nt >>> 32) ^ nt) > 0 ? ... : ... but I've seen this somewhere else already :) (of course the cheapest and fastest random would have to be taken here)
    – Eugene
    23 hours ago












up vote
4
down vote










up vote
4
down vote









T means they have the same types, not the same values, that is not an identity per-se.



It just means that BinaryOperator will be used for the same types, but providing an identity for different values... this somehow sounds like foldLeft or foldRight or foldLeftIdentity/foldRightIdentity, which java does not have.






share|improve this answer














T means they have the same types, not the same values, that is not an identity per-se.



It just means that BinaryOperator will be used for the same types, but providing an identity for different values... this somehow sounds like foldLeft or foldRight or foldLeftIdentity/foldRightIdentity, which java does not have.







share|improve this answer














share|improve this answer



share|improve this answer








edited yesterday

























answered yesterday









Eugene

59.8k980139




59.8k980139











  • what's your thought on the implementation of any for returning any value as in the merger?
    – nullpointer
    yesterday






  • 1




    @nullpointer :) that is all I can say. probably Math.random is too expensive here, I would go for something cheaper like long nt = System.nanoTime(); ((nt >>> 32) ^ nt) > 0 ? ... : ... but I've seen this somewhere else already :) (of course the cheapest and fastest random would have to be taken here)
    – Eugene
    23 hours ago
















  • what's your thought on the implementation of any for returning any value as in the merger?
    – nullpointer
    yesterday






  • 1




    @nullpointer :) that is all I can say. probably Math.random is too expensive here, I would go for something cheaper like long nt = System.nanoTime(); ((nt >>> 32) ^ nt) > 0 ? ... : ... but I've seen this somewhere else already :) (of course the cheapest and fastest random would have to be taken here)
    – Eugene
    23 hours ago















what's your thought on the implementation of any for returning any value as in the merger?
– nullpointer
yesterday




what's your thought on the implementation of any for returning any value as in the merger?
– nullpointer
yesterday




1




1




@nullpointer :) that is all I can say. probably Math.random is too expensive here, I would go for something cheaper like long nt = System.nanoTime(); ((nt >>> 32) ^ nt) > 0 ? ... : ... but I've seen this somewhere else already :) (of course the cheapest and fastest random would have to be taken here)
– Eugene
23 hours ago




@nullpointer :) that is all I can say. probably Math.random is too expensive here, I would go for something cheaper like long nt = System.nanoTime(); ((nt >>> 32) ^ nt) > 0 ? ... : ... but I've seen this somewhere else already :) (of course the cheapest and fastest random would have to be taken here)
– Eugene
23 hours ago

















 

draft saved


draft discarded















































 


draft saved


draft discarded














StackExchange.ready(
function ()
StackExchange.openid.initPostLogin('.new-post-login', 'https%3a%2f%2fstackoverflow.com%2fquestions%2f52296075%2fidentity-for-binaryoperator%23new-answer', 'question_page');

);

Post as a guest
































































Comments

Post a Comment

Popular posts from this blog

What does second last employer means? [closed]

Image
Clash Royale CLAN TAG #URR8PPP .everyoneloves__top-leaderboard:empty,.everyoneloves__mid-leaderboard:empty margin-bottom:0; up vote -3 down vote favorite Say, for example I have worked in the following companies, Company A => 2010 - 2011 Company B => 2011 - 2012 Company C => 2012 - Till Date Company D => Have offer Now assume Company D is asking to submit documents from my second last employer. So from the list of companies given, which company would be qualified as second last employer? software-industry employer share | improve this question asked Aug 10 '14 at 5:16 Rajaprabhu Aravindasamy 559 1 6 13 closed as off-topic by Jim G., jmort253 ♦ Aug 11 '14 at 1:58 This question appears to be off-topic. The users who voted to close gave this specific reason: "Questions seeking advice on company-specific regulations, agreements, or policies should be directed to your manager or HR department. Questions that address...
Read more

List of Gilmore Girls characters

Image
The season three cast—(Back row): Lane, Michel, Paris, Emily, Richard, Sookie, Miss Patty, Kirk; Front row: Jess, Luke, Lorelai, Rory, Dean This is a list of characters for the comedy-drama television series Gilmore Girls . Contents 1 Main characters 1.1 Lorelai Gilmore 1.2 Rory Gilmore 1.3 Sookie St. James 1.4 Lane Kim 1.5 Michel Gerard 1.6 Luke Danes 1.7 Emily Gilmore 1.8 Richard Gilmore 1.9 Paris Geller 1.10 Dean Forester 1.11 Jess Mariano 1.12 Kirk Gleason 1.13 Jason Stiles 1.14 Logan Huntzberger 2 Major recurring characters 2.1 Christopher Hayden 2.2 Jackson Belleville 2.3 Mrs. Kim 2.4 Tristin DuGray 2.5 Max Medina 2.6 Taylor Doose 2.7 Dave Rygalski 2.8 Marty 3 Recurring characters 3.1 Stars Hollow 3.2 Chilton 3.3 Yale 3.4 Other 4 Notable guest stars 5 Unseen characters 6 Notes 7 References Main characters Actor Character Appearances Season 1 Season 2 Season 3 Season 4 Season 5 Season 6 Season 7 A Year in the Life Lauren Graham Lorelai Gilmore...
Read more

One-line joke

A one-liner is a joke that is delivered in a single line. A good one-liner is said to be pithy - concise and meaningful. [1] Comedians and actors use this comedic method as part of their act, e.g. Jimmy Carr, Tommy Cooper, Rodney Dangerfield, Norm Macdonald, Ken Dodd, Stewart Francis, Zach Galifianakis, Mitch Hedberg, Anthony Jeselnik, Milton Jones, Shappi Khorsandi, Jay London, Mark Linn-Baker, Demetri Martin, Groucho Marx, Dan Mintz, Emo Philips, Tim Vine, Steven Wright and Henny Youngman and Arnold Schwarzenegger. [ citation needed ] Many fictional characters are also known to deliver one-liners, including James Bond, who usually includes pithy and laconic quips after disposing of a villain. [ citation needed ] Examples "A baby seal walks into a club." "A dyslexic man walks into a bra." (Anonymous) "There are three types of people, those who can count and those who can't." "Venison's dear, isn't it?" (Jimmy Carr) "An escala...
Read more