Говнокод #27437 — C++ — Говнокод.ру

C++ / Говнокод #27437

+1
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
26. 26
27. 27
28. 28
29. 29
30. 30
31. 31
32. 32
33. 33
34. 34
35. 35
36. 36
37. 37
38. 38
39. 39
40. 40
```
_Return_type_success_(return != false) bool CEnumerateSerial::UsingCreateFile(_Inout_ CPortsArray& ports)
{
  //Reset the output parameter
  ports.clear();

  //Up to 255 COM ports are supported so we iterate through all of them seeing
  //if we can open them or if we fail to open them, get an access denied or general error error.
  //Both of these cases indicate that there is a COM port at that number. 
  for (UINT i=1; i<256; i++)
  {
    //Form the Raw device name
    ATL::CAtlString sPort;
    sPort.Format(_T("\\\\.\\COM%u"), i);

    //Try to open the port
    bool bSuccess = false;
    ATL::CHandle port(CreateFile(sPort, GENERIC_READ | GENERIC_WRITE, 0, nullptr, OPEN_EXISTING, 0, nullptr));
    if (port == INVALID_HANDLE_VALUE)
    {
      const DWORD dwError = GetLastError();

      //Check to see if the error was because some other app had the port open or a general failure
      if ((dwError == ERROR_ACCESS_DENIED) || (dwError == ERROR_GEN_FAILURE) || (dwError == ERROR_SHARING_VIOLATION) || (dwError == ERROR_SEM_TIMEOUT))
        bSuccess = true;
    }
    else
    {
      //The port was opened successfully
      bSuccess = true;
    }

    //Add the port number to the array which will be returned
    if (bSuccess)
#pragma warning(suppress: 26489)
      ports.push_back(i);
  }

  //Return the success indicator
  return true;
}
```
некоторые джавамэны вот таким гордятся

> Internally the code provides 10 different ways (yes you read that right: Ten)

еще и выложено под ни с чем не совместимой лицензией

Запостил: gologub, 23 Мая 2021

Tweet
Комментарии (32) RSS
- gologub 23.05.2021 01:17 # 0
  
  ну что, спойлернуть над каким странным утверджением надо задуматься?
  Ответить
  - bormand 23.05.2021 01:21 # 0
    
    Предупреждения вырубили и забыли на место вернуть?
    Ответить
    - gologub 23.05.2021 01:34 # 0
      
      нет, там всё гораздо проще, щас если гость не угадает - я вскрою и это приведёт к лавине и извержению одновременно
      Ответить
  - MAKAKA 23.05.2021 01:44 # 0
    
    >//Check to see if the error was because some other app had the port open or a general failure
    выглядит как говно
    Ответить
    - bormand 23.05.2021 01:45 # 0
      
      Да ладно, этот код хотя бы дискетой не скрипит во время перечисления...
      Ответить
      
      MAKAKA 23.05.2021 01:48 # +1
      
      Если из ``ERROR_GEN_FAILURE`` следует ``bSuccess = true``, то я такой код ебал
      Извините
      Ответить
      
      Steve_Brown 24.05.2021 12:17 # +2
      
      Ну, может, жизнь такая. Мало ли почему не открылся, но порт есть, пользователю надо показать - пусть сам решает. Может, у кого-то именно это значение вернуло.
      Ответить
- PolinaAksenova 23.05.2021 01:38 # 0
  
  > Up to 255 COM ports are supported
  Вообще-то up to 256.
  Ответить
  - gologub 23.05.2021 01:47 # +3
    
    вот это!
    но... если копнуть глубже окажется, что искаропки поддерживается up to 1024 и расширяемо до 4096, а если забить на "com name arbiter" то ∞
    
    а несчастный примат этот код 23 года вылизывал, а он оказался полной хуйней
    Ответить
    - MAKAKA 23.05.2021 01:48 # 0
      
      приведи пример машины с 1024 ком портами
      
      Это какая-то станция для дайлап пула в крупном провайдере из 1999-го года?
      Ответить
      
      gologub 23.05.2021 02:16 # +1
      
      блютус их плодит в большом количестве, например
      Ответить
      
      bormand 23.05.2021 02:19 # 0
      
      Да и мобилки через USB ACM раньше тоже плодились почему-то, у меня что-то в духе COM23 было под конец.
      Ответить
      
      guest6 23.05.2021 03:10 # +1
      
      Рискну предположить, что ваши USB устройства каждый раз регистрируют новый порт
      Старый можно удалить, если сказать show hidden devices
      
      Кто настолько старый, что помнит переменную
      
      DEVMGR_SHOW_NONPRESENT_DEVICES
      Ответить
    - bormand 23.05.2021 01:49 # 0
      
      А есть нормальный способ, кстати?
      
      Ну там через какое-нибудь setupapi запросить по классу "последовательный порт" или что-то подобное...
      Ответить
      
      PolinaAksenova 23.05.2021 01:51 # +2
      
      Распарсить вывод mode.com. Ня забудь про кодировку кодовую страницу! ☆*:.｡.o(≧▽≦)o.｡.:*☆
      Или "change port /QUERY" распарсить, там хотя бы кириллических букв нят.
      Ответить
      
      MAKAKA 23.05.2021 01:57 # 0
      
      >change port /QUERY
      ебать откуда ты это знаеш
      Ответить
      
      guest6 23.05.2021 02:02 # +1
      
      Установил
      c:\> change logon /disable
      на знакомом сервере
      Ответить
      
      gologub 23.05.2021 01:57 # +1
      
      если порт открыт, то mode его не покажет :-P
      Ответить
      
      gologub 23.05.2021 01:53 # +2
      
      это даже документировано
      
      https://docs.microsoft.com/en-us/windows-hardware/drivers/serports/external-naming-of-com-ports
      Ответить
      
      MAKAKA 23.05.2021 01:55 # +1
      
      https://govnokod.ru/27435#comment629231
      Ответить
- MAKAKA 23.05.2021 03:02 # 0
  PS C:\> Get-WmiObject Win32_SerialPort | select DeviceID
  Ответить
  - bormand 23.05.2021 03:27 # +1
    
    Переведи на "Си".
    Ответить
    - MAKAKA 23.05.2021 05:03 # +3
      
      #define _WIN32_DCOM #include <iostream> #include <Windows.h> #include <wbemidl.h> #pragma comment(lib, "wbemuuid.lib") int main() { if (FAILED(CoInitializeEx(nullptr, COINIT_MULTITHREADED))) { std::cerr << "Failed to init com" << std::endl; return -1; } CoInitializeSecurity( nullptr, -1, nullptr, nullptr, RPC_C_AUTHN_LEVEL_DEFAULT, RPC_C_IMP_LEVEL_IMPERSONATE, nullptr, EOAC_NONE, nullptr); IWbemLocator* pLoc = nullptr; HRESULT hr = CoCreateInstance(CLSID_WbemLocator, 0, CLSCTX_INPROC_SERVER, IID_IWbemLocator, reinterpret_cast<LPVOID*>(&pLoc)); if (FAILED(hr)) { std::cerr << "Failed to create com" << std::endl; return -1; } IWbemServices* pSvc = nullptr; hr = pLoc->ConnectServer(L"\\\\.\\root\\cimv2", NULL, NULL, NULL, WBEM_FLAG_CONNECT_USE_MAX_WAIT, NULL, NULL, &pSvc); if (FAILED(hr)) { std::cerr << "Failed to connect" << std::endl; return -1; } IEnumWbemClassObject* pEnum = nullptr; hr = pSvc->CreateInstanceEnum(L"Win32_SerialPort", 0, nullptr, &pEnum); if (FAILED(hr)) { std::wcerr << L"Failed get query" << hr << std::endl; return -1; } IWbemClassObject* pObj; ULONG returned; while (1) { pEnum->Next(WBEM_INFINITE, 1, &pObj, &returned); if (returned) { VARIANT v; pObj->Get(L"Name", 0, &v, nullptr, nullptr); std::wcout << v.bstrVal; } else { break; } } }
      
      собирать под 14-й vc. Под 17-й нужно явно отключать strictString (с ним wbem не собирается, ссаный BSTR виноват) и с unicode строками
      
      Нужно сделать обработку ошибок, и завернуть наверное комы в ATL, но я пьяный
      Ответить
      
      bormand 24.05.2021 13:02 # +3
      
      > но я пьяный
      
      Оно и видно... кто по трезвости полезет ворошить WMI голыми руками...
      Ответить
- MAKAKA 23.05.2021 03:41 # 0
  #include <iostream> #include <Windows.h> #include <sstream> int main() { char buffer[MAX_PATH]{}; for (uint16_t i = 1; i <= 1024; i++) { std::stringstream name; name << "COM" << i; const std::string devName{name.str()}; if (QueryDosDeviceA(devName.c_str(), buffer, sizeof(buffer)) == 0) { const auto lastError = GetLastError(); if (lastError == ERROR_FILE_NOT_FOUND) { break; } std::cerr << "Error: " << lastError << std::endl; return lastError; } std::cout << devName.c_str() << std::endl; } return 0; } // не работает старше 4
  Ответить
  - bormand 23.05.2021 03:48 # 0
    
    > break
    
    Заебись, с USB COM девайсами это в принципе не работает, я думаю? У них не меньше 5 и дырки в нумерации, емнип.
    Ответить
    - guest6 23.05.2021 05:05 # 0
      
      с дырками не рабоатет точно) нужно все проверять
      Ответить
- MAKAKA 23.05.2021 03:49 # 0
  Get-Item HKLM:\HARDWARE\DEVICEMAP\SERIALCOMM\ | Format-Table
  Ответить
  - PolinaAksenova 23.05.2021 03:56 # 0
    
    О, а это можня уже через Ви-няпи нярмальным способом распарсить. Правда, это всё равно будет противняя куча RegOpenKey RegQueryValue RegNyaKawaii...
    Ответить
    - guest6 23.05.2021 05:06 # 0
      
      тащемто всё что угодно можно сдеоать через win32pi |(кроме WinRT), просто много буков получится. Что с реестром, что с WMI, очень много буков
      Ответить
      
      gologub 24.05.2021 00:32 # 0
      
      WMI это же говно со скриптосахаром
      сервис может быть отключен
      или хуже, настроен неторопливо стартовать, когда ты туда полезешь
      или даже еще хуже, не успеть обновить список чтобы туда включить девайс, который ты только что воткнул
      Ответить
- avamana 04.10.2021 21:05 # 0
  
  Зеленые комментарии кода в IDe выглядят как сарказм...
  Ответить
Добавить комментарий
Ошибка компиляции комментария:

Гости могут высказаться только в понедельник, среду, четверг или воскресение
Семь раз отмерь — один отрежь, guest!

А не использовать ли нам bbcode?

[b]жирный[/b] — жирный

[i]курсив[/i] — курсив

[u]подчеркнутый[/u] — подчеркнутый

[s]перечеркнутый[/s] — перечеркнутый

[blink]мигающий[/blink] — мигающий

[color=red]цвет[/color] — цвет (подробнее)

[size=20]размер[/size] — размер (подробнее)

[code=<language>]some code[/code] (подробнее)

Проверочный код: *

Говнокод: по колено в коде.

C++ / Говнокод #27437

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

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