search

ikucherenko / go-java-1

Automatically exported from code.google.com/p/go-java-1
0 stars 0 forks source link

Photographer Organizer Project #10

Open GoogleCodeExporter opened 9 years ago

GoogleCodeExporter commented 9 years ago 
1) should be extracted Domain Model classes

Original issue reported on code.google.com by kolesa.a...@gmail.com on 20 Jan 2015 at 1:54

GoogleCodeExporter commented 9 years ago

Original comment by kolesa.a...@gmail.com on 3 Feb 2015 at 1:05

GoogleCodeExporter commented 9 years ago 
Привет! Постарался сделать, как обсуждали.

Original comment by alex.sas...@gmail.com on 6 Feb 2015 at 3:05

GoogleCodeExporter commented 9 years ago 
Привет, то что наревьювил по пунктам:
1) все классы реализуют интерфейс ShowInfo с 
методом displayInfo(), где логика представления 
объекта. 
а) логика представления в модели не нужна
б) если нужно отобразить объект (например в 
логах) можем сгенерить toString()
2) присмотрись к классу java.util.Date
он много где пригодиться по коду. например 
такую пачку параметров заменить 
setReserveDate(int year, int month, int date, int hourOfDay, int minute) 
3) в методе timeEditingFotosInPackage() нарушен Information 
Expert 
4) в методе isOutOfDeadline() есть побочный эффект
6) у классов Customer, Order и др. можно дабавить 
айдишники, так как нет идентификаторов
7) по модели надо обсудить по скайпу

Original comment by kolesa.a...@gmail.com on 12 Feb 2015 at 11:22

GoogleCodeExporter commented 9 years ago 
Привет!
попробовал исправить по всем пунктам.
3)попытался исправить, не знаю, насколько 
правильно.
4)не разобрался с побочным эффектом
7)напиши мне, когда тебе удобно будет по 
скайпу поговорить.

Original comment by alex.sas...@gmail.com on 19 Feb 2015 at 10:36