41
votes
\$\begingroup\$

As you all know, we're very close to sending SWAG to some of you! One of items you'll get is this awesome T-shirt!

DAMN AWESOME T-SHIRT

As you can see, below the logo we placed a very short tagline: "Making code slightly less wrong.". This is the tagline that was proposed in chat. But not everyone is in chat. So we'd like you guys to give us your proposals and vote to pick the one we’re gonna print.

Here're rules:

  • Add proposals as answers to this meta post.
  • One proposal per answer.
  • We’ll close the list and voting in January 15th, 2016 @ 1:37pm EST. The answer with the biggest number of upvotes on that date wins and goes to printing. Caveat: this is subject to approval from the design team overlords me.

And here're guidelines:

  • It should be a one-liner. So it can't be too long (or too short :)). Probably similar in length to "Making code slightly less wrong."... more or less... :)
  • Yea, I think that's it.
\$\endgroup\$
7
  • \$\begingroup\$ Can they use.. colourful language? (like "Crap"). \$\endgroup\$
    – Dan
    Jan 7, 2016 at 10:05
  • \$\begingroup\$ @DanPantry I'd say it depends on context :) \$\endgroup\$
    – Paweł
    Jan 7, 2016 at 10:06
  • 8
    \$\begingroup\$ I can haz my name on it ? :) \$\endgroup\$
    – Emily L.
    Jan 7, 2016 at 13:35
  • \$\begingroup\$ @EmilyL. umm.. I don't think so :) \$\endgroup\$
    – Paweł
    Jan 7, 2016 at 14:15
  • 1
    \$\begingroup\$ # I survived the The Mission If only you could put links on t-shirts. \$\endgroup\$
    – Dan
    Jan 7, 2016 at 14:58
  • \$\begingroup\$ Where can I find a file with that image? I can print it on a shirt myself... \$\endgroup\$
    – Konrad Viltersten
    Jan 12, 2016 at 9:46
  • 1
    \$\begingroup\$ @KonradViltersten well the Code Review logo is a copyrighted property of Stack Exchange Inc. ..I wouldn't mess with that ;-) \$\endgroup\$
    – Mathieu Guindon
    Jan 15, 2016 at 17:00

48 Answers 48

Reset to default
1
2
32
votes
\$\begingroup\$

Making code slightly less wrong.

Just so the ones loving the existing line can vote to keep it.

Credits: @Zak

\$\endgroup\$
13
  • \$\begingroup\$ I think I really like it. \$\endgroup\$
    – Morwenn
    Jan 7, 2016 at 12:10
  • \$\begingroup\$ -1 because less wrong is (already used as) a tagline of the Skeptics community (e.g. Skeptics.SE) \$\endgroup\$
    – ChrisW
    Jan 7, 2016 at 17:00
  • 6
    \$\begingroup\$ @ChrisW and that means that the Skeptics community has some patent on using "less wrong"? \$\endgroup\$
    – Simon Forsberg
    Jan 7, 2016 at 17:15
  • \$\begingroup\$ @ChrisW I can't find any reference to "less wrong" being included in the tagline on skeptics.stackexchange.com \$\endgroup\$
    – Simon Forsberg
    Jan 7, 2016 at 17:18
  • \$\begingroup\$ @SimonForsbergMcFeely Merely that I wouldn't choose e.g. "pro choice" or "pro life" or whatever as a tagline either. I'm simply saying that this phrase already has some associations which you may or may not be aware of, and which are more-or-less unrelated to this site. \$\endgroup\$
    – ChrisW
    Jan 7, 2016 at 17:22
  • 2
    \$\begingroup\$ @SimonForsbergMcFeely Also code isn't supposed to be wrong at all, is it? So isn't "slightly less wrong" a bit of false humility, in this context? \$\endgroup\$
    – ChrisW
    Jan 7, 2016 at 17:28
  • 1
    \$\begingroup\$ Perhaps this could be revised to 'Making good code slightly less wrong'? \$\endgroup\$
    – Mast Mod
    Jan 7, 2016 at 19:39
  • \$\begingroup\$ @Mast Or "Making working code slightly less wrong" \$\endgroup\$
    – Simon Forsberg
    Jan 7, 2016 at 20:51
  • 5
    \$\begingroup\$ @SimonForsbergMcFeely "working" isn't catchy. I like the proposed tagline for it is funny and also makes reviewers humble, and highlights the fact that there is no perfect code :) \$\endgroup\$
    – Morwenn
    Jan 8, 2016 at 0:16
  • 3
    \$\begingroup\$ -1 Isn't the whole basis of this site that there's a difference between "good" and "not wrong"? \$\endgroup\$
    – Ben Aaronson
    Jan 9, 2016 at 12:13
  • \$\begingroup\$ @BenAaronson Reviews need to be reviewed too, so as long as a review hasn't been reviewed, you can only assume that the code is slightly less wrong. \$\endgroup\$
    – Morwenn
    Jan 10, 2016 at 17:51
  • \$\begingroup\$ @BenAaronson How about "Making code slightly less bad." ? \$\endgroup\$
    – Simon Forsberg
    Jan 14, 2016 at 17:27
  • \$\begingroup\$ @SimonForsberg Yeah, something along those lines, I'd like. "Less awful" could also work, though possibly too negative \$\endgroup\$
    – Ben Aaronson
    Jan 14, 2016 at 17:29
28
votes
\$\begingroup\$

Because all code can be improved.

Except Jon Skeet's

\$\endgroup\$
5
  • 2
    \$\begingroup\$ Just as long as you don't include the part about Jon Skeet, I'm okay with this one. \$\endgroup\$
    – Simon Forsberg
    Jan 7, 2016 at 17:11
  • 1
    \$\begingroup\$ @SimonForsbergMcFeely the non-bold part isn't meant to be part of the tagline \$\endgroup\$
    – Mathieu Guindon
    Jan 7, 2016 at 17:24
  • 5
    \$\begingroup\$ Why the "Because"? I prefer "All code can be improved" \$\endgroup\$
    – rolfl
    Jan 12, 2016 at 0:30
  • 3
    \$\begingroup\$ @rolfl because. \$\endgroup\$
    – Mathieu Guindon
    Jan 12, 2016 at 21:26
  • \$\begingroup\$ I prefer "All code can be improved" as well because it doesn't necessarily refer to CR as a community but also works as a standalone statement. \$\endgroup\$
    – Jeroen Vannevel
    Jan 14, 2016 at 15:38
23
votes
\$\begingroup\$

Make it work, we'll make it shine

Hints at rule #1 code must already be working, and pretty much sums up what we're all about.

\$\endgroup\$
17
votes
\$\begingroup\$

if (code.isUgly()) { code.review(); }

As a wink to all those who write code for fun and/or for a living. And I think it reads good enough as English for non-coders to "get it".

Alternative (suggested by ChrisW)

if (code.isUgly) { code.review(); }

(difference is isUgly is a property instead of method)

\$\endgroup\$
15
  • 6
    \$\begingroup\$ Ooh that would look so good on a CR mug! \$\endgroup\$
    – Mathieu Guindon
    Jan 7, 2016 at 18:20
  • \$\begingroup\$ @Mat'sMug Yes it would! \$\endgroup\$
    – Phrancis
    Jan 7, 2016 at 18:20
  • 6
    \$\begingroup\$ Maybe isUgly should be a property not a method. Also the villagers are coming with torches and pitchforks to protest your placement of curly braces. \$\endgroup\$
    – ChrisW
    Jan 7, 2016 at 18:26
  • \$\begingroup\$ OK I changed it by removing the extra round braces. \$\endgroup\$
    – Phrancis
    Jan 7, 2016 at 18:31
  • 4
    \$\begingroup\$ What's with these ugly braces and semicolon? (Obligatory pythonista comment) \$\endgroup\$
    – 301_Moved_Permanently
    Jan 7, 2016 at 19:13
  • 5
    \$\begingroup\$ At least make isUgly a function call, because publicly accessible members isUgly :) \$\endgroup\$
    – Emily L.
    Jan 7, 2016 at 19:51
  • \$\begingroup\$ I'd go with consistency and stick to the SO mug's mug.isEmpty(), so code.isUgly() works for me \$\endgroup\$
    – Mathieu Guindon
    Jan 7, 2016 at 19:56
  • 6
    \$\begingroup\$ @EmilyL. Look at that, I'm getting a code review on my tshirt tagline ;-) \$\endgroup\$
    – Phrancis
    Jan 7, 2016 at 20:18
  • \$\begingroup\$ @Mat'sMug I added both alternatives cuz. \$\endgroup\$
    – Phrancis
    Jan 7, 2016 at 20:36
  • 1
    \$\begingroup\$ Does 'Law of Demeter' suggest opportunity for improvement? The code appearing twice implies isUgly and review() could be together, as (private) implementation details of Code, for example class Code { internal void improve() { while (isUgly) review(); } bool isUgly etc. } \$\endgroup\$
    – ChrisW
    Jan 7, 2016 at 23:19
  • 8
    \$\begingroup\$ Should be while \$\endgroup\$
    – nhgrif
    Jan 7, 2016 at 23:32
  • 3
    \$\begingroup\$ @nhgrif Should be while(code){code.review();} \$\endgroup\$
    – oliverpool
    Jan 8, 2016 at 8:28
  • 1
    \$\begingroup\$ class Code { internal void improve() { while (improveMore()); } bool improveMore() { ...etc... } } \$\endgroup\$
    – ChrisW
    Jan 8, 2016 at 21:29
  • 2
    \$\begingroup\$ Forget while loops; make it recursive! Wait, no, then people will just end up on Stack Overflow instead \$\endgroup\$
    – Flambino
    Jan 10, 2016 at 20:43
  • \$\begingroup\$ @Mat'sMug On my last assignment, it should be if(true) code.review()... A bit depressing. \$\endgroup\$
    – Konrad Viltersten
    Jan 12, 2016 at 9:48
15
votes
\$\begingroup\$

If it's not broken... fix it anyway

\$\endgroup\$
2
  • 1
    \$\begingroup\$ Maybe "If it's not broke, fix it" would be catchier? \$\endgroup\$
    – Ben Aaronson
    Jan 14, 2016 at 17:30
  • \$\begingroup\$ Maybe. I was just playing off the "if it's not broken don't fix it" \$\endgroup\$
    – Raystafarian
    Jan 14, 2016 at 17:31
11
votes
\$\begingroup\$

Does this code make my ass look fat?

(source here )

\$\endgroup\$
1
  • \$\begingroup\$ Oh how I want this shirt. \$\endgroup\$
    – RubberDuck
    Jan 15, 2016 at 0:05
10
votes
\$\begingroup\$

There's always room for improvement

Credit to @Pops

\$\endgroup\$
9
votes
\$\begingroup\$

Your code... just better

I think the phrase came from somewhere on Meta, but I don't know where.

\$\endgroup\$
9
votes
\$\begingroup\$

finally { exceptional code }

Could also be try { exceptional code }, or catch .. of course, with the former being the most logical. But the above reads better, and has a longer "fuse" before you go "wait, what?" :)

Based on a comment exchange with ChrisW on this old community ad suggestion.

\$\endgroup\$
1
  • 1
    \$\begingroup\$ Except, finally runs whether or not... er, whatever. Good one ;-) \$\endgroup\$
    – Mathieu Guindon
    Jan 9, 2016 at 5:04
8
votes
\$\begingroup\$

Improving code, bit by bit

I thought I read something similar on the "New Design" post, but couldn't find it.

\$\endgroup\$
5
  • 4
    \$\begingroup\$ I love this. Especially the pun. \$\endgroup\$
    – user34073
    Jan 7, 2016 at 16:18
  • \$\begingroup\$ @Hosch250 It's been almost two days and I still don't get the pun... please, help me out. \$\endgroup\$
    – Simon Forsberg
    Jan 8, 2016 at 22:26
  • \$\begingroup\$ @SimonForsbergMcFeely one bit is 1/8th of a byte, isn't it? And changing something bit by bit is also changing things one step at a time. \$\endgroup\$
    – 301_Moved_Permanently
    Jan 8, 2016 at 22:43
  • \$\begingroup\$ @SimonForsbergMcFeely Code runs on bits and bytes. \$\endgroup\$
    – user34073
    Jan 8, 2016 at 23:15
  • \$\begingroup\$ @MathiasEttinger Okay then, I did get the pun apparently. I just didn't think it was that funny... \$\endgroup\$
    – Simon Forsberg
    Jan 9, 2016 at 0:38
8
votes
\$\begingroup\$

Leave this code a little better than you found it

Liberally inspired from Robert Baden-Powell

\$\endgroup\$
7
votes
\$\begingroup\$

A community working together for better code

Because that's how I feel :)

\$\endgroup\$
6
votes
\$\begingroup\$

Because every programmer needs a reviewer.

They just might not know it yet.

\$\endgroup\$
6
votes
\$\begingroup\$

Where good code comes to get better

\$\endgroup\$
1
  • 3
    \$\begingroup\$ "Where bad code comes to die"? \$\endgroup\$
    – Konrad Viltersten
    Jan 12, 2016 at 9:50
6
votes
\$\begingroup\$

{ Brace yourselves }

One of the most common review remarks.

\$\endgroup\$
1
  • \$\begingroup\$ ...if you will, with a bit of meme/pop culture reference (review is coming!). \$\endgroup\$
    – Lundin
    Jan 13, 2016 at 12:33
5
votes
\$\begingroup\$

The code cleanup crew

(Make it 30 characters)

\$\endgroup\$
4
votes
\$\begingroup\$

It takes a community to raise a program...

\$\endgroup\$
1
  • 6
    \$\begingroup\$ and it takes one mistake to raise an exception (sorry, had to) \$\endgroup\$
    – Flambino
    Jan 8, 2016 at 18:54
4
votes
\$\begingroup\$

 

^ That there is nothing: no tag line at all, only the CODE Q REVIEW logo.

\$\endgroup\$
4
votes
\$\begingroup\$

We can refactor it. We have the technology.

\$\endgroup\$
3
  • \$\begingroup\$ Don't leave your terminal with an interactive session when marketing might come wandering in. \$\endgroup\$
    – greybeard
    Jan 11, 2016 at 15:24
  • \$\begingroup\$ @greybeard is this a reference I should be aware of :x \$\endgroup\$
    – Dan
    Jan 11, 2016 at 15:27
  • \$\begingroup\$ None that I'm aware of. We have the technology rings neither coder nor reviewer. \$\endgroup\$
    – greybeard
    Jan 11, 2016 at 15:38
3
votes
\$\begingroup\$

Helping you code better

It's not really about making the posted code better. It's really about making the posters write better code in the future. Teach a man to fish.

\$\endgroup\$
2
votes
\$\begingroup\$

Making your code better

At first I thought "making your (working) code better", but that's too pedantic, not easy-going enough. "working" code is kind of implied, no need to mention.

\$\endgroup\$
2
votes
\$\begingroup\$

codereview.stackexchange.com

Perhaps a bit boring though.

\$\endgroup\$
4
  • \$\begingroup\$ 1. perhaps? 2. a bit? \$\endgroup\$
    – Simon Forsberg
    Jan 7, 2016 at 17:13
  • 2
    \$\begingroup\$ It would be nice if the website URL could be included though... \$\endgroup\$
    – Simon Forsberg
    Jan 7, 2016 at 17:19
  • 1
    \$\begingroup\$ @SimonForsbergMcFeely yeah. on the back perhaps? \$\endgroup\$
    – Mathieu Guindon
    Jan 7, 2016 at 17:19
  • \$\begingroup\$ I'll vote against a URL on the shirt. It just feels like it'd have more of a business card feel to it, which is a lot less cool than the feel it has now. \$\endgroup\$
    – Veedrac
    Jan 10, 2016 at 9:10
2
votes
\$\begingroup\$

Making code better, together

\$\endgroup\$
2
votes
\$\begingroup\$

Better your code

"Long enough to cover the subject, and short enough to create interest."

\$\endgroup\$
2
votes
\$\begingroup\$

Tough questions, Smart answers

From the old community ad:

\$\endgroup\$
1
vote
\$\begingroup\$

Making your code suck less

(tongue-in-cheek)

\$\endgroup\$
1
vote
\$\begingroup\$

Detail-Oreinted Developers

Says it all, I think.

PS: typo is intentional, stop fixing it ;-)

\$\endgroup\$
4
  • 1
    \$\begingroup\$ @mattsmug oh come on! \$\endgroup\$
    – jonrsharpe
    Jan 7, 2016 at 19:18
  • 3
    \$\begingroup\$ LOL! I totally just... lol \$\endgroup\$
    – Mathieu Guindon
    Jan 7, 2016 at 19:32
  • \$\begingroup\$ @jonrsharpe got me as well lol. \$\endgroup\$
    – Phrancis
    Jan 7, 2016 at 20:48
  • \$\begingroup\$ I find the fact that two members here fixed your bug rather apropos for Meta. For a T-shirt slogan, though, not so much. \$\endgroup\$
    – 200_success
    Jan 12, 2016 at 7:59
1
vote
\$\begingroup\$

We take code seriously

Because those who don't just push and deploy.

\$\endgroup\$
0
votes
\$\begingroup\$

Mission Accomplished

or better, lower case:

mission accomplished

I suggest that, because of this post (plus other reasons which may as well remain nameless) but an overt reason is that people might be proud of having finished their mission (writing working code), and celebrate that. The T-shirt associates that celebration with taking it to the next level (i.e. code review), and implies that the mission is even more accomplished after code review.

Also, given that "brevity is the soul of wit", like @Paweł said, "it can't be too short". ;-)

\$\endgroup\$
4
  • \$\begingroup\$ Fun fact: "Brevity is the soul of wit" is a quote from a character who just won't shut up, contradicts himself all the time, uses overly large metaphors, repeats himself constantly, and never stops talking, like a drunk man at a subway or a sober man who's so desperate for company he'll talk to the drunk guy just for a little company. \$\endgroup\$
    – Nic
    Jan 8, 2016 at 0:07
  • \$\begingroup\$ You're referring to Shakespeare, are you? ;-) \$\endgroup\$
    – ChrisW
    Jan 8, 2016 at 0:10
  • \$\begingroup\$ Yep! We just finished (re)reading Hamlet in English, and I found myself laughing my ass off (again) when Polonius says "Brevity is the soul of wit" right before launching into a two-dozen-line monologue that could have been summed up as "Yep, Hamlet's crazy. With love for my daughter." \$\endgroup\$
    – Nic
    Jan 8, 2016 at 0:11
  • \$\begingroup\$ Paweł said it can't be too short, but I think it could be shorter. \$\endgroup\$
    – ChrisW
    Jan 8, 2016 at 0:14
0
votes
\$\begingroup\$

We don't just commit and deploy

\$\endgroup\$
1
2

You must log in to answer this question.

Not the answer you're looking for? Browse other questions tagged .