Konvertere text

Virker det her ikke lige så godt:

$titel="Grundig Vaskemaskine";

$small = strtolower($titel);
$str = explode(" ", $small);

echo $str[0]; echo $str[1];

Mulig kode:


$input = "Panasonic Bagemaskine SD444";
$inputArr = explode(" ", $input);
$inputArr = array_slice($inputArr, 0, 2);
$input = implode(" ", $inputArr);

Eller i en lang særning:

$input = implode(" ", array_slice(explode(" ", $input), 0, 2));

Jeg er faktisk ikke sikker på om du får en fejl med:

echo $str[1];

hvis der kun er et or i din tekst. Men jeg tror det nu ikke. :^)

Der er ALTID min 2 ord ;)

nielle hvad gør dit bedre end mit eksempel ? :)

Jeg tvivler på at den løser opgaven bedre, men den er da i hvertfald mere kompakt og det var det du startede med at efterspørge :^)

function convertText($input)
{
    return implode(" ", array_slice(explode(" ", strtolower($input)), 0, 2));
}

Imidlertid er "mere kompakt" ikke en kvalitet i sig selv! One-liners er som regel svære at gennemskue for andre (inkl. dig selv om ½ års tid), og derfor ikke specielt vedligeholdelsesvenlige.

så du vil anbefale er bruge mit eget eller dit $input = implode(" ", array_slice(explode(" ", $input), 0, 2));

da de vil virke 100% og er noget kortere end det første :) ???

Jeg vil ikke anbefale den ene frem for den anden. Umiddelbart vil jeg gætte på at de er nogenlunde lige effektive.

Jeg er nu heller ikke sikker på at de er mere effektive end det du har fra starten af: Hvis din tekst er meget lang, og har mange mellemrum, kunne jeg godt forestille mig at explode() ender med at bruge flere ressourcer end at lede efter mellemrum nr. 2 som du gør.

"Kort og kompakt" kode er ikke nødvendigvis det samme som "effektiv" kode. Desuden er performance ikke nødvendigvis et-og-alt (selv om det er vigtigt), men vedligeholdelsesvenlighed er faktisk også vigtig.

Til sidst er her endnu et bud:

function convertText($input)
{
    $firstSpacePos = strpos($input, " ");
    $secondSpacePos = strpos($input, " ", $firstSpacePos);

    return ($secondSpacePos === false)
        ? strtolower(substr($input, 0, $firstSpacePos))
        : strtolower(substr($input, 0, $secondSpacePos));
}

Har du fået svar på alle dine spørgsmål, eller er der mere?

Ja tak, smid point ;)

Svar :^)