Új hozzászólás Aktív témák

• #17811 martonx veterán pmonitor #17807

  Új Válasz 2022-10-13 12:45:12 #17811

  Új hozzászólás

  Összes hozzászólása itt Válaszok az összes hozzászólására itt Válaszok erre a hozzászólásra

  Privát üzenet küldése

  

  martonx

  veterán

  válasz pmonitor #17807 üzenetére

  Ez semmit nem fog optimalizálni, de jóval áttekinthetőbbé teszi a kódod:

  if (INVALID_HANDLE_VALUE == iSearchHandle)
  {
  }
  else
  {

  helyett
  if (INVALID_HANDLE_VALUE != iSearchHandle)

  Nem kell a felesleges else.

• #17808 dqdb nagyúr pmonitor #17807

  Új Válasz 2022-10-13 11:15:47 #17808

  Új hozzászólás

  Összes hozzászólása itt Válaszok az összes hozzászólására itt Válaszok erre a hozzászólásra

  Privát üzenet küldése

  

  dqdb

  nagyúr

  válasz pmonitor #17807 üzenetére

  void search(wchar_t* sPath, wchar_t* sFileMask, FAJLKERESESEREDMENYE* pFke, char almappae, char mappae, char fajle)

  Ha már állandóan az optimalizáláson pörögsz, akkor hol vannak a const módosítók azon paraméterek elől, amelyek értékét nem módosítja a kód, és emiatt a compiler optimalizáláskor figyelembe tudja venni?

  wchar_t* sPath_1 = (wchar_t*)GlobalAlloc(LMEM_FIXED, (wcslen(sPath) + wcslen(sFileMask) + 1) * sizeof(wchar_t));

  Először is:

  

  Másodszor az első paraméterben szereplő LMEM_FIXED a LocalAlloc híváshoz tartozó konstans, itt a GMEM_FIXED konstanst kellene használni.

  if (sPath_1 == INVALID_HANDLE_VALUE) MessageBox(0, L"memória", L"Üzenet", 0);

  A dokumentáció alapján sikertelen foglalás esetében a GlobalAlloc visszatérési értéke NULL, míg az INVALID_HANDLE_VALUE értéke -1, szóval rossz a hibakezelésnél mind a feltétel, mind a kezelése, hiszen NULL pointerrel továbbengeded a futást.

  És itt abbahagytam, mert feleslegesnek éreztem folytatni. Az egy pillantásra látszik, hogy tele van a kód buffer overflow aknával.

Új hozzászólás Aktív témák