Conversation
…on from GameVisualizer to Drawable interface
…ions removed from GameVisualizer
…ced by constructor local variables
…alls replaced by controller calls, model and controller creation moved to MainApplicationFrame contructor
… ModelPositionController
… and Update method removed
…bot representation changed.
…pt, RobotRepresentation interface cleanup | issue #3
|
Стоило базировать эту ветку на второй задаче (с сохранением состояния), чтобы была возможность оценить удачность модели, выбранной во второй задаче. А сейчас будет неприятно вливать одну ветку в другую (будет много конфликтов, причем они не будут решаться легко, поскольку потребуется дописывать функциональность сохранения для нового окна). |
|
Так что здесь в качестве упражнения попрошу или влить (merge) ветку второй задачи в эту ветку, или эту ветку в ветку второй задачи (чтобы понять, почему выбранный способ исходного формирования ветки неудачен). |
| public class TermSetupGui extends JPanel { | ||
| private RepresentationTermOval m_term; | ||
|
|
||
| public TermSetupGui(RepresentationTermOval m_term){ |
There was a problem hiding this comment.
Здесь не хватает менеджера раскладки для поддержки красивой компоновки окна при изменении его размера. Сейчас, например, можно сделать окошко меньше по размеру, чем требуется для отображения содержимого. И содержимое будет обрезаться. С помощью менеджера раскладки и за счет указания минимального размера окна можно было бы добиться красивого вида во всех условиях.
|
Суть задачи я засчитываю. Но тут основное замечание (за что пока снижаю баллы) - ветка базирована на неудачной точке. В результате решения 2-й и 3-й задачи противоречат друг другу с точки зрения реализации. То есть совместить правки будет сложно. Если удастся решить эту проблему (выполнить слияние веток с сохранением всей логики) будет (почти) полный балл. Ну и замечание про менеджер раскладки хотелось бы увидеть реализованным. |
2 participants
No description provided.