search

JujaLabs / itevents

Resource to subscribe on it events
Apache License 2.0
7 stars 5 forks source link

#122 #136

Closed vaa25 closed 8 years ago

vaa25 commented 8 years ago

connect to #122

witjem commented 8 years ago
alex-anakin commented 8 years ago

@vaa25 Классы EventTechnologySqlBuilder, FilteredEventsSqlBuilder, FilterTechnologySqlBuilder.

Что означают magic numbers в этих выражениях? Почему у них именно такие значения? sql.delete(sql.length() - 2, sql.length()); sb.delete(sb.length() - 4, sb.length());

vaa25 commented 8 years ago

@alex-anakin Столько символов нужно отнять чтобы убрать лишнюю запятую в конце

alex-anakin commented 8 years ago

@vaa25, через месяц некий Вася захочет изменить билдеры (добавить метод и т.п.). Ему тоже надо будет к тебе обращаться за личной консультацией?

vaa25 commented 8 years ago

@alex-anakin Предложи как ее назвать, у меня фантазии нет

romach commented 8 years ago

@vaa25 lastCommaPositionFromEnd

vkuchyn commented 8 years ago

Please, edit your comments above and put person name to whom you are writing (person nick starts with @). Leave this habbit for future @alex-anakin https://github.com/JuniorsJava/itevents/pull/136#issuecomment-179201038 @alex-anakin https://github.com/JuniorsJava/itevents/pull/136#issuecomment-179202829 @vaa25 https://github.com/JuniorsJava/itevents/pull/136#issuecomment-179202178 @vaa25 https://github.com/JuniorsJava/itevents/pull/136#issuecomment-179205690 @romach https://github.com/JuniorsJava/itevents/pull/136#issuecomment-179205690