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

parrot.ru

Рейтинг популярных групп и каналов

В рейтинге участвует:

групп:

каналов:

Виртуальный сервер на SSD - недорого!

Аренда выделенных и виртуальных серверов (VDS/VPS), хостинг, аренда IP-адресов, администрирование, круглосуточная поддержка

qwarta.ru подробнее

Резервное копирование с проверкой на вирусы!!!

Удобный сервис создания резервных копий на любой сервер сети интернет. Отслеживайте изменения, проверяйте на вирусы. Надежно защитите свой бизнес!

go.backupland.com

Выбираете сервер? Любая конфигурация на заказ!

Аренда физических серверов любых конфигураций под любые запросы - 1С бухгалтерия, игровые сервера, нагруженные проекты, интернет-магазины!

qwarta.ru подробнее

Size: a a a

PHP

1622 membersпожаловаться на группу

2019 December 25

S

Sergey in PHP

Daniil

Всем привет, делал тестовое, зададача была сделать применяя патерны, тестовое сделал, в фидбеке получил ответ что "низкая гибкостью к изменениям, не продуманные названия классов и методов" кто поможет советом, не знаю как сделать еще гибче решение https://github.com/ldaniil/weather

ldaniil/weather

Weather application. Contribute to ldaniil/weather development by creating an account on GitHub.

Правильно ревьюверы указали на гибкость и именование - все плохо.
Для наглядности, сделай мердж-реквест с добавлением еще одной апишки для погоды.
Только без глобального рефакторинга без учета советов что тебе уже дали.
Вперед

источник

22:50пожаловаться #1

˸A

˸̧̨ ͅBlack Akula˸̧̨ ͅ ̤ ̬̪ in PHP

DI не панацея, а клей для говна

Можно и без DI реализовывать API&SPI, но зачем... 😱

источник

22:59пожаловаться #2

˸A

˸̧̨ ͅBlack Akula˸̧̨ ͅ ̤ ̬̪ in PHP

Sergey

Правильно ревьюверы указали на гибкость и именование - все плохо.
Для наглядности, сделай мердж-реквест с добавлением еще одной апишки для погоды.
Только без глобального рефакторинга без учета советов что тебе уже дали.
Вперед

Классный совет. Нас препод в универе именно так учил: давал задачу, мы реализовывали, а потом втыкал такие дополнительные требования, что нам приходилось рефакторить наш код, чтоб реализовать дополнительные требования. Лучший препод по программированию

источник

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

NT

Nikolay Turskyi in PHP

˸̧̨ ͅBlack Akula˸̧̨ ͅ ̤ ̬̪

Классный совет. Нас препод в универе именно так учил: давал задачу, мы реализовывали, а потом втыкал такие дополнительные требования, что нам приходилось рефакторить наш код, чтоб реализовать дополнительные требования. Лучший препод по программированию

Ты не можешь продумать все действия, которые могут произойти на перед. Но тут в задаче уже было сразу условие - погоду можно будет брать из любого публичного Апи. Соответственно от этого надо было отталкиваться. Создать 2 Апи. Создать 2 Адаптера с единым интерфейсом, которые возвращают DTO и т.д. Шаг за шагом. А человек видимо начал с entity, storage и т.д.

источник

23:15пожаловаться #4

˸A

˸̧̨ ͅBlack Akula˸̧̨ ͅ ̤ ̬̪ in PHP

Nikolay Turskyi

Ты не можешь продумать все действия, которые могут произойти на перед. Но тут в задаче уже было сразу условие - погоду можно будет брать из любого публичного Апи. Соответственно от этого надо было отталкиваться. Создать 2 Апи. Создать 2 Адаптера с единым интерфейсом, которые возвращают DTO и т.д. Шаг за шагом. А человек видимо начал с entity, storage и т.д.

Так и требования препода были в рамках разумного. Потом многие поломали ноги о свой говнокод...

источник

23:17пожаловаться #5

2019 December 26

ВГ

Влад Гречишкин in PHP

Всем доброго времени суток) Может кто-то подсказать реализациию, либо поделится исходниками, возможно кто-то делал выпадающий список с метками в Google Maps (API) ?

источник

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

ВГ

Влад Гречишкин in PHP

в документации получилось найти только кастомный контроллер, который при клике ведет в центр карты
https://developers-dot-devsite-v2-prod.appspot.com/maps/documentation/javascript/examples/control-custom

Google Developers

Custom Controls | Maps JavaScript API | Google Developers

источник

01:07пожаловаться #7

С

Сергей З. in PHP

Константин Грачев

https://github.com/ldaniil/weather/blob/master/core/storage/weather/StorageFactory.php#L42
Вот таких вещей быть не должно

ldaniil/weather

Weather application. Contribute to ldaniil/weather development by creating an account on GitHub.

А почему таких вещей быть не должно? Я видимо пропустил объяснение

источник

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

С

Сергей З. in PHP

Свои файбрики должны быть?

источник

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

КГ

Константин Грачев in PHP

Сергей З.

А почему таких вещей быть не должно? Я видимо пропустил объяснение

Тред почитай, ответы были даны

источник

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

С

Сергей З. in PHP

а o/c принцип, понятно. Со второго раза нашел)

источник

10:31пожаловаться #11

D

Daniil in PHP

Константин Грачев

Тред почитай, ответы были даны

выпилил я эту фабрику вообще

источник

11:30пожаловаться #12

SZ

Sergey Zolotov in PHP

источник

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

BP

Borys Poletayev in PHP

Для sql?

источник

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

D

Тогда уже Какую СУБД используете

источник

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

D

хотелось бы пощупать cassandra(сцилла),тарантул,кликхаус

источник

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

Р

Руслан in PHP

хотелось бы пощупать cassandra(сцилла),тарантул,кликхаус

Щупай, кто мешает то?

источник

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

SP

Sergey Protko in PHP

а если я юзаю постгрес в рамках авроры? это уже не постгрес?

источник

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

SP

Sergey Protko in PHP

или ты не про aws aurora?

источник

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

SZ

Sergey Zolotov in PHP

а если я юзаю постгрес в рамках авроры? это уже не постгрес?

ну она ж не постгря. только совместимая

источник

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