1. C++ / Говнокод #2344

    +59.5

    1. 1
    2. 2
    3. 3
    4. 4
    5. 5
    6. 6
    7. 7
    8. 8
    9. 9
    //и теперь шлём запрос письма
                    reply = QString("<iq type='get' from='%1' to='%2' id='mail-request-%3'>"\
                        "<query xmlns='google:mail:notify' %4 %5/></iq>")
                            .arg(from)
                            .arg(to)
                            .arg(id)
                            .arg((lastCheck.value(QString("%1").arg(account),"")=="")?QString(""):QString("newer-than-time='%1'").arg(lastCheck.value(QString("%1").arg(account))))
                            .arg((lastTid.value(QString("%1").arg(account),"")=="")?QString(""):QString("newer-than-tid='%1'").arg(lastTid.value(QString("%1").arg(account))));
                    stanzaSender->sendStanza(account, reply);

    Раскопано в недрах плагинов к psi:
    Особенно радует шедевральная конвертация int'а в строку при помощи QString("%1").arg(account),"")
    Плюс ко всему комменты на русском в интернациональном проекте это немножко моветон

    Запостил: Sauron, 29 Декабря 2009

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

    • ava nolka4 29.12.2009 05:45 # 0

      каменты на русском - это нармалек, чоты ;)
      Ответить

    • ava VampiRUS 29.12.2009 07:48 # +2

      Спасибо, буду исправляться :).
      Но можно было, это самое замечание в комменты на страничке проекта оставить.
      А то непонятно, чего ты этим постингом здесь хотел добиться, его бы могли и не заметить и ещё долгое время код мог оставаться таким.
      Ответить

      • ava Sauron 01.01.2010 15:00 # −1

        Да там весь код ужас какой то %) Множественное наследования, простыни непонятных действий. Жесть какая то, неужели не придумать, как написать лаконичнее?
        Ответить

    • ava explosion_head 29.12.2009 12:02 # 0

      в хорошем коде комментарии не нужны. Если ты пишешь код production качества, то он должен сам говорить за себя. Если и хочется что то прокомментировать, то нужно не дублировать код комментариями, а объяснять почему и зачем так было сделано.
      Если не получается разработать понятный и доступный для чтения код, то это говорит о плохой архитектуре проекта
      Ответить

    • ava guest 29.12.2009 15:47 # +1

      >комменты на русском в интернациональном проекте это немножко моветон

      В целом соглашусь, лучше хороший русский, чем неграмотный до нечитабельности английский.
      Ответить

    • ava Coach 30.12.2009 14:49 # 0

      "int'а в строку при помощи QString("%1").arg(account)"
      вполне нормальная конвертация, в духе Qt
      Ответить

      • ava Sauron 01.01.2010 15:01 # 0

        А про QString::number значит не слышали, да?
        Ответить

        • ava Coach 11.01.2010 10:11 # 0

          теперь услышал, но приём то, тем не менее, распространённый
          Ответить

    • ava VampiRUS 02.01.2010 04:48 # +1

      Нет желания оправдываться, но комментируется там исключительно гугловкое апи
      множественное наследование - система плагинов, есть что-то удобнее/лучше - предложите
      Ответить

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