Alex dolgov/hw5 #576
Alex dolgov/hw5 #576
Conversation
docker/memcached/Dockerfile
Outdated
| @@ -0,0 +1 @@ | |||
| FROM memcached:latest | |||
There was a problem hiding this comment.
Если мы не собираемся как-то менять базовый образ, то нам нет нужды описывать dockerfile, можем просто использовать базовый образ.
src/validator.php
Outdated
|
|
||
| use Exception; | ||
|
|
||
| function validateString(string $string): bool { |
There was a problem hiding this comment.
Обратите внимание на предупреждение линтера и исправьте пожалуйста форматирование.
src/index.php
Outdated
| @@ -0,0 +1,25 @@ | |||
| <?php | |||
There was a problem hiding this comment.
Пожалуйста, используйте паттерн фронт-контроллер. Пусть в index.php будет только инициализация приложения и его запуск:
$app = new App();
echo $app->run();
src/validator.php
Outdated
| @@ -0,0 +1,35 @@ | |||
| <?php | |||
There was a problem hiding this comment.
Данный файл лежит в папке, которую обслуживает nginx. Как итог, пользователь может запросить её напрямую - это плохо. Нужно создать отдельную папку public, где будет только index.php.
| { | ||
| public static function validateString(string $string): bool | ||
| { | ||
| if (preg_match('/^(\((?1)*\))*$/', $string)) { |
There was a problem hiding this comment.
Можно просто return preg_match('/^(\((?1)*\))*$/', $string)
No description provided.