sat1l / laba3

laba3

Structure

• the root folder acts as a folder for the IntelliJ IDEA project.
• UML folder contains .drawio, .svg, and .png files of the UML diagram of the project. .svg and .png variations are also displayed below.
• text.md containts the given text for completing the task.

SVG Diagram

PNG Diagram

laba3's Issues

Review

Ask.java

laba3/src/actions/Ask.java

Lines 13 to 17 in cea5b24

	public void ask(Person whoAsks, Person whoGetsAsked, String phrase, Interest whoAsksInterest){
	whoAsks.addDoing(this);
	System.out.println(whoAsks.getName() + " asks " + whoGetsAsked.getName() + " " + phrase + " with " + whoAsksInterest.toString());
	whoAsks.removeDoing(this);
	}

Предполагается, что вопрос задается с каким то интересом, но если на вход поступит аргумент пустой строка или вовсе null, стоит обработать возможные сценарии

FindItem.java

laba3/src/actions/FindItem.java

Line 40 in cea5b24

System.out.println(person.getName() + " searches for " + UObjectToFind.getName() + " on/in a " + somethingWithInventory.getName());

Небольшая странность по логике вывода. Программа оповещает о поиске только при повторной попытке это сделать.

DrinkAlco.java

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

EatSedative.java

laba3/src/actions/EatSedative.java

Lines 33 to 40 in cea5b24

	private UObject hasSedative(Person person){
	UObject toret = null;
	for (UObject item : person.getInventory()) {
	if (item.getName().equalsIgnoreCase("sedative")) {
	toret = item;
	break;
	}
	}

Программа ищет нужный предмет для поедания, ориентируясь на наименование. Стоит вынести обезболивающие в отдельный класс Sedative и "принимать в употребление" только обьекты этого класса

Review 1

Naming Conventions

Названия всех enum и интерфейсов должны именоваться по тем же правилам, что и классы. Здесь же все названия enums капсом и есть интерфейсы, начинающиеся с маленькой буквы (hasOccasion, hasAlcohol)

(Мнение) Один из классов назван с использованием транслита, выглядит странно

laba3/src/actions/NewTuchi.java

Line 7 in 7075438

public class NewTuchi extends Action {

Ненужные методы

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

laba3/src/parentclasses/Location.java

Lines 17 to 19 in 7075438

	public String getLocname(){
	return locname;
	}

laba3/src/parentclasses/Location.java

Lines 30 to 33 in 7075438

	@Override
	public String toString(){
	return locname;
	}

Работа метода bringTogether()

(Мнение)Странно, что проверяется, что только мужчина может приносить какие-либо предметы

laba3/src/actions/BringTogether.java

Lines 17 to 18 in 7075438

	public void bringTogether(Couple couple, UObject UObject, Location location){
	if (couple.getHusband().getInventory().contains(UObject)){ //yes they bring thing as a couple but actually the man carries the thing

Метод drinkAlco()

Было бы хорошо добавить проверку соответствия локации человека локации того, что он будет пить

laba3/src/actions/DrinkAlco.java

Line 16 in 7075438

public void drinkAlco(Person person, Beverage beverage, AMOUNT amount){