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

    +1004

    1. 1
    2. 2
    3. 3
    4. 4
    5. 5
    6. 6
    7. 7
    if (param["LOG"] != NULL) // пришли логи от плагина.
    {
        this->logs = new loger();
        this->logs->write(param["LOG"], param["TYPE"], this->Name, name);
        delete this->logs;
        return;
    }

    Так мы пишем логи

    Запостил: ir4y, 12 Января 2012

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

    • ava TarasB 12.01.2012 11:35 # +1

      Что будет с памятью при исключении в logs->write?
      Ещё один аргумент в пользу принуждения к оборачиванию работы с ресурсами в отдельные классы.
      Ответить

      • ava roman-kashitsyn 12.01.2012 11:58 # 0

        > Ещё один аргумент в пользу принуждения к оборачиванию работы с ресурсами в отдельные классы

        И в чём тут будет выигрыш?
        Ответить

        • ava TarasB 12.01.2012 12:02 # +1

          В безопасности кода
          Ответить

          • ava roman-kashitsyn 12.01.2012 12:07 # +3

            Мне кажется, в данном случае вообще выделение ресурсов нужно только один раз, а не на каждом акте общения с плагином. Лог всё равно кому-то нужно создавать... может быть, я не совсем понял идею.
            Ответить

    • ava defecate-plusplus 12.01.2012 11:57 # +2

      как часто приходят логи от плагина, что new/delete тут вообще оправдано?
      Ответить

      • ava ctm 12.01.2012 12:28 # 0

        а еще, что такое params?
        Ответить

        • ava defecate-plusplus 12.01.2012 12:31 # 0

          да понятно, что недешевый ассоциативный массив
          Ответить

          • ava eth0 12.01.2012 12:50 # −3

            Ассоциативный массив? В моём С++? Ох уж этот STL.

            P.S. Пока искал, что на эту тему думает яндекс, обнаружил отличный кусок копипасты.

            К счастью, некоторые технологии, реализованные в PHP можно перенести и в другие языки программирования: например, в C++.

Одна из таких технологий – ассоциативные массивы.

В ассоциативном массиве вместо числовых индексов используются ключи любых типов. Данные в ассоциативном массиве так же могут быть разнотипными. К примеру:

ass_arr array;

            Нуф сказано.
            Ответить

            • ava ir4y 12.01.2012 13:05 # 0

              Не STL код взят из приложения на Qt
              Ответить

            • ava roman-kashitsyn 12.01.2012 13:05 # +4

              > ass_arr
              Пиратский тип.
              Бутылка рома упала и укатилась вниз по палубе.
              - ASS! ARRRGHH!
              Ответить

            • ava defecate-plusplus 12.01.2012 13:10 # 0

              а в чем собсно проблема с ассоциативными массивами в с++?
              с некоторыми хитростями даже в оном можно сделать аналог
              my_arr["somestring"] = 400;
              my_arr[200] = "so what?";
              Ответить

              • ava eth0 12.01.2012 18:16 # −3

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

                • ava defecate-plusplus 12.01.2012 20:02 # 0

                  уу, я то уж думал что все претензии к с++ в том, что в std::map "просто так" тип ключа должен быть конкретным и известным на момент создания контейнера, в то время, как какие-то несомненно более крутые языки позволяют сделать нечто вроде a['b'] = 100; a["str"] = 'a'; a[200] = "str2"; сразу, без лишнего геморроя
                  Ответить

    • ava ir4y 12.01.2012 13:02 # 0

      Что самое забавное логи валятся постоянно, практически каждое действие в программе логируется
      Что бы усилить эффект приведу код класса loger

      loger::loger()
{
}

void loger::write(QString msg,QString logtype,QString login,QString modulename)
{
    QSqlQuery query;
    QString time=QString().sprintf("%02d:%02d",QTime::currentTime().hour(),QTime::currentTime().minute());
    QString date=QString().sprintf("%04d/%02d/%02d",QDate::currentDate().year(),
                                                    QDate::currentDate().month(),
                                                    QDate::currentDate().day());
    query.exec("INSERT INTO log VALUES(NULL,\""+date+" "+time+"\",\""+login+"\",\""+modulename+"\",\""+logtype+"\",\""+msg+"\")");
    qDebug() << date << " " << time << " "  << modulename << " " << logtype << " " <<  msg;
}


      по сути это статическая функция
      Ответить

      • ava guest 13.01.2012 11:04 # +3

        просто ООП-обезьны считают, что если поставить в программе слово class, то она станет мягкой и шелковистой и сегфолты внезапно исчезнут
        Ответить

        • ava Mindless 13.01.2012 17:32 # −1

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

        • ava ir4y 13.01.2012 17:35 # 0

          Жестко но справиедливо
          Ответить

    • ava Lure Of Chaos 12.01.2012 13:38 # +3

      loger, ага
      Ответить

    • ava Psionic 13.01.2012 18:29 # 0

      Хм весьма забавно, выделил памать под обьект чтобы вызвать отдуды функцию которой этот обьект впринципе и не нужен, ООП дрочер какой - то.
      Ответить

    • ava nobadya 03.03.2012 05:10 # −1

      правельно, путайте сотрудников говыряющихся в вашем, ну вы поняли...
      Ответить

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