[Spofity]

Da się jakoś streścić ten kawałek kodu? Bo wydaje mi się, że sie da ale nie mam pomysłu jak. Chodzi mi o to, żeby nie pisac za kazdym razem ifa dla danego rozszerzenia, tylko umiescic w jednym ifie, coś jak 'imagecreatefrom[$ext]'
#php #programowanie #webdev

[Spofity]

@Spofity: dużo tego nie ma, ale wolałbym to uprościć

[stosiedem]

@Spofity: zrób sobie stringa imagecreafrom + $ext i wywołaj przez call_user_func() ( ͡° ͜ʖ ͡°)

[ktos12356]

@Spofity: if lub switch jest dobrym rozwiązaniem w tym przypadku.

[OlekAleksander]

@Spofity: Lekko lepszy będzie switch-case, ale ja na przykład obu rozwiązań nie lubię :P

[Spofity]

@ktos12356: @OlekAleksander: o właśnie, switch case, kompletnie wyleciało mi z głowy. Chociaż to wiele nie zmieni, podobna ilość kodu

[Saly]

@Spofity: jak nie ma gotowej funkcji to własnie tak trzeba zrobić. Ewentualnie switch

[OlekAleksander]

@Spofity: Podobna ilość, ale czyta się lekko lepiej

[Kmycic]

@Spofity: a rozważasz użycie zewnętrznej paczki ?

[Spofity]

@Kmycic: jasne

[phervers]

@Spofity: Coś takiego np:

$filetypes = [
 'jpg' => 'jpeg',
 'png' => 'png',
 'bmp' => 'bmp',
];
if($filetypes[$ext]) {
 $img = call_user_function('imagecreatefrom' + $filetypes[$ext], $url);
} else {
 // Not supported
}

[Spofity]

@phervers: dziękuję! Tego właśnie szukałem

[Kmycic]

@Spofity: http://image.intervention.io polecam rękami nogami

[Spofity]

@Kmycic: wydaje się bardzo intuicyjne, zapisze sobie

[bmLq]

@Spofity:
$img = ('imagecreatefrom'.$ext)($url);

albo czytelniej
$createFunc = 'imagecreatefrom'.$ext;
$img = $createFunc($url);

[Spofity]

@bmLq: z tym będą problemy, bo z rozszerzeniem jpg będzie $ext=jpg, a funkcja brzmi createfromJPEG

[bmLq]

@Spofity: no to zostaje zrobić kombinację tego co zaproponował phervers i mojego albo po prostu sposób phervers'a

[nowiutki]

@Spofity: jeśli to OOP to metoda wytwórcza idealnie powinna się do tego nadać. Switch-case można wówczas zamienić na if class_exists i wywoływać stosowną faktorkę, w innym przypadku zrzucać exception.