[SOLUCIONADO] Reemplazar última coma por " y"

[zyxer]

Hola,

Tengo un script que genera una lista dinámicamente (ejemplo: azucar, pan, leche, ajo, cebolla, sal, manzana, café) lo que quiero es hacer tomar esta lista y reemplazar la última coma (,) por " y" (de manera que en nuestro ejemplo quedaría así: azucar, pan, leche, ajo, cebolla, sal, manzana y café).

La lista no siempre se genera con las mismas palabras ni tiene siempre la misma cantidad de elementos.

Desde ya, las gracias.

[Alexis88]

Muéstranos la manera en como generas la lista, probablemente en ese mismo bloque se pueda añadir la solución.

Saludos

[zyxer]

Es trayendo un montón de datos de MySQL y concatenando, pero para simplificarlo digamos que es así:

Código PHP:

<?php
 
    $array = array("pan", "azucar", "leche");
    $arrayNum = count($array);
    $i = 0;
    $lista = '';
 
    while($i<$arrayNum){
 
        $lista.=$array[$i];
        $lista.=', ';
        $i++;
 
    }
 
    $lista = substr($lista, 0, -2);
 
    echo $lista;
 
?>
 

[Italico76]

Código PHP:

<?php
 
    $array = array("pan", "azucar", "leche");
    
    $salida = null;
    for ($i=0;$i<count($array)-1;$i++)
        $salida .= $array[$i].' ,';
        
    $salida .= $array[$i];  
    
        
    echo $salida;
 

[Alexis88]

En cada iteración, verifica que el valor del contador sea igual al total de elementos del array menos uno, es decir, la última ubicación del mismo. De ser así, colocas " y ", seguido del elemento, caso contrario, va la coma y el elemento.

Código PHP:

$array = array("pan", "azucar", "leche");
$arrayNum = count($array);
$i = -1;
$lista = '';
 
while(++$i < $arrayNum)
    $lista .= strlen($lista) ? $i == $arrayNum - 1 ? ' y ' . $array[$i] : ', ' . $array[$i] : $array[$i];
 
echo $lista;
 

El resultado de esto es lo siguiente:

Código HTML:

pan, azucar y leche
 

Saludos

[Italico76]

Sin duda lo mas simple seria...

Código PHP:

<?php
   
    echo implode(',',array("pan", "azucar", "leche"));
 

[Alexis88]

Italico76, ¿Has leído el título y descripción del tema?

[Italico76]

Iniciado por Alexis88

Italico76, ¿Has leído el título y descripción del tema?

No

Ahora si

Código PHP:

  <?php
 
    $array = array("pan", "azucar", "leche", "otros");
        
    function separar(&$arr)
    {
        $cant = count($arr);    
    
        if ($cant==0)  return null;        
        $salida = null;
        
        if ($cant>1)
        {
            for ($i=0;$i<$cant-2;$i++)
                $salida .= $arr[$i].' ,';
        
            return $salida.$arr[$cant-2].' y '.$arr[$cant-1];  
            
        }else
            return $arr[0];      
    
    }
    
    echo separar($array);   

@Alexis88 : me gusto lo compacta de tu solucion lastima que es ineficiente en CPU .... si te fijas estas haciendo demasiadas preguntas por iteracion.... no hay nada que preguntar

Simula eso con 100.000 elementos y veras que tu solucion es bastante mas lenta

[Alexis88]

Aún más sencillo de como lo había planteado:

Código PHP:

$array = array("uno", "dos", "tres");
$string = implode(", ", $array);
echo substr_replace ($string, " y ", strrpos($string, ","), 2);
 

DEMO

Con esto, evitamos el uso de bucles y el script se vuelve más eficiente.

Saludos

[Italico76]

Iniciado por Alexis88

Aún más sencillo de como lo había planteado:

Código PHP:

$array = array("uno", "dos", "tres");
$string = implode(", ", $array);
echo substr_replace ($string, " y ", strrpos($string, ","), 2);
 

DEMO

Con esto, evitamos el uso de bucles y el script se vuelve más eficiente.

Saludos

Ya habria que ver la implementacion de implode() y substr_replace() pero me juego a que si consume mas tiempo porque estas recorriendo el arreglo DOS VECES!!! una vez con cada funcion

[zyxer]

JAAJAJAJAJAJAJA

Gracias a ambos, creo que con lo que han dicho puedo trabajar.

[Italico76]

Iniciado por zyxer

JAAJAJAJAJAJAJA

Gracias a ambos, creo que con lo que han dicho puedo trabajar.

Nos alegramos

[Alexis88]

Pues, parece ser que siempre es más eficiente el utilizar funciones nativas: http://ideone.com/ERE5WJ

Y la diferencia se hace más notoria (y preocupante) cuando el array contiene más elementos:

• Ideone
• Sandbox
• Codepad

De cualquier modo, nuestro amigo zyxer aprendió más de una forma de hacer lo que buscaba.

Saludos

[Italico76]

Me quede pensando... tenia que haber una solucion eficiente y simple para tratar al ultimo elemento distinto ya sea en un foreach() o en una funcion de arrays.......

Código PHP:

function separar(&$arr)
{
    switch (count($arr))
    {
    case 0: return null;
    case 1: return $arr[0];
    default:    
        $ult = array_pop($arr);
        return implode (' ,',$arr).' y '.$ult;
    }    
}
 

Lamentablemente siempre existan 3 casos base y toca de alguna forma tratarlos (no elementos, 1 elemento, 2 o mas)

[Alexis88]

La solución es buena, pero solamente resulta eficiente cuando el array contiene pocos elementos, pues cuando contiene muchos, que es lo común cuando se trabaja con BD, deja de serlo.

• Con 100 elementos
• Con 500 elementos
• Con 1000 elementos

Por eso opté por no utilizar alguna estructura e ir directamente por las funciones nativas, porque, como alguna vez leí en este foro, este lenguaje no es muy eficiente que digamos.

Saludos

[Italico76]

JO JO JO......Ud es un TRAMPOSO!!!! si Ud invierte el orden de las llamadas a las funciones la suya pierde lejos!!!!!!!!!!

Para 500:

Código resutlados pruebas:

Italico: 24.439208984375
Alexis: 51.1572265625
 

y lo unico que hice fue cambiar el orden de las llamadas:

Código PHP:

<?php
function italico(&$array)
{
    switch (count($array))
    {
    case 0: return null;
    case 1: return $array[0];
    default:    
        $ult = array_pop($array);
        return implode (' ,',$array).' y '.$ult;
    }    
}
 
function alexis($array) {
    $string = implode(", ", $array);
    return substr_replace ($string, " y ", strrpos($string, ","), 2);
}
 
$array = range(1, 500);
 
$ini2 = microtime(true) * 1000;
for ($i = 0; $i < 1000; $i++)
    alexis($array);
$fin2 = microtime(true) * 1000;
$tiempoAlexis = $fin2 - $ini2;
 
$ini1 = microtime(true) * 1000;
for ($i = 0; $i < 1000; $i++)
    italico($array);
$fin1 = microtime(true) * 1000;
$tiempoItalico = $fin1 - $ini1;
 
 
 
echo "Italico: " . $tiempoItalico."\n";
echo "Alexis: " . $tiempoAlexis;
 

El ultimo corre mas rapido ya que esos datos ya se procesaron y ni siquiera creando copias de los datos si son iguales el efecto "memoria" desaparece:

Código PHP:

<?php
function italico1(&$arr)
    {
        $cant = count($arr);    
    
        if ($cant==0)  return null;        
        $salida = null;
        
        if ($cant>1)
        {
            for ($i=0;$i<$cant-2;$i++)
                $salida .= $arr[$i].' ,';
        
            return $salida.$arr[$cant-2].' y '.$arr[$cant-1];  
            
        }else
            return $arr[0];      
    
    }
 
function italico2(&$arr)
{
    switch (count($arr))
    {
    case 0: return null;
    case 1: return $arr[0];
    default:    
        $ult = array_pop($arr);
        return implode (' ,',$arr).' y '.$ult;
    }    
} 
 
function alexis($array) {
    $string = implode(", ", $array);
    return substr_replace ($string, " y ", strrpos($string, ","), 2);
}
 
$a[] = range(1, 500);
$a[] = range(1, 500);
$a[] = range(1, 500);
 
 
$ini2 = microtime(true) * 1000;
for ($i = 0; $i < 1000; $i++)
    alexis($a[1]);
$fin2 = microtime(true) * 1000;
$tiempoAlexis = $fin2 - $ini2;
 
$ini1 = microtime(true) * 1000;
for ($i = 0; $i < 1000; $i++)
    italico1($a[0]);
$fin1 = microtime(true) * 1000;
$tiempoItalico1 = $fin1 - $ini1; 
 
$ini1 = microtime(true) * 1000;
for ($i = 0; $i < 1000; $i++)
    italico2($a[2]);
$fin1 = microtime(true) * 1000;
$tiempoItalico2 = $fin1 - $ini1;
 
  
echo "Italico (script #1): " . $tiempoItalico1."\n";
echo "Italico (script #2): " . $tiempoItalico2."\n";
echo "Alexis: " . $tiempoAlexis;
 

Puede probar...y si sigue cambiando de lugar las llamadas..... con los mismos datos, mismo "problema"

[Alexis88]

¿Tramposo? ¿Sabes que antes de ejecutar cada función, tomo el tiempo en milésimas de segundo que corresponda a ese momento? Hay que aprender a controlar el ego, mi estimado.

El cuerpo de benito

[Italico76]

El post anterior se hizo largo (mucho codigo)..... voy a postear la solucion "justa" donde podras ver que si cambias el orden, el resultado cambia poco (porque de todas formas hay repeticion de valores)

Código PHP:

<?php
function italico1(&$arr)
    {
        $cant = count($arr);    
    
        if ($cant==0)  return null;        
        $salida = null;
        
        if ($cant>1)
        {
            for ($i=0;$i<$cant-2;$i++)
                $salida .= $arr[$i].' ,';
        
            return $salida.$arr[$cant-2].' y '.$arr[$cant-1];  
            
        }else
            return $arr[0];      
    
    }
 
function italico2(&$arr)
{
    switch (count($arr))
    {
    case 0: return null;
    case 1: return $arr[0];
    default:    
        $ult = array_pop($arr);
        return implode (' ,',$arr).' y '.$ult;
    }    
} 
 
function alexis($array) {
    $string = implode(", ", $array);
    return substr_replace ($string, " y ", strrpos($string, ","), 2);
}
 
$a[] = range(1, 1000,2);
$a[] = range(1, 1500,3);
$a[] = range(1, 3500,7);
 
$ini1 = microtime(true) * 1000;
for ($i = 0; $i < 1000; $i++)
    italico1($a[0]);
$fin1 = microtime(true) * 1000;
$tiempoItalico1 = $fin1 - $ini1; 
 
$ini1 = microtime(true) * 1000;
for ($i = 0; $i < 1000; $i++)
    italico2($a[2]);
$fin1 = microtime(true) * 1000;
$tiempoItalico2 = $fin1 - $ini1;
 
$ini2 = microtime(true) * 1000;
for ($i = 0; $i < 1000; $i++)
    alexis($a[1]);
$fin2 = microtime(true) * 1000;
$tiempoAlexis = $fin2 - $ini2; 
 
 
 
 
echo "Italico (script #1): " . $tiempoItalico1."\n";
echo "Italico (script #2): " . $tiempoItalico2."\n";
echo "Alexis: " . $tiempoAlexis;
 

En el orden que quieras.....

Alexis: 55 seg aprox
Italico #1: 162 seg aprox
Italico #2: 25 seg aprox << winner

Asi que... es cierto que se puede ganar velocidad con funciones nativas y de hecho las he utilizado en la ultima version pero a difierencia tuya no recorro el arreglo varias veces con distintas funciones (nativas o no) y por eso a la final.......... gano :)

[Alexis88]

Claro, campeón, tu "ganas".

http://ideone.com/ajHblr

[Italico76]

Bueno...ahi la diferencia fue de 2 segundos apenas......

Declaremos el empate ;)

[Alexis88]

Jeje, como gustes (aunque ahí veo 4 segundos de diferencia ). Damos por concluido el tema.

Saludos

[Italico76]

Antes del cierre... me doy por vencido:

lo corrí para 10.000 elementos.... y UFFF... ahi me das una buena paliza

Italico (script #1): 3927.8950195312
Italico (script #2): 2406.8630371094
Alexis: 1203.8640136719

Y eso en este orden:

Código PHP:

$ini1 = microtime(true) * 1000;
for ($i = 0; $i < 1000; $i++)
    italico1($a);
$fin1 = microtime(true) * 1000;
$tiempoItalico1 = $fin1 - $ini1; 
 
$ini2 = microtime(true) * 1000;
for ($i = 0; $i < 1000; $i++)
    alexis($a);
$fin2 = microtime(true) * 1000;
$tiempoAlexis = $fin2 - $ini2;  
 
$ini1 = microtime(true) * 1000;
for ($i = 0; $i < 1000; $i++)
    italico2($a);
$fin1 = microtime(true) * 1000;
$tiempoItalico2 = $fin1 - $ini1;
 

Ya no me explico porque tu funcion crece con un exponente menor que la mia en tiempo excepto que..... array_pop() sea una carreta


Mi conclusion: al escribir un codigo eficiente no solo hay que ser eficientes con la algoritmica sino tambien estudiosos para la eleccion de las funciones nativas