search

Lemyr621 / totoshka

0 stars 0 forks source link

Замечания по lw7 #8

Closed ivklgn closed 5 years ago

ivklgn commented 5 years ago
  1. Форматирование кода PHP должно быть как и js - 2 пробела. Форматирование файла летит
  2. Нет 2 маршрутов для получения вылета по id и получения всех вылетов (GET маршруты)
  3. Неверно построен обработчик запросов. Ваш API для клиента должен реализовывать хотя бы такой интерфейс:
GET /flight_tickets.php - получили все билеты
GET /flight_tickets.php?id=1 - получили билет с id = 1
POST /flight_tickets.php - добавили билет
UPDATЕ /flight_tickets.php?id=1 - обновили билет с id = 1
DELETE /flight_tickets.php?id=1 - удалили билет с id = 1

  1. Маршруты для удаления и обновления сделаны неверно. Если вы в Postman попробуете сделать DELETE/UPDATE запросы - ничего не произойдет. CRUD не строится только на POST и GET запросах

  2. API должна возвращать понятный ответ при выполнении действия, а не делать беребрасывание через header('location: index.php');

  3. Нет проверок на входные параметры для обработчика

ivklgn commented 5 years ago
  1. Отформатируйте код в едином стиле:
  $position = strpos($string, '&');
  $destination=substr($string, 0, $position); // присвоение должно быть разделено пробелами

  1. var_dump существует для отладки, в PUT и DELETE следует так же как и в GET/POST вернуть JSON с полезной информацией. Например при PUT {info: 'update successful', удаление тоже это касается

  2. в удалении нет проверки на входной id.