Телеграмм чат группы prophp7 страница 10994

13:20пожаловаться #1

Egor Gruzdev in PHP

Вячеслав

Всем привет, возник спор с товарищем - нормально ли передавать в метод аргумент по ссылке? Я использую этот метод как отдельно, так и в качестве колбэка для array_walk. Мне старший товарищ говорит, что это плохо, верить и рефакторить?)

все таки лучше:
private function formatLogin(string $login) : string
{
...

13:21пожаловаться #2

Sergo in PHP

проблема в том что можно выстрелить себе в ногу где-то позже

13:21пожаловаться #3

Artem Zakirullin in PHP

Вячеслав

имею ввиду вот такой код

Рефакторь

а зачем?)

чтоб не дублировать код, использую этот метод в качестве колбэка

13:21пожаловаться #5

Вячеслав in PHP

Блин( Ну ок, спасибо(

Альберт Степанцев... in PHP

13:21пожаловаться #6

АС

Вячеслав

имею ввиду вот такой код

вы собиратесь в ходе работы функции изменять $login?
ваша сигнатура говорит именно об этом

Алексей Гевондян... in PHP

13:26пожаловаться #7

АГ

тут скорее про замену array_walk на array_map

13:27пожаловаться #8

Вячеслав in PHP

ну смысл такой, что в одном участке кода я вызываю этот метод напрямую, в другом в качестве колбэка для array_walk. В любом случае уже исправил, заменил walk на map и возвращаю string.

13:28пожаловаться #9

а что там на пульс проекта смотреть, если мозилла от файрфокса не отказывается, а их Servo при этом на расте?

Так файрфокс то сейчас не на servo. И как раз разработчики серво и были уволены.

14:03пожаловаться #10

Так файрфокс то сейчас не на servo. И как раз разработчики серво и были уволены.

А на чём он уже? На Хромиуме?

14:03пожаловаться #11

А на чём он уже? На Хромиуме?

На gecko, как и был.

14:04пожаловаться #12

На gecko, как и был.

всм?

14:04пожаловаться #13

Egor Gruzdev in PHP

А на чём он уже? На Хромиуме?

Mozilla/5.0 (Windows NT 10.0; Win64; x64; rv:78.0) Gecko/20100101 Firefox/78.0

14:04пожаловаться #14

всм?

Ну в прямом смысле. servo писали как замену gecko, но он только развивался. Там разве что webrender был впилен в фф.

14:05пожаловаться #15

Ну в прямом смысле. servo писали как замену gecko, но он только развивался. Там разве что webrender был впилен в фф.

Насколько я видел по новостям, Servo уже был внедрён и паралельно с него забирали части в Gecko

14:05пожаловаться #16

Ну в прямом смысле. servo писали как замену gecko, но он только развивался. Там разве что webrender был впилен в фф.

Есть какой-либо источник, что сейчас именно gecko? (user agent не принимается)

14:06пожаловаться #17

Артём in PHP

Есть какой-либо источник, что сейчас именно gecko? (user agent не принимается)

Тебя разморозили вчера, штоли?

14:06пожаловаться #18

Артём

Тебя разморозили вчера, штоли?

А тебя?

14:06пожаловаться #19

Есть какой-либо источник, что сейчас именно gecko? (user agent не принимается)

"mozilla-central" contains current Firefox and Gecko development. Releases branch off of this repository. Also known as "Trunk" or "nightly".
Вот такое сойдет?