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

    +12

    1. 01
    2. 02
    3. 03
    4. 04
    5. 05
    6. 06
    7. 07
    8. 08
    9. 09
    10. 10
    11. 11
    12. 12
    13. 13
    14. 14
    15. 15
    16. 16
    17. 17
    18. 18
    AttachmentService::MimeType AttachmentService::mimeTypeByCodeKey(const QString &codeKey) const
 {
     if(codeKey.toLower() == QString("MimeType_PDF").toLower())       return MimeType_PDF;
     if(codeKey.toLower() == QString("MimeType_JPEG").toLower())      return MimeType_JPEG;
     if(codeKey.toLower() == QString("MimeType_PNG").toLower())       return MimeType_PNG;
     if(codeKey.toLower() == QString("MimeType_HTML").toLower())      return MimeType_HTML;
     if(codeKey.toLower() == QString("MimeType_ZHTML").toLower())     return MimeType_ZHTML;
     if(codeKey.toLower() == QString("MimeType_TEXT").toLower())      return MimeType_TEXT;
     if(codeKey.toLower() == QString("MimeType_XLS").toLower())       return MimeType_XLS;
     if(codeKey.toLower() == QString("MimeType_DOC").toLower())       return MimeType_DOC;
     if(codeKey.toLower() == QString("MimeType_PPT").toLower())       return MimeType_PPT;
     if(codeKey.toLower() == QString("MimeType_GIF").toLower())       return MimeType_GIF;
     if(codeKey.toLower() == QString("MimeType_BMP").toLower())       return MimeType_BMP;
     if(codeKey.toLower() == QString("MimeType_MPEG").toLower())      return MimeType_MPEG;
     if(codeKey.toLower() == QString("MimeType_MP4").toLower())       return MimeType_MP4;
     if(codeKey.toLower() == QString("MimeType_MP3").toLower())       return MimeType_MP3;
     return MimeType_UNKNOWN;
}

    Коллега откопал в моём коде, написанном под какими-то веществами по ходу.
    Изначально было сравнение строк как есть, без приведения в нижний регистр, а потом понадобилось подстраховаться от левого регистра символов.

    Запостил: FlySnake, 09 Мая 2014

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

    • ava Мистер Хэнки 09.05.2014 14:53 # +1

      в плюсах нет Enum.valueOf ?
      Ответить

      • ava bormand 09.05.2014 15:08 # +2

        Нету. Там enum это просто набор именованных целочисленных констант, да еще и с кривым скопом.
        Ответить

    • ava defecate-plusplus 09.05.2014 17:58 # 0

      > было сравнение строк как есть
      а что, в ку-говне всё ещё нет годного ассоциативного контейнера?
      Ответить

      • ava bormand 09.05.2014 17:59 # 0

        QMap<QString, AttachmentService::MimeType>
        QHash<QString, AttachmentService::MimeType>

        Выбирай ;)
        Ответить

        • ava FlySnake 09.05.2014 23:59 # 0

          На практике подобные вещи ухудшают читабельность и увеличивают кол-во г-кода. Тупой "свитч" по строкам имхо нагляднее
          Ответить

          • ava Xom94ok 10.05.2014 10:40 # 0

            > Тупой "свитч" по строкам
            Но ведь C++ не может свитч не по интегральному типу...
            Ответить

            • ava FlySnake 10.05.2014 12:13 # 0

              А потому и "тупой" в кавычках - if else if,,, и есть тупой свитч :)
              Ответить

        • ava defecate-plusplus 10.05.2014 00:35 # +1

          > Выбирай
          ололо, где же свой функтор на сравнение?
          а в хеше - и функтор на хеш?

          автор не спроста же захотел insensitive
          Ответить

          • ava bormand 10.05.2014 06:23 # 0

            Ну ок, убедил.
            Ответить

          • ava Xom94ok 10.05.2014 10:39 # 0

            > ололо, где же свой функтор на сравнение?
            > а в хеше - и функтор на хеш?
            QHash использует перегрузку свободной функции qHash для расчета хеша. Как-то так:
            class QovnoString : public QString { /*...*/ };

uint qHash(const QovnoString & str, uint seed = 0) {
    return qHash(str.toLower(), seed);
}

QHash<QovnoString, AttachmentService::MimeType> map;
            Ответить

            • ava bormand 10.05.2014 10:42 # +1

              Но это один хер не поможет сделать case-insensitive коллекцию. Ну разве что замутить потомка от строки с переделанными операторами сравнения и qHash().
              Ответить

              • ava Xom94ok 10.05.2014 11:34 # 0

                Не поленился, качнул кутешечку. 
                #include <QCoreApplication>
#include <QString>
#include <QHash>
#include <QDebug>

class QovnoString : public QString {
public:
    QovnoString(const char * c) : QString(c) { }
    bool operator== ( const QovnoString & other ) const
    {
        return toLower() == other.toLower();
    }
};

uint qHash(const QovnoString & str, uint seed = 0) {
    return qHash(str.toLower(), seed);
}

QHash<QovnoString, int> map;

int main(int argc, char *argv[])
{
    QCoreApplication a(argc, argv);

    map["abcd"] = 100500;
    qDebug() << map["AbCd"];

    return a.exec();
}
                Мне это говно еще придется кушать некоторое время, так что ничего, кроме равнодушия, я к кутешечке не испытываю.
                Ответить

              • ava Elvenfighter 10.05.2014 12:10 # 0

                Всё гораздо проще :) В хеш вставлять ключи в нижнем регистре и вынимать тоже:
                QHash<QString, AttachmentService::MimeType> map;

AttachmentService::MimeType AttachmentService::mimeTypeByCodeKey(const QString &codeKey) const 
{
    return map[codeKey.toLower()];
}
                Ответить

                • ava bormand 10.05.2014 12:21 # +2

                  > В хеш вставлять ключи в нижнем регистре и вынимать тоже
                  Дык это нарушение инкапсуляции. Если так постоянно делать - где-нибудь в обяз забудешь сделать lower case и кровь-кишки.
                  Ответить

    • ava Psionic 09.05.2014 19:13 # 0

      nested тип не смутил AttachmentService::MimeType никого? Обычно такие штуки вызывают неприятие у общественности.
      Ответить

      • ava bormand 09.05.2014 19:23 # 0

        > вызывают неприятие у общественности...
        ... которая не видела буст etc.

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

        • ava Dummy00001 09.05.2014 23:17 # 0

          > у енумов протекают скопы.

          долго пытался понять про что ты тут талдычишь. а потом осознал что это то почему я энумы изредка в структуры заворачиваю. но только изредка. потому что "протекающие скопы" из нот э баг - иц э фича.
          Ответить

          • ava guest 09.05.2014 23:51 # +2

            Нене, уже же давно C++11 с enum class
            Ответить

          • ava bormand 10.05.2014 06:22 # 0

            В сишке оно может и было фичей, т.к. неймспейсов не было, и один хер везде лепили префиксы.
            Ответить

    • ava nslz 25.08.2021 08:59 # 0

      vanished
      Ответить

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