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

• #6222 kovisoft őstag pmonitor #6219

  Új Válasz 2021-04-16 16:46:03 #6222

  Ú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

  

  kovisoft

  őstag

  válasz pmonitor #6219 üzenetére

  Ami részletet idemásoltál, abból nem nyilvánvaló, de itt az okokat jobban meg lehet érteni, ha megnézed a forráskód historyját. Ebből látszik, hogy eredetileg csak a switch volt ott, a goto és az előtte lévő rész hiányzott. Aki hozzáadta a raw függvényparamétert, az gondolom nem akarta, hogy az egyébként változatlan switch kompletten mint módosult kód jelenkezzen a historyban. Illetve (ami szintén nem látszik az idemásolt részleten) mivel a switch már így is elég nagy, eléggé tele van if-ekkel és eléggé be van indentálva, ezért az is lehet, hogy nem akart még egyet növelni az indentálás mértékén. Valószínűleg másképp csinálta volna, ha nulláról most írja meg az egész függvényt.

  Szóval ez inkább a "történelmileg így alakult" kódra példa.

• #6221 sztanozs veterán pmonitor #6219

  Új Válasz 2021-04-16 16:31:12 #6221

  Ú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

  

  sztanozs

  veterán

  válasz pmonitor #6219 üzenetére

  Amúgy csinálhatsz nyugodtan egy pull request-et és goto-talaníthatod a GCC-t, ha nagyon cikinek érzed a goto használatát a c fordítóban, így 2021-ben.

• #6220 sztanozs veterán pmonitor #6219

  Új Válasz 2021-04-16 16:20:01 #6220

  Ú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

  

  sztanozs

  veterán

  válasz pmonitor #6219 üzenetére

  Ott mondjuk pont meg lehetne egyszerűen, bár szerintem ez a felírás egyértelműbb, mint ha a sswitch egy negált feltételben volna (egyébként szerintem amúgy is pont erre az alakra fogja fordítani a fordító). De "optimalizáld" ki lsz a 2921. sorban kezdődő switch-ből a goto-t, hogy ne nézzen ki nagyon csúnyán...
  Ezek (mind a 300 sorban), mind később a switchben szemantikailag pont jól néznek ki:
  ha (feltétel) ugorj a labelre
  És amúgy itt sem látsz olyat, hogy a goto "összevissza" ugrálna, csak előrefelé.

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