¿Seria necesario hacer el if?

[Strings]

En el siguiente codigo seria necesario crear un if?

Código PHP:

  $news = $connection->prepare("INSERT INTO news(id,user,title,content,date,ip) VALUES ('', '".$_SESSION['username']."', :title, :content, '".date('H:i:s d/m/Y')."','".$_SERVER['REMOTE_ADDR']."')");
$news->bindParam(':title',$_POST['title']);
$news->bindParam(':content',$_POST['content']);
$news->execute();
if($news->rowCount() > 0):
    echo 'Noticia creada correctamente';
else:
    echo 'Error al crear la noticia';
endif;  
   

[pateketrueke]

¿Necesario para qué?

Por cierto, tu código está mal, si vas a usar bindParam() deberías hacerlo igual para las demás variables de sesión, fecha e IP.

Es decir, bindParam() no está limitado a variables de formulario únicamente, sólo piensa que toda variable debería estar limpia sí o sí.

Un ejemplo de fragilidad en tu código es que se puede falsear el valor del REMOTE_ADDR desde una cabecera, así que tu consulta no está bien protegida.

[Strings]

No sabia que se podria falsear. Muchas gracias

[Strings]

Aunque la fecha la puedo dejar sin filtrar no?

[pateketrueke]

Sí claro, pero por consistencia todo lo deberías manejar con bindParam(), es bueno separar responsabilidades.

La idea es que la consulta sea sólo eso y no más, eso facilitará su lectura y modificación, de otra forma habría que estar lidiando siempre con la concatenación, etc.

Que no se trate de una variable sensible no evita que metas código mal.

[Strings]

Gracias, tu cuando creas un insert into creas un if para saber si lo ha ingresado correctamente?

[pateketrueke]

Claro, validar es siempre un requerimiento.

[Strings]

Claro, pero si lo as filtrado bien, siempre se tiene que ingresar correctamente, entonces ese if creo que es innecesario

[jonni09lo]

Olvidas una regla fundamental en programación TODO es RELATIVO, nada te determina que te funcionará siempre al 100%; Además si sigues la regla de oro de "Nunca confíes en el usuario" siempre verificarás todo, ya que a parte de ser un buen estándar, te curas siempre en salud y evitas posibles puntos débiles en tu aplicativo

Saludos