Thunderrabbit: terrible code I found at work

2008-06-03T06:01:36Z

terrible code I found at work

New page

3 June 2008

Bad news: broken chdir

[[Image:warning chdir- no such file etc on line 283.png]]

Worse news: continuing after broken chdir

[[Image:file list after not checking broken chdir on line 283.png]]

I saw the above, so I was like, WTF? and went to find the code below, which runs on our live server. The same code runs on our test server which uses different data. And runs on our other test server which uses a different directory structure.

<source lang="php">
<?php

// snip

$cwd = getcwd();
chdir('/var/apache/ssl/htdocs/admin/csv/user/');
$fileList = array();
$dateList = array();

if ($dir = opendir(".")) {
$i = 0;
while (false !== ($file = readdir($dir))) {
if ($file != "." && $file != "..") {
$fileList[$i] = $file;
$dateList[$i] = date("Y/m/d H:i:s", fileatime($file));
$i++;
}
}
closedir($dir);
}
chdir($cwd);

// snip

?>
</source>

There are some redeeming qualities to this code:

# it's indented per block level
# it doesn't allow . nor .. to be processed
# it closes its file handle $dir
# it saves and restores its working directory after it's done.

There are also, however, a couple of points that are not as redeeming.

# hard coding '/var/apache/ssl/htdocs/admin/csv/user/'
# not checking to see if the hard code worked

In preparing to fix them, I found this, or should I say ''these'':

[[Image:omg a whole slew of hardcoded paths.png]]

Programming:Terrible code - Revision history

Thunderrabbit: terrible code I found at work