rob-knight
commented
10 years ago I like not this code. Bring me some other code…
On Feb 20, 2014, at 12:37 PM, adamrp notifications@github.com<mailto:notifications@github.com> wrote:
MIND = BLOWN
Started innocently enough.... All I wanted to do was strip newlines out of some database fields before they were written to a mapping file, which led me to this code.
I'm not even going to mention the fact that there are two NEARLY identical functions in this file (aside from the names of the functions, which are also nearly identical, I challenge you to spot the differences without using a diff tool). Or the fact that WTF is going on in the first half of each function is almost inscrutable. Instead I would like to highlight lines 383https://github.com/qiime/qiime_web_app/blob/master/python_code/write_mapping_file.py#L383 through 433.
This is no longer an issue I intend to fix (since replacing rather than fixing this code would be a better use of time), I just wanted to invite anyone reading this post to scrutinize these 50 lines of code and count all the crazy.
— Reply to this email directly or view it on GitHubhttps://github.com/qiime/qiime_web_app/issues/633.
adamrp
ElDeveloper
MIND = BLOWN
Started innocently enough.... All I wanted to do was strip newlines out of some database fields before they were written to a mapping file, which led me to this code.
I'm not even going to mention the fact that there are two NEARLY identical functions in this file (aside from the names of the functions, which are also nearly identical, I challenge you to spot the differences without using a diff tool). Or the fact that WTF is going on in the first half of each function is almost inscrutable. Instead I would like to highlight lines 383 through 433.
This is no longer an issue I intend to fix (since replacing rather than fixing this code would be a better use of time), I just wanted to invite anyone reading this post to scrutinize these 50 lines of code and count all the crazy.