pasteling | "struggling intern" at 129.162.1.31 pasted "Here it is.." (220 lines, 5.7K) at http://sial.org/pbot/25279 |
strugglingintern | woohoo |
strugglingintern | there it goes... |
rindolf | strugglingintern: oh my god! |
rindolf | strugglingintern: this code looks really bad. |
strugglingintern | heh... |
rindolf | strugglingintern: if ($records eq 0 ) - don't you want ($records == 0)? |
rindolf | strugglingintern: do you have unit tests? |
rindolf | strugglingintern: and you should factor it better. |
rindolf | strugglingintern: and possibly use Template Toolkit or something. |
rindolf | strugglingintern: and you may have some HTML-injection (or XSS) problems. |
Ani-_ | rindolf: and probably SQL injection problems. |
rindolf | Ani-_: indeed! |
strugglingintern | :-/ |
Ani-_ | strugglingintern: really, ask them to review that code when they get back. |
strugglingintern | alright |
strugglingintern | I appreciate it anyway |
rindolf | strugglingintern: how long have you been programming perl? |
strugglingintern | hah, about 3 months |
strugglingintern | This isn't all my code ;) |
strugglingintern | I'll look into it |
strugglingintern | Thanks guys (and gals). |
rindolf | strugglingintern: you're welcome. |
strugglingintern | not my choice ;) |
strugglingintern | it's my 3rd week here :) |
rindolf | Amazing how much more experienced programmers can tell a code is bad from a quick glance. |
rindolf | It's a bit scary. |
Ani-_ | nothing amazing about it. It's called skill. :) |
The_SB | yeah even I can tell it by a look |
ology | It's not scary or amazing at all. |
ology | It is called experience! |
Ani-_ | rindolf: what do you find scary? |
Ani-_ | rindolf: that experienced programmers can tell it? Or the code itself? |
Ani-_ | I would disagree on the first one but agree on the later! :) |
rindolf | Ani-_: no, that I'm so experienced. |
rindolf | Ani-_: I hope I don't sound out as a snob. |
ology | narcissism is fun |
rindolf | I should get a life. |
rindolf | Get a girlfriend, go to movies. |
rindolf | Instead all I do is write Perl. |
rindolf | And chat about writing Perl. |
Ikarus | a life, tried that, didn't mix with me |