Jawa / Говнокод #578 Ссылка на оригинал

0

  1. 1
  2. 2
  3. 3
  4. 4
  5. 5
  6. 6
  7. 7
  8. 8
  9. 9
  10. 10
  11. 11
  12. 12
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 guest, (Updated )

Комментарии (2) RSS

  • ava guest # Ссылка на оригинал 0

    Говногость (1 час вперед) и:
    Ну я так делаю иногда:)
    Че в этом такого?
    Ответить

  • ava guest # Ссылка на оригинал 0

    Дмитрий:
    А "такого" в нём вот что:

    1) если два вызова get пройдут в одну и ту же миллисекунду, то вернётся одно и то же значение (current должен быть как минимум статиком, как максимум потокобезопасным (например AtomicLong)
    2) для одного и того же объекта с течением времени будут возвращаться разные ID
    3) наличие set функции у поля, которое должно быть уникальным, настораживает. Прикол ещё в том, что это установленное значение нигде не используется
    Ответить

Добавить комментарий

Помни, guest, за тобой могут следить!

    А не использовать ли нам bbcode?

    ↑
    8
    ↓