← пред | след →
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
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

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
там тернарник уместнее
он был)
← пред | след →