Problema para leer mensajes en php

[luisochonews]

Lo que pasa es de que hice un sistema de usuarios con mensajes privados, y los usuarios pueden ver una lista de mensajes recibidos en una bandeja de entrada, pero el problema es que cuando un usuario quiere ver y seleccionar algun mensaje que le enviaron, resulta que ocurre un error que dice que "No puedes leer los mensajes privados de otras personas". Aquí está un FRAGMENTO del código php porque AQUÍ ocurre el problema. Ojalá se resuelva pronto...:

Código PHP:

if($_GET[msg]) {
if(!$_GET[tipo]) { $ver=" destinatario='$_COOKIE[nick]'"; } else { $ver=" remitente='$_COOKIE[nick]'"; 
$com=mysql_query("SELECT * FROM mensajes WHERE id='$msg' and $ver");}
if(mysql_num_rows($com) == 0) {
echo "No puedes leer los mensajes privados de otras personas";
}
 

Quien quiera ver el código completo, aquí les pongo un link de descarga directo para que lo descarguen y lo revisen, EL ERROR ESTÁ A PARTIR DE LA LÍNEA 64:
https://dl.dropbox.com/u/69007781/mensajes.php

[Triby]

Iniciado por luisochonews

Les pondré un +karma si me ayudan :)

Y yo me abstrendré de darte un -karma si en lugar de agregar un mensaje editas el anterior para evitar ser sancionado por suma de mensajes y reflotar tema sin un aporte real.

Primero, siempre que uses un array asociativo, los índices deben estar entre comillas:

Mal: $_GET[msg]
Bien: $_GET['msg']

Segundo, siempre que uses un elemento de array dentro de una cadena, debe estar entre llaves:

Mal: " destinatario='$_COOKIE[nick]'" // Faltan llaves y comillas en el índice
Mal: " destinatario='{$_COOKIE[nick]}'" // Faltan comillas en el índice
Bien: " destinatario='{$_COOKIE['nick']}'"

Tercero, cuando ejecutes una consulta, asegúrate de que no hay errores:

$com = mysql_query(/* aquí tu consulta */) or die(mysql_error());

Cuarto, si usas datos manupulables por el usuario ($_GET, $_POST, $_COOKIE, etc.) verifícalos y escápalos para evitar problemas con tu base de datos, por ejemplo:

- Si esperas un dato numérico, asegúrate que sea así, lo más fácil es multiplicarlo por 1: $id_msg = $_GET['msg'] * 1;
- Si esperas un dato alfanumérico (cadena, texto, etc.) debes escaparlo antes de incluirlo en la consulta: $nick = mysql_real_escape_string($_COOKIE['nick']);

Quinto, tú eres el interesado en recibir ayuda, entonces deberías proporcionar toda la información necesaria directamente aquí en lugar de poner un enlace para descarga.

Finalmente, corrige tu código y si sigues teniendo problema, entonces regresas y nos cuentas qué es lo que pasa.

[luisochonews]

Iniciado por Triby

Y yo me abstrendré de darte un -karma si en lugar de agregar un mensaje editas el anterior para evitar ser sancionado por suma de mensajes y reflotar tema sin un aporte real.

Primero, siempre que uses un array asociativo, los índices deben estar entre comillas:

Mal: $_GET[msg]
Bien: $_GET['msg']

Segundo, siempre que uses un elemento de array dentro de una cadena, debe estar entre llaves:

Mal: " destinatario='$_COOKIE[nick]'" // Faltan llaves y comillas en el índice
Mal: " destinatario='{$_COOKIE[nick]}'" // Faltan comillas en el índice
Bien: " destinatario='{$_COOKIE['nick']}'"

Tercero, cuando ejecutes una consulta, asegúrate de que no hay errores:

$com = mysql_query(/* aquí tu consulta */) or die(mysql_error());

Cuarto, si usas datos manupulables por el usuario ($_GET, $_POST, $_COOKIE, etc.) verifícalos y escápalos para evitar problemas con tu base de datos, por ejemplo:

- Si esperas un dato numérico, asegúrate que sea así, lo más fácil es multiplicarlo por 1: $id_msg = $_GET['msg'] * 1;
- Si esperas un dato alfanumérico (cadena, texto, etc.) debes escaparlo antes de incluirlo en la consulta: $nick = mysql_real_escape_string($_COOKIE['nick']);

Quinto, tú eres el interesado en recibir ayuda, entonces deberías proporcionar toda la información necesaria directamente aquí en lugar de poner un enlace para descarga.

Finalmente, corrige tu código y si sigues teniendo problema, entonces regresas y nos cuentas qué es lo que pasa.

Podrás creer que con todas estas correcciones que me mencionastes sigue exactamente igual, con el mismo error, creo que estaba bien como estaba, pero tal vez tiene que HABER OTRA CORRECCIÓN, eso es lo que estoy buscando.

Gracias por el aporte, me sirvio de aprendizaje, pero para esto NO FUNCIONO :(

[Triby]

Iniciado por luisochonews

Podrás creer que con todas estas correcciones que me mencionastes sigue exactamente igual, con el mismo error (1), creo que estaba bien como estaba (2), pero tal vez tiene que HABER OTRA CORRECCIÓN (3), eso es lo que estoy buscando (4).

1- Por supuesto que te creo, porque el codificar correctamente no soluciona problemas de lógica, esos son aparte.

2- Como estaba no funcionaba, pero si no descartas errores por mala codificación, es más difícil encontrar la solución.

3- Sí, eso es definitivo, pero como te decía, si no codificas bien, más te vas a complicar.

4- Me parece bien, sigue buscando, te deseo suerte.

[luisochonews]

Por eso creé este tema PARA QUE ME AYUDEN Y TENGAN EL ESPÍRITU DE SERVICIO DE ENCONTRAR UN ERROR QUE AÚN NO PUEDO ENCONTRARLO, para mí es muy difícil, por eso necesito ayuda de otros que sepan sobre este tipo de errores

[Triby]

Mira, en resumen: Si tú no estás dispuesto a hacer las cosas bien, veo muy complicado que alguien quiera invertir tiempo tratando de ayudarte, porque al final vas a decir "eso es solución, tiene que haber otra forma".

Me retiro del tema deseándote suerte nuevamente.

[Carloosolrac]

Por qué cierras el ELSE después de la consulta?.

[KsrZ]

luis.. lo ideal es que fueras mas ordenado con el codigo(usa tabulaciones), y evitar las cosas repetidas Ejemplo:

Código PHP:

  $datos[mensaje] = str_replace("[b]","<b>",$datos[mensaje]);
$datos[mensaje] = str_replace("[/b]","</b>",$datos[mensaje]);
$datos[mensaje] = str_replace("[hr]","<hr>",$datos[mensaje]);
$datos[mensaje] = str_replace("[i]","<i>",$datos[mensaje]);
$datos[mensaje] = str_replace("[/i]","</i>",$datos[mensaje]);
$datos[mensaje] = str_replace("[sub]","<u>",$datos[mensaje]);
$datos[mensaje] = str_replace("[/sub]","</u>",$datos[mensaje]);
$datos[mensaje] = str_replace("[marquee]","<marquee>",$datos[mensaje]);
$datos[mensaje] = str_replace("[/marquee]","</marquee>",$datos[mensaje]);
$datos[mensaje] = str_replace("[left]","<div align=left>",$datos[mensaje]);
$datos[mensaje] = str_replace("[/left]","</div>",$datos[mensaje]);
$datos[mensaje] = str_replace("[center]","<div align=center>",$datos[mensaje]);
$datos[mensaje] = str_replace("[/center]","</div>",$datos[mensaje]);
$datos[mensaje] = str_replace("[right]","<div align=right>",$datos[mensaje]);
$datos[mensaje] = str_replace("[/right]","</div>",$datos[mensaje]);
$datos[mensaje] = str_replace("[justify]","<div align=justify>",$datos[mensaje]);
$datos[mensaje] = str_replace("[/justify]","</div>",$datos[mensaje]);
$datos[mensaje] = str_replace("[img]","<img src=\"",$datos[mensaje]) ;
 
//1 esta mal, va entre comillas ['mensaje'] para que todas esas repeticiones??? 
//si puedes hacer esto..
$busca = array('[b]','[/b]','[hr]','[i]','[/i]'); //etc
$remplaza = array('<b>','</b>','<hr>','<hr>','<i>','</i>'); //etc
$datos['mensaje'] = str_replace($busca , $remplaza , $datos[mensaje]); 
   

un texto como key en un array recuerda siempre que utiliza comillas, debes tratarlo como texto, no como una constante...
un numero si no necesita comillas.

Código PHP:

  $a[1]; //bien
$a[b]; //si b no es una constante previamente definida, esta mal
$a['b']; //si b es una key esta bien, si es una constante, esta mal 
   

otra recomendación.. para evitarte tanto if else solo para imprimir algo, aprende a usar expresiones ternarias
http://php.net/manual/es/language.op...hp#example-120

ten cuidado si las usas, aplica bien (), ejemplo en un echo

Código PHP:

  echo ($bar === true)? 'bar es true!' : (($bar2 === true)? 'bar2 es true!' : 'ambas son falsas..');
 
//esto equivale a
 
if($bar === true) echo 'bar es true!';
else if ($bar2 === true) echo 'bar2 es true!';
else echo 'ambas son falsas..'; 
   

a medida que cambias tu código, ve actualizándolo para poder ver el "editado nuevo" y no el viejo, sino es imposible ayudarte en base a el mismo código.