Нашли или выдавили из себя код, который нельзя назвать нормальным, на который без улыбки не взглянешь? Не торопитесь его удалять или рефакторить, — запостите его на говнокод.ру, посмеёмся вместе!
+140.8
private String mPrimaryCorrelationKey;
public String getPrimaryCorrelationKey()
{
long current = System.currentTimeMillis();
current++;
return new Long(current).toString();
}
public void setPrimaryCorrelationKey(String mPrimaryCorrelationKey) {
this.mPrimaryCorrelationKey = mPrimaryCorrelationKey;
}соответствующий геттер для этой проперти должен возвращать всегда уникальный id
Запостил: 
guest,
19 Февраля 2009
Follow us!
Ну я так делаю иногда:)
Че в этом такого?
А "такого" в нём вот что:
1) если два вызова get пройдут в одну и ту же миллисекунду, то вернётся одно и то же значение (current должен быть как минимум статиком, как максимум потокобезопасным (например AtomicLong)
2) для одного и того же объекта с течением времени будут возвращаться разные ID
3) наличие set функции у поля, которое должно быть уникальным, настораживает. Прикол ещё в том, что это установленное значение нигде не используется