Foros del Web » Programando para Internet » PHP »

[SOLUCIONADO] ¿Cómo sería este PHP de forma correcta?

Estas en el tema de ¿Cómo sería este PHP de forma correcta? en el foro de PHP en Foros del Web. Código PHP: <?php // Primero comprobamos que ningún campo esté vacío y que todos los campos existan. if(isset( $_POST [ 'titulo' ]) && !empty( $_POST [ 'titulo' ]) && isset( $_POST [ 'contenido' ]) && !empty( $_POST [ 'contenido' ])) { // Si entramos es que todo se ha realizado correctamente $link  =  ...
  #1 (permalink)  
Antiguo 23/12/2014, 10:37
 
Fecha de Ingreso: diciembre-2014
Ubicación: Madrid
Mensajes: 274
Antigüedad: 9 años, 4 meses
Puntos: 5
¿Cómo sería este PHP de forma correcta?

Código PHP:
<?php

// Primero comprobamos que ningún campo esté vacío y que todos los campos existan.
if(isset($_POST['titulo']) && !empty($_POST['titulo']) &&
isset(
$_POST['contenido']) && !empty($_POST['contenido'])) {

// Si entramos es que todo se ha realizado correctamente

$link mysql_connect("localhost","usuario","contraseña");
mysql_select_db("webdb",$link);

// Con esta sentencia SQL insertaremos los datos en la base de datos
mysql_query("INSERT INTO noticias (titulo,contenido)
VALUES ('{$_POST['titulo']}','{$_POST['contenido']}')"
,$link);

// Ahora comprobaremos que todo ha ido correctamente
$my_error mysql_error($link);

if(!empty(
$my_error) {

echo 
"Ha habido un error al insertar los valores. $my_error";

} else {

echo 
"Los datos han sido introducidos satisfactoriamente";

}

} else {

echo 
"Error, no ha introducido todos los datos";

}

?>
  #2 (permalink)  
Antiguo 23/12/2014, 10:39
 
Fecha de Ingreso: diciembre-2014
Ubicación: Madrid
Mensajes: 274
Antigüedad: 9 años, 4 meses
Puntos: 5
Respuesta: ¿Cómo sería este PHP de forma correcta?

Dreamweaver dice que están mal las líneas 19 y 23. ¿Cómo sería correctamente?
  #3 (permalink)  
Antiguo 23/12/2014, 10:53
Avatar de hhs
hhs
Colaborador
 
Fecha de Ingreso: junio-2013
Ubicación: México
Mensajes: 2.995
Antigüedad: 10 años, 10 meses
Puntos: 379
Respuesta: ¿Cómo sería este PHP de forma correcta?

Ya te diste cuenta que tienes 2 else en tu condición ?
http://php.net/manual/es/control-structures.else.php
__________________
Saludos
About me
Laraveles
A class should have only one reason to change.
  #4 (permalink)  
Antiguo 23/12/2014, 10:55
 
Fecha de Ingreso: octubre-2014
Ubicación: Buenos Aires
Mensajes: 278
Antigüedad: 9 años, 6 meses
Puntos: 12
Respuesta: ¿Cómo sería este PHP de forma correcta?

Hola diego por lo que veo a simple vista te esta faltando cerrar un parentesis ) en esta parte.
Código PHP:
Ver original
  1. //tu código
  2. if(!empty($my_error) {
  3.  
  4. //código correcto
  5.  
  6. if(!empty($my_error)) {


Código PHP:
Ver original
  1. <?php
  2. // Primero comprobamos que ningún campo esté vacío y que todos los campos existan.
  3. if(isset($_POST['titulo']) && !empty($_POST['titulo']) &&  isset($_POST['contenido']) && !empty($_POST['contenido'])) {
  4.  
  5.     // Si entramos es que todo se ha realizado correctamente
  6.  
  7.     $link = mysql_connect("localhost","usuario","contraseña");
  8.     mysql_select_db("webdb",$link);
  9.  
  10.     // Con esta sentencia SQL insertaremos los datos en la base de datos
  11.     mysql_query("INSERT INTO noticias (titulo,contenido)
  12.    VALUES ('{$_POST['titulo']}','{$_POST['contenido']}')",$link);
  13.  
  14.     // Ahora comprobaremos que todo ha ido correctamente
  15.     $my_error = mysql_error($link);
  16.  
  17.     // Debes corregir acá agregar cierre de parentesis " ) "
  18.     if(!empty($my_error) {
  19.  
  20.     echo "Ha habido un error al insertar los valores. $my_error";
  21.  
  22.     } else {
  23.  
  24.     echo "Los datos han sido introducidos satisfactoriamente";
  25.  
  26.     }
  27.  
  28. }else {
  29.  
  30.     echo "Error, no ha introducido todos los datos";
  31.  
  32. }
  33.  
  34. ?>

Tabula el código para que se entienda mejor donde se abre un llave y donde cierra.

Espero te sirva.

Saludos.
__________________
http://www.sp-vision.net

Etiquetas: Ninguno
Atención: Estás leyendo un tema que no tiene actividad desde hace más de 6 MESES, te recomendamos abrir un Nuevo tema en lugar de responder al actual.
Respuesta




La zona horaria es GMT -6. Ahora son las 09:24.