Skip to content

Lab 2 improvements#9

Open
elteammate wants to merge 6 commits intoDimch84:masterfrom
elteammate:lab02-improvements
Open

Lab 2 improvements#9
elteammate wants to merge 6 commits intoDimch84:masterfrom
elteammate:lab02-improvements

Conversation

@elteammate
Copy link

Небольшие изменения во вторую лабораторную работу, разбитые на 6 коммитов с индивидуальными изменениями.

  1. Lab 2: disallow "id" and "icon" fields in PUT /product - согласно спецификации, PUT имеет право менять id, а после задания "В" еще и icon. Менять icon странно, так как это read-only поле, указывающее на имя/путь к файлу. Менять id возможно (формально, REST API не ограничивает это действие), но это добавляет заморочек (особенно в реальных REST API при работе с ORM в бэкенде), и как правило не оправдано.

  2. Lab 2: hint on icon initialization - подсказывает, что можно инициализировать картинки чем-то фиксированным, или оставлять в поле null. Это разрешает небольшую неоднозначность, которая возникает в предыдущей фразе "Для простоты будем считать, что у каждого продукта картинка одна", так как это предполагает присутствие какой-то картинки после сразу запроса POST /product. Вопрос: откуда эту картинку взять? Я предлагаю 2 варианта решения, и, думаю, оба разумны. Конечно, все равно есть некоторая неоднозначность (картинка одна, а может быть нуль), но это кажется приемлемым в присутсвии такого комментария.

  3. Lab 2: specify new product schema in its entirety - просто полностью выписывает схему для продукта в задании B. Потому что почему бы и нет? Она немного странно смотрелать в виде одной единственной строчки.

  4. Lab 2: allow imprecise arithmetic - если решать эту задачу честно, то возникает проблема с тем, что последний пакет может иметь размер меньше L, и в задаче минимизации требуется целочисленное деление. Как будто это упрощение задумывалось, и я его озвучил.

  5. Lab 2: improve wording - теперь предложение читается лучше.

  6. Lab 2: remove future warning - убирается сточка "В последующих домашних заданиях вам будет предложено расширить функционал данного сервиса", так как в дальнейших лабах я не нашел отсылок к ней. На всякий случай, это стоит проверить не мне!

Можете друпнуть или ревертнуть изменения, которые Вам не понравились. Не забудьте сделать squash при мерже.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant