Fix That Code

collaborative code refactoring

Coding is an art. There are many ways to paint an apple, but which is better? Most of the code we write everyday can be written in better ways to achieve different objectives. Sometimes all it takes is a fresh pair of eyes. Code quality can be observed through various identifying markers. Perhaps you're not aware of them, don't have time to look for them or are too caught up in the moment to notice?

That's where FixThatCode.com steps in! Its a place where you can get feedback on your code, challenge other developers to improve it, work out ideas, in general just collaboratively discuss or rewrite a piece of code. It allows you show pieces of bad code you find, but to also fix them and teach people how to avoid repeating those mistakes.

Come on in and help us write better code!

You can get help in these categories:

Make this code better

How can this code be improved? Readability, performance, architecture, all aspects.

Make this code faster

This code needs to fly, looking good does not matter, it just needs to be fast, how can it be improved?

Coding Horror, be scared

We see things that should never be done or repeated, these are good examples, teach people how to fix them.

Give me feedback

What's good or bad about this code? People trying to get a general idea of what their code looks like.

I dare you to improve this

I think this code is awesome, I challenge you to prove otherwise.

Latest Code Entries

Make this code better

Making deck class in python 0 reactions

Hey guys I am trying to make a class that is called deck the class have the following methods __init__ – a constructor that creates a new 52 card deck dealCard removes a random card from the deck. cardsLeft returns the number of cards remaining in the deck and the deck should use a dictionary t

CaVeMaN about 24 days ago
Make this code better

surf ads frame not work with all browsers 2 reactions

Hi i have PTC script,but when the member want surf ads with some browsers thats ads countdownt and when reach to 0 it is enter in infinit loop you can see the surf ads bar in http://alhawari.host-ed.me

Moalhawari about 28 days ago
Coding Horror, be scared

Class is not abstract and doesn't override compareTo in comparable interface? 1 reactions

this code is supposed to compare 3 ints and return the largest using the comparable interface. what am i doing wrong?

Persian Version about 2 months ago
Give me feedback

Post.template.php 0 reactions

<?php /** * Simple Machines Forum (SMF) * * @package SMF * @author Simple Machines * @copyright 2011 Simple Machines * @license http://www.simplemachines.org/about/smf/license.php BSD * * @version 2.0 */ // The main template for the post page. function template_main() { glo

Orianthos about 5 months ago
Give me feedback

Post.template.php 0 reactions

<?php /** * Simple Machines Forum (SMF) * * @package SMF * @author Simple Machines * @copyright 2011 Simple Machines * @license http://www.simplemachines.org/about/smf/license.php BSD * * @version 2.0 */ // The main template for the post page. function template_main() { glo

Orianthos about 5 months ago