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

    +20

    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
    19. 19
    20. 20
    21. 21
    22. 22
    23. 23
    24. 24
    25. 25
    Ini ini(m_serviceProvider);
        if( ini.load( gameInputStream ) == false )
        {
            LOGGER_ERROR(m_serviceProvider)("StartupConfigLoader::loadGame_ Invalid load game settings '%s'"
                , _gameIniPath.c_str()
                );

            return false;
        }

    IniUtil::getIniValue( ini, "Project", "Name", _settings.projectName, m_serviceProvider );
    IniUtil::getIniValue( ini, "Project", "Company", _settings.companyName, m_serviceProvider );
    IniUtil::getIniValue( ini, "Project", "Codename", _settings.applicationSettings.projectCodename, m_serviceProvider );
    IniUtil::getIniValue( ini, "Project", "Version", _settings.applicationSettings.projectVersion, m_serviceProvider );
    IniUtil::getIniValue( ini, "Locale", "Default", _settings.defaultLocale, m_serviceProvider );
    IniUtil::getIniValue( ini, "Game", "ContentResolution", _settings.applicationSettings.contentResolution, m_serviceProvider );     
    IniUtil::getIniValue( ini, "Game", "FixedContentResolution", _settings.applicationSettings.fixedContentResolution, m_serviceProvider );
    IniUtil::getIniValue( ini, "Game", "PersonalityModule", _settings.personalityModule, m_serviceProvider );
    IniUtil::getIniValue( ini, "Game", "AlreadyRunning", _settings.alreadyRunning, m_serviceProvider );
    IniUtil::getIniValue( ini, "Game", "WindowModeCheck", _settings.applicationSettings.windowModeCheck, m_serviceProvider );
    IniUtil::getIniValue( ini, "Window", "Size", _settings.applicationSettings.windowResolution, m_serviceProvider );
    IniUtil::getIniValue( ini, "Window", "Bits", _settings.applicationSettings.bits, m_serviceProvider );
    IniUtil::getIniValue( ini, "Window", "Fullscreen", _settings.applicationSettings.fullscreen, m_serviceProvider );
        IniUtil::getIniValue( ini, "Window", "MaxFPS", _settings.maxfps, m_serviceProvider );
    IniUtil::getIniValue( ini, "Window", "VSync", _settings.applicationSettings.vsync, m_serviceProvider );

    Это оттуда уже другой. Они все сговорились что-ли? Ынтерпрайз головного мозга.
    PS: boost::property_map не осилил.

    Запостил: LispGovno, 31 Января 2014

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

    • ava LispGovno 31.01.2014 23:21 # +2

      serviceProvider головного моска
      Ответить

    • ava bormand 31.01.2014 23:44 # +2

      > IniUtil::getIniValue(ini, ...)
      Отличное использование ООП. Да еще и возвращает результат через анус (видимо ради type inference).
      Ответить

      • ava wvxvw 01.02.2014 01:20 # +3

        А в новом Ц++ добавять zipWith, и код чудесно превратится в:
        HaskellToCPP::zipWith(::std::bind(IniUtil::getIniValue, ini), ::std::vector<::std::string>(

        И тут писатель задумается над списками инициализации, и поймет, что нужно ждать следующего обновления стандарта.

        ЗЫ. А помните как в конце девяностых было круто слать анимированые сообщения в статус-баре браузера, типа:
        .:::you are the 21'st visitor:::.
        Ответить

    • ava laMer007 01.02.2014 00:02 # −3

      Ну может просто старый код просто или написано новичком, а никто не проконтролировал. Теперь все используют. Переписывать то лень.
      Ответить

    • ava guest 01.02.2014 06:14 # +1

      с каких пор boost::property_map является new-free решением?
      Ынтерпрайз головного мозга - это когда такие хомячки как ты, постят всякую срань ни разу не задумываюсь почему было написано так. Удачи тебе в твоем колесе )
      Ответить

      • ava bormand 01.02.2014 07:15 # +1

        А зачем нужно new-free решение где-то кроме горячих циклов? А в частности - зачем оно нужно при чтении конфига, которое всяко идет один раз за жизнь проги.
        Ответить

        • ava guest 01.02.2014 13:53 # +1

          есть такая система как marmalade, и разные мобильные решения, на которых new-free решения очень полезно. Даже для ini фаилов, например можно также там найти xml sax parser который тоже new-free это было сделано по фитбеку клиента, раньше был какойто популярный xml parser и он читал xml локализации ~1 минуту(такая специфика мобилок) - new-free 0ms. На PC что то, что то давало примерно 0ms. Такие вот пироги с картошкой.
          Ответить

          • ava bormand 01.02.2014 14:01 # 0

            А из-за new ли так тормозил старый парсер? :)

            P.S. Что-за мобилки такие тормозные и умеющие в с++, явно же не андроид? Старенький симбиан? И какой размер конфига?
            Ответить

            • ava guest 01.02.2014 14:48 # +1

              именно из за new, std::map + std::string беспощадны к new.
              там все плохо с new + free,
              ты не поверишь это бы iPad1
              Ответить

              • ava bormand 01.02.2014 15:05 # +2

                > там все плохо с new + free
                Ну я погуглил немного сейчас. Куча народу жалуется на тормоза мармеладного аллокатора. Он походу тупо загибается от сильной фрагментации, которую ему устраивают стринг и мап.

                Так что я верю ;)
                Ответить

                • ava anonimb84a2f6fd141 02.02.2014 03:51 # −1

                  Кстати, а жавочный / донтетовский гц умеет дефрагментировать память? И когда он это делает?
                  Ответить

                • ava Soul_re@ver 02.02.2014 10:09 # +1

                  Кстати, все СТЛ контейнеры и, вроде как, бустовские умеют в кастомные аллокаторы.
                  Если отожрать себе пул памяти при запуске и реализовать в нём свою суперэффективную систему выделения памяти для дерева конфигов, это решит проблему?
                  Ответить

                  • ava crastinus 02.02.2014 10:36 # 0

                    В property_tree похоже нет такой возможности. Кстати, аллокатор, передаваемый в шаблон, не видел нигде кроме container.
                    Ответить

                  • ava bormand 02.02.2014 10:38 # 0

                    Если сможешь реализовать - то да ;)

                    Но это же, имхо, на порядок сложнее, чем sax'ом распарсить.
                    Ответить

                  • ava LispGovno 02.02.2014 10:52 # +3

                    Считай что практически не возможно. Во много раз надежнее и проще написать свой дом или сакс
                    Ответить

                    • ava bormand 02.02.2014 11:06 # +6

                      > написать свой дом
                      Воспитать сына и посадить дерево.

                      Кстати, а ведь в старых добрых аллокаторах либц и мсвц нет такой адской просадки от фрагментации. У них поди несколько фрилистов под разные размеры, чтобы меньше сканить при поиске дырок?
                      Ответить

                      • ava LispGovno 02.02.2014 11:16 # +7

                        Воспитать дерево, посадить сына.
                        Ответить

                      • ava LispGovno 02.02.2014 11:19 # 0

                        Да наверняка в оймухе тоже используется это. Кеширование цпу только у них говно. Любой проход по листу приводит к печали вселенской.
                        Ответить

                        • ava bormand 02.02.2014 12:06 # +2

                          > оймухе
                          Дык у мармеладки судя по докам свой велосипедный аллокатор. Отдельный пул указанного в конфиге размера и все такое. Так что походу тормозить под подобной нагрузкой он будет где угодно, и от железа мало что зависит.
                          Ответить

                          • ava LispGovno 02.02.2014 12:12 # 0

                            То есть это горе от ума глупости?
                            Ответить

                            • ava bormand 02.02.2014 12:14 # +1

                              Да не, есть вероятность, что на меньших нагрузках и фрагментации он работает пошустрее libc'шного. Ради этого его поди и запилили.
                              Ответить

                        • ava bormand 02.02.2014 12:16 # 0

                          Насчет ARM'а и нормального аллокатора вот что пишут:
                          On a crappy ARM board with a single core glibc can do 300 000 malloc/free pairs per second.
                          Ответить

                    • ava WGH 02.02.2014 13:32 # 0

                      >сакс
                      Кстати, какого хрена люди юзают SAX (push)? Ведь pull-парсер не менее эффективен, а код выглядит не как говно, а как нормальный рекурсивный спуск.
                      Ответить

            • ava guest 01.02.2014 14:49 # +1

              размер довольно большой 6к строк xml
              Ответить

      • ava LispGovno 01.02.2014 08:41 # +1

        У тебя такой длинный конфиг? Он каждый кадр считывается и сохраняется на диск? Нет? Тогда это нафиг не сдалось.
        Ответить

        • ava LispGovno 01.02.2014 09:00 # +1

          А если каждый кадр, то у тебя явно другие проблемы. С головой и с временем работы диска. На фоне его времени работы ты даже и не заметил бы нью.
          Ответить

          • ava WGH 01.02.2014 16:22 # +1

            Диск-то закешируется системой.
            Ответить

    • ava roman-kashitsyn 01.02.2014 07:31 # +2

      > property_map
      Ты хотел сказать property_tree?
      Ответить

      • ava LispGovno 01.02.2014 08:38 # +1

        Я их путаю. Проперти мап не позволяет создавать под ноды, а трии этого недостатка лишен?
        Ответить

        • ava roman-kashitsyn 01.02.2014 12:38 # +3

          tree - это комбайн-конвертер для популярных форматов конфигов. map - кусок ускорь.граф, написаннай для абстрагирования способа доступа к свойствам вершин, ребёр, всего графа.
          Ответить

    • ava Horse3 01.02.2014 12:06 # −7

      показать все, что скрытоКакой багор )))
      Ответить

    • ava LispGovno 03.02.2014 10:37 # 0

      Как можно одновременно использовать сервислокатеры и не признавать ооп?
      Ответить

      • ava kegdan 03.02.2014 12:53 # +2

        Лицемерие моралфагов хорошо видно у противников абортов (про-лайф). Если у такого противника (вернее, противницы) случается незапланированная беременность, они таки делают аборт, несмотря на свои убеждения.

        lurk
        http://lurkmore.to/_/55784#mws_7RODybD
        Ответить

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