0

intentando imprimir código HTML, concatenando las variables dentro una funcion onClick. No consigo ordenar bien las comillas para que funcione.

echo  "<button onclick='myFunction('" . $_id . ",'" . $_tipus . "'" . "," . "'" . $_desc . ")'  style='width:10px;border:none;margin-right:50px;'>
      <img src=" . plugin_dir_url( __FILE__ ) . 'img/modify.png' . " style='width:25px;margin-right:20px;' alt='Submit Form'>
</button>"; 

OUTPUT:

<button onclick="myFunction(1," despatx','la="" reserva="" serà="" anual')'="" style="width:10px;border:none;margin-right:50px;">

OUTPUT DESEADO:

<button onclick="myFunction(1,'despacho','la reserva sera anual')"  style="width:10px;border:none;margin-right:50px;">
1
  • Y si intentas usar otro tipo de comillas para que no te martilices la vida concatenando cadenas? Puedes usar este tipo de comillas ``. En las veces que lo he usado me ha ayudado a escapar tanto las comillas simples como las dobles.
    – LB17
    Commented el 7 may. 2022 a las 22:26

1 respuesta 1

1

Lo más simple es que uses printf, que te permite poner un marcador donde irían los valores, pasando los valores al final.

Algo así más o menos:

printf("<button onclick=\"myFunction(%s,%s,%s)\") style=\"width:10px;border:none;margin-right:50px;\">
      <img src=\"%simg/modify.png\" style=\"width:25px;margin-right:20px;\" alt=\"Submit Form\">
</button>",$_id,$_tipus,$_desc,plugin_dir_url( __FILE__ )); 

Si quieres guardar para concatenar con otra cosa, puedes usar sprintf, que funciona del mismo modo, con la diferencia de que guardarías el resultado en una variable.

Para más detalles consulta la documentación.

Más sobre comillas y concatenaciones

En el código he implementado una práctica que me parece la más coherente, consiste en usar comillas dobles en todo momento. Esto tiene una gran ventaja: que puedes poner variables dentro de la cadena. Esto es útil en casos en que no uses printf o sprintf. En ese caso, las comillas dobles que sean parte de esa cadena las puedes escapar de este modo: \".

Por ejemplo, podrías hacer esto:

$redClass="button-red";
$str="<button id=\"btnOne\" class=\"$redClass\">Botón rojo 1</button>";
// ... más código
$str.="<button id=\"btnTwo\" class=\"$redClass\">Botón rojo 2</button>";
echo $str;

Para más detalles sobre esto consulta la pregunta: Imprimir variables y cadenas en PHP, comillas simples o dobles.

Advertencia sobre malas prácticas

En tu código estás usando dos malas prácticas que deberías evitar:

  • Las funciones in-line se consideran una mala práctica
  • Poner código CSS in-line también se considera una mala práctica, es mejor usar clases y definir las reglas de estilo en un archivo .css.
2
  • Muchas gracias!! Lo tendré en cuenta!
    – Aleix Roca
    Commented el 8 may. 2022 a las 10:04
  • @AleixRoca revisa la respuesta editada. He agregado otra información y un enlace a otra pregunta que podría ser de utilidad en un futuro. Saludos.
    – A. Cedano
    Commented el 8 may. 2022 a las 10:21

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.