0

Tengo este codigo php

    $longitud = count($emotions);
    for ($i=1; $i<$longitud; $i++) {
    echo '
    <img src="images/emojis/emoji'.$i.'.png" 
    onclick="changeValue('.$emotions[$i].')" alt="Emoji" id="emojier" >
    ';
     } 

y este javascript(esta en la misma pagina)

function changeValue(value){
 $("#texter").val($("#texter").val()+value);
}

Lo que hace esto es que añade el valor que le envio a un input, el valor es el del array[i] lo que no entiendo es por que no funciona si no da error, si cambio el valor que envio a "$i" funciona todo correctamente, entonces no entiendo. Por favor, ayudenme. Gracias.

4
  • 1
    Que es lo que contiene el array $emotions?
    – alanfcm
    el 21 ene. 2020 a las 18:08
  • que tal si cambias tu cadena: "<img src='images/emojis/emoji$i.png' onclick='changeValue('$emotions[$i]')' alt='Emoji' id='emojier' >"
    – Shassain
    el 21 ene. 2020 a las 18:13
  • 1
    Usar funciones in line es una mala práctica, no se respeta el principio de separación de código. Si mañana tu función changeValue() necesita manejar otro parámetro y la tienes in line dentro de 500 o 5,000 archivos, vas a tener que empezar a buscar todos esos archivos para agregar ese nuevo parámetro. Otro problema menor, que es lo que te pasa a ti aquí, es que se crea un código totalmente confuso con tantas comillas e historias que hacen el código complicado. Es mejor escuchar los elementos desde Javascript y lanzar desde allí tu función, así el HTML queda totalmente independiente.
    – A. Cedano
    el 21 ene. 2020 a las 18:23
  • No ha funcionado nada y el array $emotions contiene una larga lista de strings, es decir estan entre "comillas" todos y separados por coma.
    – Laan
    el 21 ene. 2020 a las 23:29

1 respuesta 1

0

El problema es que le estás pasando un valor que no está encerrado entre comillas y entonces no lo toma como string. Prueba con esto:

$longitud = count($emotions);
for ($i=1; $i<$longitud; $i++) {
echo '
<img src="images/emojis/emoji'.$i.'.png" 
onclick="changeValue(\''.$emotions[$i].'\')" alt="Emoji" id="emojier" >
';
 } 
2
  • No funciona, da igual.
    – Laan
    el 21 ene. 2020 a las 23:30
  • @Laan Perdón, eso daría un error por usar comillas dobles, ahí lo modifiqué por comillas simples el 22 ene. 2020 a las 14:39

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.