0

¿Qué puedo hacer para que los comentarios se queden en la página y por lo tanto sean públicos y visibles para cualquier usuario que llegue a mi página?

<form method="get" action="conexion.php">
    <input type="text" name="name">
    <textarea rows="5" cols="5" name="commento"></textarea>
    <input type="submit" value="enviar">
</form>

<?php
$conexion = mysql_connect('localhost', 'root', 'admin');
mysql_select_db('test', $conexion);

$query = "SELECT * FROM commenti";
while($fila = mysql_fetch_array($query)){
    echo $fila['nome'];
    echo $fila['commento'];
    echo $fila['fecha'];
}
?>

El error es el siguiente:

mysql_fetch_array() expects parameter 1 to be resource, string given in

1 respuesta 1

1

En primer lugar, mysql* fue declarada obsoleta en PHP 5.5.0 y eliminada en PHP 7.0.0. En su lugar debería utilizarse las extensiones MySQLi o PDO_MySQL.

También tener en cuenta, para evitar la inyección SQL, te aconsejo el uso de sentencias mysqli::prepare

Ejemplo MySQLi (Estilo por procedimientos):

//Conexión.
$conexion = mysqli_connect("localhost", "root", "admin", "test");

/* comprobar la conexión */
if (mysqli_connect_errno()) {
    printf("Falló la conexión: %s\n", mysqli_connect_error());
    exit();
}

$name = $_GET['name'];
$commento = $_GET['commento'];
date_default_timezone_set('Europe/Madrid');
$fecha= date("Y-m-d H:i:s");

$sql = "INSERT INTO commenti (nome, commento, fecha) VALUES ('$name', '$commento', '$fecha')" ;
$query = mysqli_query($conexion,$sql);
//Caso, si fallara nuestra sentencia.
if (!$query) {
    printf("Error: %s\n", mysqli_error($conexion));
}

echo $name;
echo $commento;
echo $fecha;

A tu respuesta, como obtener los comentarios para su publicación, lo podrías hacer de la siguiente manera:

//Conexión.
$conexion = mysqli_connect("localhost", "root", "admin", "test");

/* comprobar la conexión */
if (mysqli_connect_errno()) {
    printf("Falló la conexión: %s\n", mysqli_connect_error());
    exit();
}

//Sentencia.
$sql = "SELECT nome,commento,fecha FROM commenti ORDER BY fecha DESC";
//Ejecutar sentencia.
$query = mysqli_query($conexion,$sql);

//Comprobamos existencia de registros.
if(mysqli_num_rows($query)>0) {
   //Obtenemos datos desde BD.
   while($row  = mysqli_fetch_assoc($query)) {
     //Mostramos datos comentarios.
     echo $row['nome'];
     echo $row['commento'];
     //etc..
   }
} else {
   echo'0 comentarios encontrados en el sistema.';
}

En caso de querer mostrar unas cantidades, podrías utilizar LIMIT

$sql = "SELECT nome,commento,fecha FROM commenti ORDER BY fecha DESC LIMIT 10";
7
  • la verda que no entiendo lo que trae de nuevo la nueva version de php, lamento que no soporte algunas funciones, de todos modos seguire usando php5 y mysql hasta la ultima. respecto a tu solucion creo que dejar un while activo pondria lento la navegacion
    – steven
    Commented el 8 may. 2017 a las 21:41
  • @steven Dependerá de los registros (comentarios) que vayas a mostrar, siempre podrás utilizar la función LIMIT y mostrar una sola cantidad, así no mostrarías 40.000 si uno tendría tantos registros en la Base de datos o crear una paginación de PHP que sería lo ideal en ese caso. El enlace que te deje sobre cómo evitar la inyección SQL, te explica bien, porque no usar mysql, simplemente ahora mismo tu código es vulnerable. ¡Espero verte ayudado, un saludo!
    – D.Bulten
    Commented el 8 may. 2017 a las 21:59
  • interesante, nunca escuche hablar de esa funcion, no tengo miedo a injeccciones sql porque los datos de mi registro pueden ser cifrados y si en caso el quaker llegara a entrar, encontraria contenido cifrado, ademas para que pueden querer un sitio de pocas visitas
    – steven
    Commented el 8 may. 2017 a las 22:11
  • Podría quedar así $sql = "SELECT nome,commento,fecha FROM commenti ORDER BY fecha DESC LIMIT 10"; de esta manera solo te va mostrar los últimos 10 registros, he puesto 10 como si quieres poner 1 o 2 o 3, ya depende la cantidad que deseas mostrar. Respecto a la inyección SQL, es un consejo, piensa que siempre hay personas aburridas en la vida, que no miren si una web tenga o no importancia o visitas, simplemente por entrenamiento, podrán manipular fácilmente tu Base de datos y eliminar toda tu Base de datos.
    – D.Bulten
    Commented el 8 may. 2017 a las 22:23
  • muchas gracias por el consejo lo tomare en cuenta, todavia no entiendo bien la diferencia entre mysqli y pdo, ambos sirven para evitar este tipo de ataques?
    – steven
    Commented el 8 may. 2017 a las 22:26

Tu Respuesta

By clicking “Publica tu respuesta”, you agree to our terms of service and acknowledge you have read our privacy policy.

¿No es la respuesta que buscas? Examina otras preguntas con la etiqueta o formula tu propia pregunta.