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

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

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

          А надо?
          Если код не сложный и не требует особых размышлений, я обычно фигачу его в одну строчку, а потом делаю clang-format.
          Пусть роботы пробелы расставляют.
          Ответить
          • Не-не, погоди. Писать код можно как кому больше нравится, но clang-format же ты в итоге запускаешь.
            Ответить
            • Это я к тому, что форматирование — дело поправимое. Можно весь код за 5 минут одним скриптом отформатировать.
              Ответить
              • Да кто ж спорит. Конкретно этот код я читаю прямо с гитхаба, в процессе изучения виндовых функций для работы с девайсами. Т.е. сопровождать мне его, к счастью, не надо.

                Клонить себе репозиторий и форматировать из-за того, что кому-то не нравятся пробелы, откровенно лень. Вот и приходится страдать.
                Ответить
      • P.S. Да, если форматирование исправить, то код нормальный. Возможно, обсуждать подходы к форматированию не принято на этом сайте. Если так, то учту это на будущее.
        Ответить
    • показать все, что скрытоМой расширенный, огнедышащий анус ждёт новой порции спермы.
      Ответить
    • показать все, что скрытоАгде тут плюсплюс
      Ответить
      • > Код взят из этого файла:
        > https://github.com/Microsoft/Windows-driver-samples/blob/master/setup/devcon/devcon.cpp

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

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