WingNews logo WingNews GitHub [2]
top | item 9427532

Is My Code Good?

29 points| jlsync | 11 years ago |ismycodegood.com | reply

38 comments

order
[+] ThatOtherPerson|11 years ago|reply
Read the bar at the bottom:

    Lovingly (and satirically) created by your friends at test double. Designed by Derek Briggs.
It's not meant to be serious, although I'm not sure what the point is.

Edit: It seems to be (perhaps unintentionally) clickbait for programmers. We can't seem to resist visiting a website that claims to rate our code quality.

[+] madaxe_again|11 years ago|reply
This snippet sheds some light:

b=["Thoughtfulness of Names","Expressiveness of Tests","Empathy for Maintainers","Future-proof Avoidance","Conscientious Logging","Commit Message Clarity","Dependency Restraint","Release Strategy","Coherent Versioning","Convention Adherance","Pairing — Navigation","Pairing — Driving","Expectation Management","Googling for Answers","Respectful of Others"],n=function(){return _(b).chain().shuffle().first(5).value()},p=function(b){return _(a).include(b)?[96,98]:[0,40,60,70,80,85,90]},o=function(a){return null==a&&(a=[0]),_(a).chain().map(function(a){return _.random(a,100)}).sample().value()},c={59:"F",63:"D-",66:"D",69:"D+",73:"C-",76:"C",79:"C+",83:"B-",86:"B",89:"B+",93:"A-",96:"A",100:"A+"}

["pixeljanitor","bkeepers","tkaufman","searls","jasonkarns","andrewvida","theotherzach","bostonaholic","davemo","neall","kbaribeau","danthompson","crebma","dustintinney"]

Those guys all get an A+.

YHBT!

[+] catshirt|11 years ago|reply
"although I'm not sure what the point is"

satire, clearly.

wether or not it's good satire is up for debate.

[+] atmosx|11 years ago|reply
If only rating code could be done that easily...
[+] madaxe_again|11 years ago|reply
"Tell the world you're a C+ developer!" (sic)

it tells me, after reviewing my code, and saying my non-existent tests are excellent.

I think I'll demur.

Edit: looking at the source, C+ is meant to be the grade, not the language, which makes more sense, even if this is just a fun farce.

[+] tericho|11 years ago|reply
C+ is a language?
[+] AdmiralAsshat|11 years ago|reply
Is there somewhere which gives an explanation of what I'm being graded on and why a given grade was assigned?
[+] nchelluri|11 years ago|reply
That's totally key IMO; even a link to the grading code would go some distance to that goal.

EDIT: Ok, I'm starting to think this whole site is some kind of a joke.

[+] k__|11 years ago|reply
This.

I mean, Googling for Answers?!

[+] hodgesmr|11 years ago|reply
Looks like the output is random when you incognito.
[+] kbenson|11 years ago|reply
I have a feeling it's not serious.

> Lovingly (and satirically) created by your friends at...

[+] jjar|11 years ago|reply
This seems to be part of a new trend of HN Clickbait...

Next thing you know Buzzfeed will try to buy the site.

[+] searls|11 years ago|reply
Hey HN, I actually made this site as a companion to my RailsConf talk yesterday (I'm sure it was humorously confusing out of context).

The talk's slides are up here: https://speakerdeck.com/searls/sometimes-a-controller-is-jus...

And video here (better versions coming soon): https://youtu.be/LdWMcs9EEOE?t=2h56m

Humorously I joked about it hitting HN during the talk, and then it actually did. We as developers can't resist something that'll quantify for us whether we're good at writing software, even when we know it's flawed.

[+] naoru|11 years ago|reply
I was expecting a big "NO".
[+] DaveSapien|11 years ago|reply
Nice, I'm an A class Developer! My secret: do very little on GitHub.
[+] BukhariH|11 years ago|reply
It's completely randomly generated.

Hint: If you type in a GitHub username that doesn't exist like "IBetThisUserDoesNotExist" you still get a grade :)

[+] dimman|11 years ago|reply
This tool makes as much sense as most project managers I've met. (It doesn't, it's just a joke)
[+] jonahx|11 years ago|reply
torvalds

Overall grade: C

Commit Message Clarity - F

Conscientious Logging - B+

Release Strategy - B-

Expressiveness of Tests - B+

Respectful of Others - D-

[+] simlevesque|11 years ago|reply
Release Strategy, Future-proof Avoidance and Commit Message Clarity have nothing to do with determining if the code is good.

You should change the name of the website to something like : Are you a good contributer ?

edit : Oh it's a joke ? ha. ha. ha.

[+] tux3|11 years ago|reply
Those are funny sometimes. I like the "Empathy for Maintainers" one.

Torvalds:

Empathy for Maintainers: C-

Future-proof Avoidance: C-

Conscientious Logging: A+

Googling for Answers: B

Commit Message Clarity: B-

https://i.imgur.com/Lwu64Ms.jpg

[+] jnem|11 years ago|reply
I got a B+ and Im pretty sure Ive never published anything with my GitHub account.
[+] fyolnish|11 years ago|reply
Thanks for complementing me for my tests? (That I never wrote)
[+] infinitone|11 years ago|reply
Definitely random as I got an A- and trovalds got a C o_O