Creo que a lo que @loncho_rojas se refiere es que tu tu código resulta ilegible de tantas etiquetas y tan poco orden.
Si miras con cuidado abres y cierras las etiquetas de PHP en forma encadenada... ¿para qué? Eso lo único que hace es llenar ese script de basura.
En esencia si quitamos todo lo innecesario, ese código quedaría así:
Código PHP:
Ver original<?php
$ip = $_SERVER['REMOTE_ADDR'];
$query_Recordset1 = "SELECT * FROM Fav WHERE ip = '$ip'";
$ifero = mysql_num_field($Recordset1);
do {
echo $row_Recordset1['id'];
// liberamos la conexión
if ($ifero == $id) {
echo "Ya hay un registro con los datos que pusiste!";
}
?>
...Cosa que nos resulta algo más entendible.