Говнокод

public static Document getDoc(CarPartsAd ad) {
        Document doc = new Document();
        doc.add(new Field("id", ad.getId().toString(), YES, NOT_ANALYZED));
        doc.add(new Field("mark", Long.toString(ad.getMarkId()), Field.Store.NO, NOT_ANALYZED));
        doc.add(new Field("model", Long.toString(ad.getModelId()), Field.Store.NO, NOT_ANALYZED));
        doc.add(new Field("frame", Long.toString(ad.getFrameId()), Field.Store.NO, NOT_ANALYZED));
        doc.add(new Field("engine", Long.toString(ad.getEngineId()), Field.Store.NO, NOT_ANALYZED));
        doc.add(new Field("type", Long.toString(ad.getTypeId()), Field.Store.NO, NOT_ANALYZED));
        doc.add(new Field("front_rear", Long.toString(ad.getFrontRear()), Field.Store.NO, NOT_ANALYZED));
        doc.add(new Field("right_left", Long.toString(ad.getRightLeft()), Field.Store.NO, NOT_ANALYZED));
        doc.add(new Field("up_down", Long.toString(ad.getUpDown()), Field.Store.NO, NOT_ANALYZED));
        doc.add(new Field("contact", Long.toString(ad.getContactId()), Field.Store.NO, NOT_ANALYZED));
        doc.add(new Field("town", Long.toString(ad.getTownId()), Field.Store.NO, NOT_ANALYZED));
        doc.add(new Field("part_state", Long.toString(ad.getPartState()), Field.Store.NO, NOT_ANALYZED));
        doc.add(new Field("original_flag", Long.toString(ad.getOriginalFlag()), Field.Store.NO, NOT_ANALYZED));
        doc.add(new Field("price", Integer.toString(ad.getPriceRub()), Field.Store.YES, NOT_ANALYZED));
        doc.add(new Field("price_flag", (ad.getPriceRub() > 0 ? "0" : "1"), Field.Store.NO, NOT_ANALYZED));
        doc.add(new Field("date_sort", Long.toString(ad.getSort()).substring(0, 9), Field.Store.YES, NOT_ANALYZED));
        doc.add(new Field("oem", format(ad.getOem()).replaceAll(" ", "").toLowerCase(), Field.Store.NO, NOT_ANALYZED));
        doc.add(new Field("producer_code", format(ad.getProducerCode()).replaceAll(" ", "").toLowerCase(), Field.Store.NO, NOT_ANALYZED));
        doc.add(new Field("uplift_rating", Integer.toString(ad.getUpliftRating()), Field.Store.NO, NOT_ANALYZED));
        doc.add(new Field("ads_source", Integer.toString(ad.getAdsSource()), Field.Store.NO, NOT_ANALYZED));
        return doc;
    }

лучик ненависти авторам lucene за то, что заставляют писать такое :)

Запостил:

zlob.jc, 20.10.2010 (Updated 24.03.2018)

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

sectus 16 лет ago # 0

Лень писать? : )
Меня смущает только последний параметр: чего это он всё время повторяется? А в остальном "всё пучком".
Ответить
- zlob.jc 16 лет ago # 0 ⇈
  
  встречая в коде иногда пугаюсь, а как надо что то поправить пью чашку кофе для храбрасти.
  последний параметр обязательный, ну и в моем случае всегда тот же , из песни слов не выкинешь 🙂
  Ответить
  - sectus 16 лет ago # 0 ⇈
    
    Не увидел я здесь говнокода.
    Ответить
Lure Of Chaos 16 лет ago # 0

мне особенно нравится
Long.toString
Integer.toString
к этому
ad.getId().toString

и еще, когда нибудь ad==null

а вообще, я думаю, и здесь можно выкрутиться и обойтись без копипасты.
Ответить
- Bugdog 16 лет ago # 0 ⇈
  
  кодогенерацией 🙂
  Ответить
  - zlob.jc 16 лет ago # 0 ⇈
    
    в проекте всего 3 таких метода, стоит городить самокат? 🙂
    Ответить
3.14159265 16 лет ago # 0

так включи голову и не пиши.

говно не в люцене, а в сабже, код можно сократить съекономив на doc.add(new Field(, toString и NOT_ANALYZED например.

а настройки полей для индексации вынести в массив или конфиг
Ответить
- zlob.jc 16 лет ago # 0 ⇈
  
  //а настройки полей для индексации вынести в массив или конфиг
  есть такой принцип не плодить лишных сущностей
  Ответить
  - Lure Of Chaos 16 лет ago # 0 ⇈
    
    не плодить лишных сущностей
    и писать на ассемблере, без подпрограмм и тем более ООП
    Ответить
    - xXx_totalwar 16 лет ago # 0 ⇈
      
      http://bit.ly/aW9ajd
      Ответить
nikelin 16 лет ago # 0

Ммм, автор не пробовал использовать аннотации или выносить это дело в XML, на худой конец?
Ответить