search

RamSaw / SoftwareDesign

Repository for software design course assignments
MIT License
0 stars 1 forks source link

Hw07, Farutin, Egorov, Pravilov #6

Open VadimFarutin opened 5 years ago

ArtyomLobanov commented 5 years ago

Добавьте README с инструкцией что ли. Какие клавиши что делают? Как загрузить карту из файла? И ссылку на архитектурное описание туда же можно добавить

vladimirrim commented 5 years ago

@ArtyomLobanov пофиксили все замечания

RamSaw commented 5 years ago

Исправления перенесите, пожалуйста, в основную ветку. Дальше для удобства будем поддерживать и исправлять одну (актуальную) версию проекта, а не по отдельной для каждого дз.

@ArtyomLobanov а как предлагается дальше делать? Мержить в master? Просто это по сути наверное это и есть основная ветка, потому что в ней появился первый код рогалика. А как потом будет в 8 дз? Мы его сразу смержим в это дз до проверки или смержим после проверки? Просто кажется, что мержить будет очень больно, там же архитектура довольно сильно менялась...

RamSaw commented 5 years ago

@ArtyomLobanov пофиксили код и диздок. Но диздок скоро опять изменится, так как туда добавится сетевой рогалик.

ArtyomLobanov commented 5 years ago

Исправления перенесите, пожалуйста, в основную ветку. Дальше для удобства будем поддерживать и исправлять одну (актуальную) версию проекта, а не по отдельной для каждого дз.

@ArtyomLobanov а как предлагается дальше делать? Мержить в master? Просто это по сути наверное это и есть основная ветка, потому что в ней появился первый код рогалика. А как потом будет в 8 дз? Мы его сразу смержим в это дз до проверки или смержим после проверки? Просто кажется, что мержить будет очень больно, там же архитектура довольно сильно менялась...

Не важно, какая ветка будет основной. Эта вполне подходит. Хотелось собрать где-то актуальную и исправленную версию проекта, чтобы не повторять одни и те же замечания/исправления в каждом дз с нуля. Если сильно перелопатили проект в сетевом рогалике, ок - пусть будет в отдельной ветке.

VadimFarutin commented 5 years ago

И ещё баг: если уменьшить окошко игры, а потом растянуть, то получается что-то неожиданное. Сильно закапываться в это, наверное, не стоит. Может, библиотека предоставляет возможность запретить изменение размера окна?

Не нашел такой функциональности и способов ее достичь ни в этой версии библиотеки, ни в новой. Добавил полный сброс и перерисовку содержимого при изменении размеров, но это тоже не помогло.

VadimFarutin commented 5 years ago

@ArtyomLobanov Исправил и слил в эту ветку все версии, кроме сетевой

VadimFarutin commented 5 years ago

И сохранения не удаляются при смерти

Добавил

VadimFarutin commented 5 years ago

@ArtyomLobanov Исправил и ответил

VadimFarutin commented 5 years ago 
1. Пару тестов на работу стратегий добавьте ещё.
2. А как использовать вещи, полученные после 10 уровня? Их даже увидеть нельзя
   (Кстати, тут ещё видно баг с несоответствием уровня у декоратора)
3. Если сейв удалён, то при попытке загрузиться всё ломается
  1. Добавил тесты
  2. Теперь вещи выводятся в столбик. Максимальное количество ограничено в BasePlayer значением 10, поэтому наследники должны сами выбирать, что делать, когда экипировки становится больше (Player просто перестает добавлять новые, можно заменять старые, например)
  3. Fixed
VadimFarutin commented 5 years ago

@ArtyomLobanov Done (и еще при растягивании окна теперь все-таки не рисуется мусор)

ArtyomLobanov commented 5 years ago

Комментарий куда-то не туда добавился

Просто довольно странные ситуации получаются. Например, вот сейчас трусливый моб может убежать наверх, но вместо этого стоит на месте image Собственно, это последний исправимый штраф за второе дз.

ArtyomLobanov commented 5 years ago

Ограниченный инвентарь, который автоматически пополняется и из которого ничего нельзя выкинуть, это костыль, конечно. Не надо так.