連続して通信をしてソースをとってきてそこからソースを切り抜きするみたいなことをしているのですが、
使わなくなった変数は使い回したほうがいいでしょうか?
一旦nullいれたりunsetしたほうがいいのかな?とは思うのですが冗長なのでそのまま入れています。

$src = file_get_contents('http://example.com/');
$dom = new DOMDocument();
@$dom->loadHTML($src);
〜略

$src = file_get_contents('http://next.example.com/');
$dom = new DOMDocument();

のような感じになってます。
ところどころ違う処理が入るのでループしてるわけではありません。

あと
$a = "hoge";
$a = strtoupper($a);
$a = str_replace("O","A");

みたいなことは普通にやってもいいですよね?