Size: a a a

2020 January 18

MB

Max B in phpGeeksJunior
Вот что я накрокодили по этому тестовому
https://github.com/melfis55/CLI_images_parser
источник

MB

Max B in phpGeeksJunior
И что я получил в ответ.

Здравствуйте
Да, получила фидбек и, к сожалению, по результатам не готовы рассматривать вас на эту должность.
Основные замечания:
1. проигнорированы требования задания:
1.1. не используется composer
1.2. код не расширяемый

2. в коде присутствуют недопустимые решения
2.1. сокрытие ошибок
2.2. отсутствие неймспейсов
2.3. смешанная стилистика кода (где-то есть тайпхинтинг, где-то нет)
2.4. закомментированные участки кода
2.5. хардкод параметров вызова CURL
2.6. нарушение принципа DRY
2.7.. нарушение принципов SOLID
источник

АП

Алексей Пастушенко... in phpGeeksJunior
источник

АП

Алексей Пастушенко... in phpGeeksJunior
то есть ежли абракадабра - то тоже пытаться парсить ?
источник

MB

Max B in phpGeeksJunior
Там есть проверка, возвращает ли контент
источник

АП

Алексей Пастушенко... in phpGeeksJunior
ок.. типа всякие локалхоты...
источник

АП

Алексей Пастушенко... in phpGeeksJunior
ладно
источник

АП

Алексей Пастушенко... in phpGeeksJunior
источник

P

Panda🤔 in phpGeeksJunior
Ря
источник

АП

Алексей Пастушенко... in phpGeeksJunior
между src и " - могут случаться и пробелы
источник

P

Panda🤔 in phpGeeksJunior
Где пср12
источник

АП

Алексей Пастушенко... in phpGeeksJunior
источник

АП

Алексей Пастушенко... in phpGeeksJunior
Сичтаешь что http://ya.ru и https://ya.ru/hello?asdasd - одинаковы ?
источник

АП

Алексей Пастушенко... in phpGeeksJunior
источник

MB

Max B in phpGeeksJunior
Нет
источник

P

Panda🤔 in phpGeeksJunior
Докер хде
источник

MB

Max B in phpGeeksJunior
Panda🤔
Докер хде
А надо было?
источник

АП

Алексей Пастушенко... in phpGeeksJunior
нахрена там continue
источник

P

Panda🤔 in phpGeeksJunior
Max B
А надо было?
Почему нет
источник

P

Panda🤔 in phpGeeksJunior
Вот мне никак не запустить сейчас
источник