Домашнее задание №4 - Броузерное приложение и балансировщик #1169
Conversation
balancer/memcached/Dockerfile
Outdated
| @@ -0,0 +1 @@ | |||
| FROM memcached:alpine | |||
There was a problem hiding this comment.
Не нужно описывать образ с одним FROM. Просто используйте в docker-compose сам базовый образ memcached:alpine
There was a problem hiding this comment.
Вот здесь не понял, будьте добры подробнее опишите
There was a problem hiding this comment.
Просто используйте в docker-compose секцию image и там укажите вот этот образ.
| @@ -0,0 +1,44 @@ | |||
| <?php | |||
There was a problem hiding this comment.
Какие конкретно замечания, я их не вижу.
There was a problem hiding this comment.
Откройте пулл-реквест в github на последней вкладке - где показываются изменения.
There was a problem hiding this comment.
Замечания линтера так и не поправлены.
Вы их можете посмотреть и на странице https://github.com/otusteamedu/PHP_2023/pull/1169/checks, там есть подсвеченный красным шаг пайплайна, нажмите на него и будет вывод код снифера. Так же вы можете запустить снифер локально.
balancer/src/code/Validator.php
Outdated
| $message = []; | ||
| //Проверяем строку регуляркой, чтобы не прошла строка )( | ||
| if (preg_match("/^\(.*\)/", $postVar)){ | ||
| if (preg_match_all("/(\()/u", $postVar, $m)){ |
There was a problem hiding this comment.
Излишне усложнили. Есть более простое и эффективное решение - простой цикл в котором перебираем символа и увеличиваем счётчик на единицу если видим '(' и уменьшаем если видим ')'. Если во время итерации счётчки стал < 0, лишняя ')'. Если после цикла счётчик != 0 - то лишняя '(' или ')'
balancer/src/code/Validator.php
Outdated
| for ($i=1; $i<=$openCount; $i++){ | ||
| //Если в процессе выполнение понимаем что строка пуста, отдаем 200 код, все ок | ||
| if (empty($postVar)){ | ||
| $message['code'] = 200; |
There was a problem hiding this comment.
Задача данного класса - провалидировать строку. Он не должен формировать HTTP ответ. Просто верните отсюда bool.
| @@ -0,0 +1,44 @@ | |||
| <?php | |||
There was a problem hiding this comment.
Откройте пулл-реквест в github на последней вкладке - где показываются изменения.
| @@ -0,0 +1,44 @@ | |||
| <?php | |||
balancer/nginx/Dockerfile
Outdated
| @@ -0,0 +1,14 @@ | |||
| FROM ubuntu:latest | |||
There was a problem hiding this comment.
Есть образ с уже установленым nginx лучше использовать его.
https://hub.docker.com/_/nginx
| @@ -0,0 +1,44 @@ | |||
| <?php | |||
There was a problem hiding this comment.
Замечания линтера так и не поправлены.
Вы их можете посмотреть и на странице https://github.com/otusteamedu/PHP_2023/pull/1169/checks, там есть подсвеченный красным шаг пайплайна, нажмите на него и будет вывод код снифера. Так же вы можете запустить снифер локально.
No description provided.