Hacker News Reader

Comment by po | Hacker News Reader

"point" can be a verb too. :-)

Just today I renamed a method someone named "showButton" because I thought

  result = showButton()

was a call that showed the button and returned a boolean result. In reality it just returned a boolean based on if it was configured to show a button. The new name: "shouldShowButton"

But then I got to thinking… well… maybe I'm suggesting that the button should be shown…

There's something to be said for context.

View on HN

I usually write out a bunch of names without thinking to hard about them and then try to read them. In this instance, given that you've already had showButton and shouldShowButton i'd probably look at:

  result = buttonShown()
  result = buttonVisible()
  result = buttonIsVisible()
  result = visibleButton()
  result = !buttonHidden()

To a lot of programmers this would seem like a waste of time, but I'm with the article's author (and, I'd hope, other HN readers) in that it's time well spent.

[Redacted] Misread your post.

Reply on news.ycombinator.com