1. Java / Говнокод #578

    +140.8

    1. 01
    2. 02
    3. 03
    4. 04
    5. 05
    6. 06
    7. 07
    8. 08
    9. 09
    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, 19 Февраля 2009

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

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

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

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