Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

remove all "foos" and "bars" #261

Open
timruffles opened this issue Dec 16, 2014 · 9 comments
Open

remove all "foos" and "bars" #261

timruffles opened this issue Dec 16, 2014 · 9 comments

Comments

@timruffles
Copy link
Collaborator

foos and bars are horrible in example code as it gives no context on why you'd do something.

@dashed
Copy link

dashed commented Dec 18, 2014

@timruffles
Copy link
Collaborator Author

?

@chrisranderson
Copy link
Contributor

Agreed. They could at least be named "some" and "another" so it's easier to remember which function or variable is which. "first" and "second" could work too.

Foo and bar just add mental strain to the examples:
var foo = {name: 'kitten'}
foo.name; // kitten
foo['name']; // kitten

var get = 'name';
foo[get]; // kitten

foo.1234; // SyntaxError
foo['1234']; // works

Why would you ever name foo kitten? It makes it harder to remember what's going on. Easier to use var person, and person.name. If you keep to a theme throughout the document I bet it would erase a lot of the foo bar overhead.

@hugotox
Copy link
Contributor

hugotox commented May 29, 2015

foo and bar are specially confusing for non english speakers (like me) because we can't translate/understand these words

@timruffles
Copy link
Collaborator Author

Still a long way to go:

$ ack -hc '\b(foo|bar|baz|qux)\b' doc/en/
164

@roccomuso
Copy link
Contributor

Totally Agree.

@timruffles
Copy link
Collaborator Author

PRs would be gratefully accepted on this one. Puns win extra points.

On Tue, Nov 3, 2015 at 9:28 PM, Rocco Musolino notifications@github.com
wrote:

Totally Agree.


Reply to this email directly or view it on GitHub
#261 (comment)
.

@siddharthkp
Copy link
Contributor

My tiny contribution: #373

@alieldeba
Copy link

agreed with that

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

7 participants