PunBB Forums — Code repetition

Re: Code repetition

null@example.com (Adam Atlas) — Mon, 03 Mar 2008 23:00:21 +0000

OP was exaggerating. There's plenty of foreach in the code, when it makes sense to use it.

Re: Code repetition

null@example.com (Strofanto) — Mon, 03 Mar 2008 22:48:21 +0000

Out of curiosity, why you don't use foreach?

Re: Code repetition

null@example.com (Smartys) — Sun, 02 Mar 2008 23:02:53 +0000

FYI, I'm not sure whether you meant it to or not, but your post comes off sounding very rude.

I personally don't see any particular advantage to doing things that way in the specific instance you're mentioning (especially since your example actually takes more lines then what is required otherwise). What are the advantages?

Code repetition

null@example.com (orlandu63) — Sun, 02 Mar 2008 22:53:49 +0000

There are many instances of code repetition found in PunBB. Are you guys not aware of it or what? Like in changeset 1540 (http://dev.punbb.org/changeset/1540), you can do:

$basename = $pun_config['o_avatars_dir'].'/'.$id;
foreach(array('gif', 'jpg', 'png') as $ext) {
 $file = "basename.$ext";
 if(file_exists($file)) {
  @unlink($file);
 }
}

Oh wait, I forgot you guys don't use foreach()