Boolean.toString(true) and Boolean.parseBoolean("true")?

Changelog:

-Added Utilities method textToBoolean(String)

-Added Utilities method booleanToText(Boolean)

 

 

Umm....

booleanToText(Boolean) == (Boolean.toString(Boolean) || (Boolean+""))

textToBoolean(String) == Boolean.parseBoolean("true");

Umm....

booleanToText(Boolean) == (Boolean.toString(Boolean) || (Boolean+""))

textToBoolean(String) == Boolean.parseBoolean("true");

 

parseBoolean uses equalsIgnoreCase and I wanted to make matching methods. If it's really that big of a deal I can remove it. Just figured I would make a small private method public.

 

From the API:

 

It's pretty much identical..

parseBoolean uses equalsIgnoreCase and I wanted to make matching methods. If it's really that big of a deal I can remove it. Just figured I would make a small private method public.

 

You need to look at the default method first. Your method is completely redundant.

 

Read my post above yours, I edited it. I know what the default method looks like, that's why I made mine. If it's really that big of a problem I'll just make it private again.

Read my post above yours, I edited it. I know what the default method looks like, that's why I made mine. If it's really that big of a problem I'll just make it private again.

 

I'm not saying that it is a problem, just useless at best. There is no reason for it to public or even be private for that matter, when you can achieve the same by using the default methods, with the added benefit of less lines of code.

 

Not trying to put you down, just being a critic.

I'm not saying that it is a problem, just useless at best. There is no reason for it to public or even be private for that matter, when you can achieve the same by using the default methods, with the added benefit of less lines of code.

 

Not trying to put you down, just being a critic.

 

booleanToText is identical, but it was only to match textToBoolean. I wanted to assess that the string was in fact "true" without manipulation. Therefore putting it to lowercase first or toString() wasn't viable. The only reason why I mentioned it is because all API additions/changes go into the changelog. 

 

For the average person yes, it's really not needed.

 

Edit: I'll update API documentation to make it a little bit less confusing and remove booleanToText on the next update.

Very nice. Good work Alek.

Awesome man, 3 minutes is completely fair.

 

 

Thanks for the update! 

Question is: worth updating?

Goodjob

Alek for president pl0x