V.Sultanyarov HomeWork #2
Conversation
akrasenkov
left a comment
akrasenkov
left a comment
There was a problem hiding this comment.
Не описан интерфейс и не реализован "Экран". Так всё тоже работает, но в задании была запрошена его реализация.
src/edu/androidclub/Application.java
Outdated
| ) | ||
| .getName() // Получить имя предмета | ||
| );*/ | ||
| Item item ; |
There was a problem hiding this comment.
По гайдлайнам Java, пробел перед ; не ставится. Кроме того, здесь можно строки Item item ; и item = itemBox.emit(new Coordinates(1, 1)); объединить в одну (необязательно, но так лучше).
src/edu/androidclub/Application.java
Outdated
| public static class Cola extends Item { | ||
| public Cola() { | ||
| super("Cola"); // Вызов конструктора класса-родителя (класса Item) | ||
| super("Cola",50); // Вызов конструктора класса-родителя (класса Item) |
src/edu/androidclub/Application.java
Outdated
| public static class Sprite extends Item { | ||
| public Sprite() { | ||
| super("Sprite"); | ||
| super("Sprite",55); |
src/edu/androidclub/ProductsBox.java
Outdated
| Stack<Item> items = scheme.get(coordinates); | ||
| if(items.empty()){ | ||
| return false; | ||
| }else{ |
| if(items.empty()){ | ||
| return false; | ||
| }else{ | ||
| return true; |
There was a problem hiding this comment.
Так тоже ничего, но в данном случае можно заменить ветвление на return !items.empty(); - "вернуть true если НЕ пусто".
src/edu/androidclub/Monitor.java
Outdated
| public class Monitor implements AppScreen { | ||
|
|
||
| public void printText(String text){ | ||
|
|
src/edu/androidclub/Application.java
Outdated
| item = itemBox.emit(new Coordinates(1, 1)); | ||
| System.out.println("| Выдан товар: " + item.getName() +" по цене - " + item.getCost() + "|"); // OK | ||
| Item item = itemBox.emit(new Coordinates(1, 1)); | ||
| monitor.printText("| Выдан товар: " + item.getName() +" по цене - " + item.getCost() +"|");// OK |
There was a problem hiding this comment.
Лучше вынести рамку в Monitor.
src/edu/androidclub/Application.java
Outdated
| Stack<Item> sprites = new Stack<>(); // 2 | ||
|
|
||
| //Создал монитор для вывода | ||
| Monitor monitor = new Monitor(); |
There was a problem hiding this comment.
Абсолютно правильно. Можно также для нормализации сделать AppScreen monitor = new Monitor();. Надеюсь, суть такого финта прослеживается)
akrasenkov
left a comment
akrasenkov
left a comment
There was a problem hiding this comment.
Рамки не перенесены в Monitor.
akrasenkov
left a comment
akrasenkov
left a comment
There was a problem hiding this comment.
Рамки всё ещё не перенесены...
2 participants
No description provided.