А как бы ты в этом примере сделал?

|
soul_abduction_ceremony 2022-07-02 16:47:22
а, ладно)
soul_abduction_ceremony 2022-07-02 16:47:26
не знал
the_ElePHPant 2022-07-02 16:48:02
Но вообще да, лучше аксессоры называть не так как поля
soul_abduction_ceremony 2022-07-02 16:48:04
но все равно по мне звучит как ответственность для отдельного класса/метода
oleksandr_moik 2022-07-02 16:48:12
soul_abduction_ceremony 2022-07-02 16:45:33
а если ему надо будет получать саму ссылку а не путь в Storage, тогда что делать?

в случае с файлами, возможно отделиним методом getUrlFor($attribute) и в нем проверку — нал или ссилку

led23 2022-07-02 16:48:13
soul_abduction_ceremony 2022-07-02 16:46:43
а если тебе не ссылка нужна а $this->logo сам, неизменный

можно не сам logo, а logo_url

led23 2022-07-02 16:48:21
тк логичнее
oleksandr_moik 2022-07-02 16:48:52
led23 2022-07-02 16:48:13
можно не сам logo, а logo_url

кстати да

led23 2022-07-02 16:51:16
laravel_web-801829.jpg

led23 2022-07-02 16:51:21
Типа такого получается..
led23 2022-07-02 17:03:55
А есть ли смысл писать ретерн налл?
led23 2022-07-02 17:04:19
Как я понимаю, он в любом случае его вернет
мне как будто хочется его написать, на в мидделваэ не пишут и забивают
soul_abduction_ceremony 2022-07-02 17:04:53
ну вообще это глупо выглядит, я бы не писал, тогда уже сделай в конце return $this->logo;
soul_abduction_ceremony 2022-07-02 17:05:09
или что то такое
led23 2022-07-02 17:05:16
soul_abduction_ceremony 2022-07-02 17:04:53
ну вообще это глупо выглядит, я бы не писал, тогда уже сделай в конце return $this->logo;

это еще менее интуитивно

led23 2022-07-02 17:05:47
laravel_web-801836.jpg
IDE хочет ретерн
soul_abduction_ceremony 2022-07-02 17:05:54
самое интуитивное это сделать по нормальному отдельный класс)
led23 2022-07-02 17:06:04
soul_abduction_ceremony 2022-07-02 17:05:54
самое интуитивное это сделать по нормальному отдельный класс)

похоже что да

soul_abduction_ceremony 2022-07-02 17:06:05
led23 2022-07-02 17:05:47
IDE хочет ретерн

потому что у тебя указан return type ?string

led23 2022-07-02 17:06:28
а void != null?
soul_abduction_ceremony 2022-07-02 17:06:35
led23 2022-07-02 17:06:04
похоже что да

ну просто метод getUrlFromStorage

soul_abduction_ceremony 2022-07-02 17:06:38
led23 2022-07-02 17:06:28
а void != null?

неа

led23 2022-07-02 17:07:09
но условно же

fn()=>{} == fn () => {return null;}

soul_abduction_ceremony 2022-07-02 17:07:09
led23 2022-07-02 17:06:28
а void != null?

ничего не вернуть — будет ошибка declared return type string|null, none returned.

soul_abduction_ceremony 2022-07-02 17:07:52
led23 2022-07-02 17:07:09
но условно же

fn()=>{} == fn () => {return null;}

двойное равно может и да, тройное по идее не должно)

oleksandr_moik 2022-07-02 17:08:04
led23 2022-07-02 17:07:09
но условно же

fn()=>{} == fn () => {return null;}

тут не указано что возвращать.

led23 2022-07-02 17:08:09
так с синтаксисом напутал, тут fn не так работает
led23 2022-07-02 17:08:30
но в случае воид все равно вернется null?
oleksandr_moik 2022-07-02 17:08:31
led23 2022-07-02 17:07:09
но условно же

fn()=>{} == fn () => {return null;}

ести указать ?string то уже будет ругатся

led23 2022-07-02 17:08:42
oleksandr_moik 2022-07-02 17:08:31
ести указать ?string то уже будет ругатся

интересно стало
сейчас провреб

psytrgles 2022-07-02 17:08:56
в рнр функции неявно возвращают null, но это не значит что если ты возвращаешь null, то надо опускать return;
return $this->cover ? Storage::url() : null;
psytrgles 2022-07-02 17:09:38
void — когда функция ничего не возвращает.
led23 2022-07-02 17:09:41
oleksandr_moik 2022-07-02 17:08:31
ести указать ?string то уже будет ругатся

и правда TypeError

oleksandr_moik 2022-07-02 17:09:42
led23 2022-07-02 17:08:30
но в случае воид все равно вернется null?

void — функция нигда ничего не вернет

led23 2022-07-02 17:09:56
psytrgles 2022-07-02 17:09:38
void — когда функция ничего не возвращает.

ну неявно же она это делает?

psytrgles 2022-07-02 17:10:06
led23 2022-07-02 17:09:56
ну неявно же она это делает?

потому что нет типа void

oleksandr_moik 2022-07-02 17:10:22
led23 2022-07-02 17:09:56
ну неявно же она это делает?

да

PSYTRGLES in Laravel для начинающихTelegram
в рнр функции неявно возвращают null, но это не значит что если ты возвращаешь null, то надо опускать return;
return $this->cover ? Storage::url() : null;
soul_abduction_ceremony 2022-07-02 17:10:27
led23 2022-07-02 17:09:56
ну неявно же она это делает?

если ты укажешь return type то будет ошибка

psytrgles 2022-07-02 17:10:39
ты сейчас сам себе противоречишь, говоришь — я возвращаю null или стринг, но на деле возвращаешь стринг или НИЧЕГО
soul_abduction_ceremony 2022-07-02 17:11:05
soul_abduction_ceremony 2022-07-02 17:07:09
ничего не вернуть — будет ошибка declared return type string|null, none returned.

ну я уже писал об этом да

psytrgles 2022-07-02 17:11:06
если функция может вернуть null, то ты его и верни
led23 2022-07-02 17:11:32
ну вот и хочется)
тогда оставлю так
led23 2022-07-02 17:11:50
Спасибо)
soul_abduction_ceremony 2022-07-02 17:11:51
psytrgles 2022-07-02 17:11:06
если функция может вернуть null, то ты его и верни

но это ж вообще херовая практика) может конечно в моделях это простительно, но и то хуйня по моему

soul_abduction_ceremony 2022-07-02 17:12:19
led23 2022-07-02 17:11:32
ну вот и хочется)
тогда оставлю так

все лишь бы не потратить 2 минуты и сделать класс / функцию для этого, где все по нормальному расписать)

psytrgles 2022-07-02 17:12:21
soul_abduction_ceremony 2022-07-02 17:11:51
но это ж вообще херовая практика) может конечно в моделях это простительно, но и то хуйня по моему

почему? ты сейчас говоришь, что если функция возвращает ЧТО-ЛИБО, то это херово

led23 2022-07-02 17:12:23
led23 2022-07-02 16:51:16

а как бы ты в этом примере сделал?

led23 2022-07-02 17:12:41
soul_abduction_ceremony 2022-07-02 17:12:19
все лишь бы не потратить 2 минуты и сделать класс / функцию для этого, где все по нормальному расписать)

Я имею в виду не буду убирать ретерны нулловые

psytrgles 2022-07-02 17:13:00
led23 2022-07-02 17:12:41
Я имею в виду не буду убирать ретерны нулловые

там тернарник уместнее

led23 2022-07-02 17:13:06
psytrgles 2022-07-02 17:13:00
там тернарник уместнее

он был)

|