>>250
問題なさそうだけどな
$userlines = file($userlist); の後に
print_r($userlines);
で確認

質問とは関係ないが正規表現を使わないならsplitよりexplodeのほうがはるかに高速だとマニュアルには書いてある
それとLocationするとクライアントとの通信が無駄なので
require $ok; exit;
の方がいいよ。
それと変数だけを" "で囲むのも無駄
print $user_id[$i];
if($user_id[$i] == $id){
でおk