php - 単純なリファクタリング

Question

$ filename値を割り当てるための行が1行しかないように、このコードをリファクタリングするにはどうすればよいですか。

$i = 1;
$filename = $config->PHOTO_PATH . $list->ID . "_" . $i . ".jpg"
while(file_exists($filename)) 
{
    doSomething();
    $i++;
    $filename = $config->PHOTO_PATH . $list->ID . "_" . $i . ".jpg";
}

Answer 1

$i = 1;

$ok=true;
while($ok) 
{
    $filename = $config->PHOTO_PATH . $list->ID . "_" . $i++ . ".jpg";
    $ok=file_exists($filename)
    if ($ok){
       doSomething();
    }
}

Answer 2

$filename = 'prefix' .
            (preg_replace('/prefix([0-9]+)suffix/', '$1', array_pop(glob('prefix*suffix'))) + 1) .
            'suffix';

Answer 3

次のコードを置き換えるだけです。

$i++;
$filename = $config->PHOTO_PATH . $list->ID . "_" . $i . ".jpg";

これのために：

$filename = $config->PHOTO_PATH . $list->ID . "_" . ++$i . ".jpg";

Answer 4

$i = 1;
do {
    doSomething();
    $filename = sprintf('%s%s_%d.jpg', $config->PHOTO_PATH, $list->ID, $i++);
} while (file_exists($filename));