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

    +26

    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
    for(devIndex=0;SetupDiEnumDeviceInfo(devs,devIndex,&devInfo);devIndex++) {

        if(doSearch) {
            for(argIndex=skip,match=FALSE;(argIndex<argc) && !match;argIndex++) {
                TCHAR devID[MAX_DEVICE_ID_LEN];
                LPTSTR *hwIds = NULL;
                LPTSTR *compatIds = NULL;
                //
                // determine instance ID
                //
                if(CM_Get_Device_ID_Ex(devInfo.DevInst,devID,MAX_DEVICE_ID_LEN,0,devInfoListDetail.RemoteMachineHandle)!=CR_SUCCESS) {
                    devID[0] = TEXT('\0');
                }
...
}

    Лучший стиль форматирования кода от индусов из MS.
    Код взят из этого файла:
    https://github.com/Microsoft/Windows-driver-samples/blob/master/setup/devcon/devcon.cpp
    В других файлах код примерно аналогичный.

    Запостил: demoth, 07 Сентября 2016

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

    • ava void_main 07.09.2016 16:03 # −19

      показать все, что скрытоЕсть такой прием ведения дискуссий в интернете: закончились аргументы - доебись до орфографии.
      У программистов есть аналогичный прием проведения кодревью: не к чему доебаться - доебись до форматирования.
      Ответить

    • ava void_main 07.09.2016 16:08 # −19

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

      • ava demoth 07.09.2016 16:30 # 0

        Не очень понимаю, какое отношение манера ведения дискуссии имеет к данному случаю.

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

        • ava roman-kashitsyn 07.09.2016 16:53 # 0

          > Например, я во всём коде не буду ставить отступы.

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

          • ava demoth 07.09.2016 17:26 # 0

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

            • ava roman-kashitsyn 07.09.2016 17:38 # +1

              Это я к тому, что форматирование — дело поправимое. Можно весь код за 5 минут одним скриптом отформатировать.
              Ответить

              • ava demoth 07.09.2016 17:45 # 0

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

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

      • ava demoth 07.09.2016 16:38 # 0

        P.S. Да, если форматирование исправить, то код нормальный. Возможно, обсуждать подходы к форматированию не принято на этом сайте. Если так, то учту это на будущее.
        Ответить

    • ava CTEPTOP 07.09.2016 17:36 # −22

      показать все, что скрытоМой расширенный, огнедышащий анус ждёт новой порции спермы.
      Ответить

    • ava BagorCtretora 08.09.2016 12:53 # −20

      показать все, что скрытоАгде тут плюсплюс
      Ответить

      • ava Soul_re@ver 08.09.2016 12:56 # 0

        > Код взят из этого файла:
        > https://github.com/Microsoft/Windows-driver-samples/blob/master/setup/devcon/devcon.cpp

        В расширении.
        Ответить

        • ava cpp_neq_c 08.09.2016 14:02 # 0

          Как раз по теме в подкасте http://cppcast.com/2016/06/andrew-pardoe/ PM-а С++ компилятора спросили если откроют код msvc, одно из мелких препятствии это как раз стыд за определенную часть написанного кода.
          Ответить

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