S
Size: a a a
2020 November 05
AM
давай так, ты напишешь что там не так, я тебя разбаню в пхп чате)
Я в пхп чате разбанен, вроде как
T🐜
Ага,и все массивчики которые я в столбик разложил превратятся в несуразную хуету
можно еще юзать DI, заприватить почти все методы, разложить по объектам вот эту кучу методов. А так - пойдет. Не для себя же пишешь ))
AM
давай так, ты напишешь что там не так, я тебя разбаню в пхп чате)
да и в всё равно хотел тут по минимуму написать своё мнение
АХ
The Ant 🐜
можно еще юзать DI, заприватить почти все методы, разложить по объектам вот эту кучу методов. А так - пойдет. Не для себя же пишешь ))
Ты посмотри сколько там этих объектов
АХ
Если бы все объекты которые там вызываются были бы вложены в одну партянку,этим можно было бы весь земной шар 2 раза обернуть
T🐜
Ты посмотри сколько там этих объектов
да я вижу, но можно еще больше сделать :D
АХ
The Ant 🐜
да я вижу, но можно еще больше сделать :D
Предложи сделаю
AM
я ж тебе писал вроде
мой бан там поменяли на огранку на 1 день, или потом изменили на вечный бан опять, или соврали
АХ
The Ant 🐜
да я вижу, но можно еще больше сделать :D
я пытался, основная трабла это проварка на isset и не empty это нельзя вложить в функцию
АХ
т.к. если не придёт аргумент в функцию интерпретатор пошлёт тебя куда подальше
АХ
Остальное всё в объектах. данные шлют разные совершенно
M
The Ant 🐜
все отлично же. Не слушай этих симфонистов, они токсики и вообще быдланы, любят оскорблять просто так.
ctrl + l в шторме ток почаще тыкай
ctrl + l в шторме ток почаще тыкай
Просто так?
T🐜
Просто так?
что просто так?
VK
Предложи сделаю
Строка 415-421
Ты проверяешь, не пустой ли массив, если не пустой, то присваиваешь этот массив другому массиву и проверяешь на то, существует ли этот массив? зачем?
Ты проверяешь, не пустой ли массив, если не пустой, то присваиваешь этот массив другому массиву и проверяешь на то, существует ли этот массив? зачем?
M
The Ant 🐜
что просто так?
Перечитай свое предложение
VK
if(!empty($found_document['owner_tin'])){
$invoice_arr['SellerTin'] = $found_document['owner_tin'];
if(isset($invoice_arr['sellertin'])){
$company_seller = MailBox::where('mailbox_tin', $invoice_arr['sellertin'])->first();
$mailbox_id_seller = $company_seller['mailbox_id'];
}
}T🐜
я пытался, основная трабла это проварка на isset и не empty это нельзя вложить в функцию
вот например метод
можно вынести в отдельный сервис. инжектить в него газл со своими настройками. -1 метод в твоей портянке
public static function technical_timestamp($signature_hex)можно вынести в отдельный сервис. инжектить в него газл со своими настройками. -1 метод в твоей портянке
T🐜
так покопать там много можно чего сделать