[SOLUCIONADO] Arrays sin guión al principio

[mikehove]

Hola amigos de esta comunidad productiva. Les ruego su ayuda porque ya estoy quedando pelado tanto reventarme para ver cómo poder ordenar estos arrays a mi antojo.

El tema es que tengo esta función:

Código PHP:

  function get_tree($id, $level = 0)
{
    $result = mysql_query('SELECT id, label FROM menu WHERE parent='.$id);
    while ($row = mysql_fetch_array($result))
    {
        echo str_repeat('-', 1), $row['label'], "\r\n";
    
        get_tree($row['id'], $level + 1);
    }
}
 
echo get_tree(6,1); 
   

como resultado (por ejemplo):

- Archive - Snippet

El asunto es que necesito que el resultado salga de esta manera: (sin el guión al principio):

Archive - Snippet


Ya probé con implode()... No sé cómo solucionarlo.

[guardarmicorreo]

Iniciado por mikehove

Hola amigos de esta comunidad productiva. Les ruego su ayuda porque ya estoy quedando pelado tanto reventarme para ver cómo poder ordenar estos arrays a mi antojo.

El tema es que tengo esta función:

Código PHP:

  function get_tree($id, $level = 0)
{
    $result = mysql_query('SELECT id, label FROM menu WHERE parent='.$id);
    while ($row = mysql_fetch_array($result))
    {
        echo str_repeat('-', 1), $row['label'], "\r\n";
    
        get_tree($row['id'], $level + 1);
    }
}
 
echo get_tree(6,1); 
   

como resultado (por ejemplo):

- Archive - Snippet

El asunto es que necesito que el resultado salga de esta manera: (sin el guión al principio):

Archive - Snippet


Ya probé con implode()... No sé cómo solucionarlo.

prueba así

Código PHP:

function get_tree($id, $level = 0)
{
    $result = mysql_query('SELECT id, label FROM menu WHERE parent='.$id);
    while ($row = mysql_fetch_array($result))
    {
        echo $row['label'].str_repeat('-', 1)."\r\n"; //si necesitas salto de linea además de \r\n, para que se vea en html agrégale <br>
    
        get_tree($row['id'], $level + 1);
    }
}
 
echo get_tree(6,1);
 

[mikehove]

Buenas "Guardarmicorreo". Gracias por responder.

Probé y su resultado fue:

Código PHP:

  Archive- Snippet- 
   

Sí funcionó sacarle el guión del principio. Buenísimo!

¿Habrá alguna manera para evitar el último guión? Quedando así:

Código PHP:

  Archive- Snippet 
   

[bathorz]

Prueba implode()

Código PHP:

$array=array('key1'=>'valor1','key2'=>'valor2','key3'=>'valor3');
 
echo implode('-',$array);
 

Rdo:
valor1-valor2-valor3

[mikehove]

Pruebo de este modo:

Antes probando ese método no tuve éxito.

Ahora pruebo así:

Código PHP:

   echo $row['label'].implode('-',$row); 
   

y su resultado es:

Código PHP:

  Archive8-8-Archive-ArchiveSnippet9-9-Snippet-Snippet 
   

Logra el objetivo de que no aparezca el guión al principio ni al final, pero la estoy errando.

¿Sabes qué? Me confunde el hecho de trabajar con la BD. Quizás esté fallando la variable dentro de esa línea.

[pateketrueke]

Claro que la estás errando, ¿si sabes lo que hace implode() exactamente?

Dicha función sirve para unir valores de un array mediante un separador:

Código PHP:

// MAL
while (/* ... */) {
  echo 'bar'.implode('-', $foo);
}
 
// BIEN
$foo = array();
 
while (/* ... */) {
  $foo []= 'bar';
}
 
echo implode(', ', $foo); // bar, bar, bar
 

¿Se entiende?

Ahora, tu necesitas imprimir una serie de valores separados por un guión entonces implode() es exactamente lo que necesitas, pero hay un inconveniente: al parecer no sabes usar arrays.

Ese es el más grave de tus problemas, puedes usar las funciones incorrectas todo el tiempo que quieras, si no comprendes lo que tienes que hacer ni lees el manual de las funciones que usas, ¿cómo sabes que las estás usando correctamente?

El que trabajes con datos de la BD es lo más trivial del asunto, el hecho es que ni siquiera sabes trabajar correctamente con variables de tipo array + strings.

La clave consiste en no imprimir los valores, sino acumularlos en un array para después, al final del ciclo, imprimirlos usando implode().

Pon atención en eso.

[guardarmicorreo]

mikehove, pateketrueke ha sido lo suficientemente claro, voy a ir más allá planteándotelo desde otra perspectiva.

normalmente, cuando quieres imprimir texto estático dentro de un bucle junto a una variable, en tu caso usando while, se hace así

Código PHP:

while (/*cualquier condición*/)
{
echo $array['clave']. "-"; //aquí utilizas innecesariamente la función str_repeat();
//¿sigues creyendo que necesitas hacer un echo $row['label']?
}
 

en tu caso necesitas implode(); si te molestas en leer la documentación (algo que adecuadamente te ha insistido pateketrueke) verás que esta función separa los valores de un array. recibe dos parámetros, uno de ellos es opcional. este parámetro opcional si se especifica es un caracter string.

entonces si haces

Código PHP:

echo implode ("-", $array a separar);
 

los valores del array, independientemente si las claves son indexadas o asociativas serán imprimidas así:

Código PHP:

valor1-valor2-valor3-valor4 etc.
//no se imprimen las claves.
 

entonces aquí hay tres cosas claras:

no necesitas un bucle para utilizar implode.

no necesitas una función como str_repeat(); para imprimir una sola vez en cada iteración un guión "-".

y

si quieres imprimir las claves no puedes hacerlo con echo $array['clave']; puesto que te imprimirá el valor.

ahora piensa qué cambios tienes que hacer en tu código para que te funcione tal y como quieres.

[mikehove]

Gracias por escribirme amigos y enseñarme.
Estuve leyendo la documentación y disculpen todavía no encuentro el éxito que deseo.

Este es el resultado obtenido:

Código:

function get_tree($id, $level = 0)
{
    $result = mysql_query('SELECT id, label FROM menu WHERE parent='.$id);
        $guion = array();
    while ($row = mysql_fetch_array($result))
    {
          $guion[] = $row['label'];         
       get_tree($row['id'], $level + 1);		
    }
 echo implode(" > ", $guion);

}

echo get_tree(6,1);

Imprime:

Código PHP:

  SnippetArchive 
   

Seguiré investigando dónde estaré pifiándola.

[guardarmicorreo]

Iniciado por mikehove

Gracias por escribirme amigos y enseñarme.
Estuve leyendo la documentación y disculpen todavía no encuentro el éxito que deseo.

Este es el resultado obtenido:

te comento el código indicándote los errores

Código PHP:

function get_tree($id, $level = 0)
{
    $result = mysql_query('SELECT id, label FROM menu WHERE parent='.$id);
    $guion = array();
    while ($row = mysql_fetch_array($result))
    {
         $guion[] = $row['label'];
 
         //¿qué motivo tienes para volver a llamar a la función dentro de la función, cual es tu idea?
         //le sumas valores a $level pero arriba lo igualas a cero :S
         get_tree($row['id'], $level + 1);      
    }
    //esta parte la estás haciendo bien. te debería imprimir valor>valor2>valor3>...
    echo implode(" > ", $guion);
 
}
 
//nuevamente ¿por qué como segundo parámetro le pasas el 1 si arriba lo igualas a cero?
echo get_tree(6,1);
 

Imprime:

Código PHP:

SnippetArchive
 

Seguiré investigando dónde estaré pifiándola.

[ocp001a]

Estás usando una función recursiva, por lo que tu arreglo "$guion" solo contiene un elemento cada llamada por lo que implode no agrega el separador. Y estás usando $level sin sentido, ya que sólo la usas para pasar como parámetro.

Si sólo vas a tener una rama del árbol a la vez, y la rama no tiene más de una rama en cada nivel,creo que podría quedar así;

Código PHP:

function get_tree($id,$level=0){
    $result = mysql_query('SELECT id, label FROM menu WHERE parent='.$id);
    while ($row = mysql_fetch_array($result)){
          if($level)//si estamos en el segundo nivel o más
               echo '>';
          echo $row['label'];         
          get_tree($row['id'],$level+1);        
    }
}
 
get_tree(6,1);
 

El problema viene si las ramas no son completamente lineales y / o si se trata de más de una rama.

[mikehove]

Buenas ocp001a. Gracias por el aporte (y a mis compañeros)!.

Te comento que funciona!, solo llamé a la función con el argumento: $level=0.

Quedando tu código, con la siguiente línea:

Código:

get_tree(6,0);

Imprime:

Archive>Snippet

Muchísimas gracias!!!!

P/d: Continuaré investigando este Universo!

[mikehove]

Hola amigos!. Espero que estén bien. Reabro el tema después de mucho tiempo porque me encontré con un problema parecido con otra función recursiva y no se cómo corregir la flecha o el guión que no salga al principio.
Probé con implode() y no tuve éxito.

Este es el código:

Código:

function get_path($id)
{
	$result = mysql_query('SELECT name, parent_id FROM categorias WHERE id='.$id);
	$row = mysql_fetch_array($result);	
	if ($row['parent_id'] != NULL)
	{		
		get_path($row['parent_id']);		
		echo ' > ';
		echo $row['name'];		
	}	
}
	
get_path(9);

Resultado:
> Inicio > Portasahumerios

Objetivo es que quede así:

Código:

Inicio > Portasahumerios

Ayuda por favor, chicos!

[Triby]

Es exactamente lo mismo que el anterior, solo tienes que adaptar con las nuevas variables y listo.

[mikehove]

Disculpame Triby, pero no es el mismo procedimiento para mí porque no tiene "while", tampoco el parámetro "level", lo intenté de muchas formas.
¿Me puedes decir por favor dónde crees que está el error?

[pateketrueke]

Iniciado por mikehove

Disculpame Triby, pero no es el mismo procedimiento para mí porque no tiene "while", tampoco el parámetro "level", lo intenté de muchas formas.
¿Me puedes decir por favor dónde crees que está el error?

De hecho, la recursividad en si misma es una forma de un "loop".

La idea, es que la función devuelva un array, que irás concatenando indefinidamente.

Al final, usarás dicho array para aplicarle un join().

Código PHP:

function re($id) {
  $out = array();
 
  if ($row->parent) {
    $out = re($row->parent);
  }
 
  $out []= $row->name;
 
  return $out;
}
 
var_dump(re(10));
 

[mikehove]

Hola pateketrueke.
¿De esta forma eliminaré la flecha que sale al principio del resultado? Porque no le encuentro sentido tu código, disculpa.

Mi código completo

function get_path($id)
{
$result = mysql_query('SELECT name, parent_id FROM categorias WHERE id='.$id);
$row = mysql_fetch_array($result);
if ($row['parent_id'] != NULL)
{
get_path($row['parent_id']);
echo ' > ';
echo $row['name'];
}
}

get_path(9);

Resultado:
> Inicio > Portasahumerios

Objetivo: Eliminar la primera flecha, como muestra el resultado del código anterior.
Que quede de esta forma:

Inicio > Portasahumerio

Maldito guión que no puedo eliminar... Por favor

[mikehove]

Pido ayuda amigos, la verdad que desconozco, probé de muchas maneras,
¿cómo podré seguir?
ES SOLO SACAR LA FLECHA DEL PRINCIPIO DEL RESULTADO:

Esto no:
> Inicio > Portasahumerios

Esto si:
Inicio > Portasahumerios

Código completo:

function get_path($id)
{
$result = mysql_query('SELECT name, parent_id FROM categorias WHERE id='.$id);
$row = mysql_fetch_array($result);
if ($row['parent_id'] != NULL)
{
get_path($row['parent_id']);
echo ' > ';
echo $row['name'];
}
}

get_path(9);

[ocp001a]

Podrías meter todo en un arreglo y al final unir con implode.

Código PHP:

function get_path($id){
   $result = mysql_query('SELECT name, parent_id FROM categorias WHERE id='.$id);
   $row = mysql_fetch_array($result);
   if ($row['parent_id'] != NULL){
       $ret[]=get_path($row['parent_id']);
       //echo ' > ';
       //echo $row['name'];
      $ret[]=$row['name'];
   }
   return $ret;
}
 
$res=get_path(9);
 
echo implode(' > ',$res);
 

[mikehove]

Hola ocp001a. Gracias por responder. Esto es increíble. Había tratado antes de tu código incluirlo en un array y acomodarlo con implode y fue un fracaso.

Estos son los errores que me salen con tu código:

Código:

Notice: Undefined variable: ret 

Notice: Array to string conversion in C:... Array > Portasahumerios

Le había agregado también antes del if $ret = array(); y tampoco funciona.

La idea en el resultado es:

Inicio > Portasahumerios

[ocp001a]

Si te da error al regresar $ret es que no existe por que no entró al if, en tal caso, se puede crear antes

Código PHP:

$ret=array();
   if ($row['parent_id'] != NULL){
.
.
.
 

[mikehove]

Este es el aviso en pantalla:

Código:

Notice: Array to string conversion in C:..... line 23

Array > Portasahumerios

Línea 23:

Código:

echo implode(' > ',$res);

Este es el código completo:

Código:

function get_path($id){
   $result = mysql_query('SELECT name, parent_id FROM categorias WHERE id='.$id);
   $row = mysql_fetch_array($result);
   $ret=array();
   if ($row['parent_id'] != NULL){
       $ret[]=get_path($row['parent_id']);
       //echo ' > ';
       //echo $row['name'];
      $ret[]=$row['name'];
   }
   return $ret;
}
 
$res=get_path(9);
 
echo implode(' > ',$res);

[Triby]

Se soluciona fácilmente con una variable estática:

Código PHP:

function get_path($id) {
   static $separa = '';
   // Aquí tu consulta
   if ($row['parent_id'] != NULL){
       get_path($row['parent_id']);
       echo $separa; // La primera vez no mostrará nada
       echo $row['name'];
       $separa = ' > '; // A partir de la segunda mostrará mayor-que
   }
 
}
 

Aunque no entiendo porque con implode no funciona, habría que hacer un var_dump($res); para ver el contenido, seguramente hay un elemento vacío.

[mikehove]

Tribyyy GRACIASS! Y a todos los compañeros, muuchas gracias por la bendita ayuda psicológicamente!

TU CÓDIGO FUNCIONA!!

El tema con el código anterior al tuyo no se por qué no funcionaba, hice la prueba con var_dump($res) y este es el resultado:

Código:

array(2) { [0]=> array(2) { [0]=> array(0) { } [1]=> string(6) "Inicio" } [1]=> string(15) "Portasahumerios" }

Código completo del caso anterior al tuyo:

function get_path($id){
$result = mysql_query('SELECT name, parent_id FROM categorias WHERE id='.$id);
$row = mysql_fetch_array($result);
$ret=array();
if ($row['parent_id'] != NULL){
$ret[]=get_path($row['parent_id']);
//echo ' > ';
//echo $row['name'];
$ret[]=$row['name'];
}
return $ret;
}

$res=get_path(2);
var_dump($res);
echo implode(' > ',$res);

[pateketrueke]

El problema de la solución de Triby es que una segunda llamada a dicha función ya habrá seteado el valor de $separa y bueno, ya entenderán.

[mikehove]

Hola pateketrueke. Tienes razón, cuando llamó otras veces vuelve la flecha al principio.

Triby y Pateketrueke, ¿cómo se puede corregir? Por favor

Código completo:

function get_path($id){
static $separa = '';
$result = mysql_query('SELECT name, parent_id FROM categorias WHERE id='.$id);
$row = mysql_fetch_array($result);

if ($row['parent_id'] != NULL){
get_path($row['parent_id']);
echo $separa; // La primera vez no mostrará nada
echo $row['name'];
$separa = ' > '; // A partir de la segunda mostrará mayor-que

}

}

Repito la llamada:

<?php
get_path(2);
?>

</br>
<?php
get_path(2);
?>

Imprime:

Inicio > Portasahumerios
> Inicio > Portasahumerios

[Triby]

Como ves, lo más fácil no siempre es lo adecuado.

Respecto a lo que obtienes del var_dump()... no entiendo porque el primer elemento es un array con dos elementos y uno de ellos está vacío.

Código:

array(2) {
    [0]=> array(2) {
        [0]=> array(0) { }
        [1]=> string(6) "Inicio"
    }
    [1]=> string(15) "Portasahumerios" }

Tal vez solo falta un if para evitar agregar algo si $row['name'] es una cadena vacía; de cualqueir forma, la lógica sigue siendo incorrecta, porque supongo que tendrás una telaraña si vas a niveles mas extensos.

Esto muy aparte de que no es recomendable tener consultas así en funciones recursivas; puede ser útil cuando tienes pocos visitantes, pero se convertirá en un verdadero dolor de cabeza cuando las peticiones se incrementen.

Tal vez te serviría leer esto: http://www.forosdelweb.com/f18/lista...2/#post3181868

[mikehove]

Perfecto Triby. Ahora lo leo.

[pateketrueke]

La estrategia que había sugerido desde un inicio debería bastar, sin embargo no era código copy+paste para que lo usaras sin restricción.

A) La idea era exponer el concepto de usar arrays para ir acumulando los valores, internamente ir concatenando dichos arrays y de manera externa hacerles join() para unirlos.

B) Ahora, también se puede hacer con una variable auxiliar (a modo de emular el comportamiento de la variable estática) pero ello también podría acarrear otros problemas.

¿Y si en lugar de imprimir la cadena deseo almacenarla?

¿Y si quiero cambiar el separador?

La solución (A) sería la correcta entonces, mientras la solución (B) es siempre más frágil (aunque posible).

A)

Código PHP:

function get_path($id) {
  // $result y $row ...
 
  if ($row['parent_id'] != NULL) {
    // existe un parent_id, inicializamos el acumulador
    $salida = get_path($row['parent_id'], $separador, true);
  } else {
    // inicializamos el acumulador vacio
    $salida = array();
  }
 
  // agregamos el valor al acumulador
  $salida []= $row['name'];
 
  // devolvemos el acumulador
  return $salida;
}
 
echo join('>', get_path(1));
 

B)

Código PHP:

function get_path($id, $separador = '>', $mostrar_separador = false) {
  // $result y $row ...
 
  if ($row['parent_id'] != NULL) {
    // existe un parent_id, mostramos separador
    get_path($row['parent_id'], $separador, true);
  }
 
  echo $row['name'];
 
  if ($mostrar_separador) {
    echo $separador;
  }
}
 
echo get_path(1);
 

A propósito de lo siguiente:

array(2) {
[0]=> array(2) {
[0]=> array(0) { }
[1]=> string(6) "Inicio"
}
[1]=> string(15) "Portasahumerios" }

Eso se debe a que se agrega el resultado de get_path() y es un array, por eso se está generando un array de arrays independientemente de si hay o no un valor.

Lo correcto sería "aplanar" el array, o en el caso de la estrategia (A) reemplazar el array por el acumulador que resuelve el parent_id.

[Triby]

Recordaba que había sugerido una solución más cercana a lo que necesitas, pero no encontraba el mensaje, está aquí: http://www.forosdelweb.com/f18/aport...3/#post4284145

Sugerencia: Haz un var_dump() para ver el contenido del array y puedas adaptarlo a tus necesidades.

[mikehove]

¿pateketrueke a qué te refieres con la variable $separador de la estrategia A?
El $separador es "= '>'"- línea 8???

Código PHP:

function get_path($id) {
  // $result y $row 
  $result = mysql_query('SELECT name, parent_id FROM categorias WHERE id='.$id);
  $row = mysql_fetch_array($result);
 
  if ($row['parent_id'] != NULL) {
    // existe un parent_id, inicializamos el acumulador
    $salida = get_path($row['parent_id'], $separador = '>', true);
  } else {
    // inicializamos el acumulador vacio
    $salida = array();
  }
 
  // agregamos el valor al acumulador
  $salida []= $row['name'];
 
  // devolvemos el acumulador
  return $salida;
}
 
echo implode('>', get_path(2));
 

Imprime:

Código HTML:

>Inicio>Portasahumerios
 

En cuanto a Triby también voy a evaluar los enlaces que me pasaste