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

• #4202 skylaner senior tag stranger28 #4193

  Új Válasz 2013-09-25 09:32:24 #4202

  Ú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

  

  skylaner

  senior tag

  válasz stranger28 #4193 üzenetére

  Szia!

  Én annyit tennék hozzá, hogy mivel 0-tól 100-ig fogsz számokat tárolni így felesleges memória pazarlás int-et használni.
  unsigned char *palyak;

  Illetve a többi változó is lehetne char/unsigned char.

• #4197 buherton őstag stranger28 #4193

  Új Válasz 2013-09-24 23:34:32 #4197

  Ú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

  

  buherton

  őstag

  válasz stranger28 #4193 üzenetére

  Kiegészítés (#4195) dabadab hsz-re.

  1. feleslegesek a typecast-ok és a jövőben kerüld ezeket, mert könnyen kerülhetsz olyan helyzetbe, amikor jön a fej vakarás, mert nem fut a kód, mint például:

  void foo(long *bar)
  {
  *bar = 23424234;
  }
  
  int main(void)
  {
  char bar;
  foo(&bar);
  return 0;
  }

  2. formázás: ez az adott sor végére rakni a { nem C-szerű formázás, hanem tán a Java-ból jön, és ajánlott ott használni. Továbbá próbáld kialakítani a saját stílusod, és tartsd magad ahhoz, mert könnyebb neked és másnak is olvasni a kódod.

  int* foo; helyett int *foo; mert ebből is lehetnek gondok, mint int* foo, bar; és két pointer helyett lesz egy pointer és egy változó, és nincs olyan típus, hogy int pointer, hanem int-re mutató pointer van csak.

  3. óvatosan az ilyenekkel:
  for(i=0;i<n;i++)
  if (palyak[i]==0)
  {
  nemsi_index[a]=i;
  a++;
  }

  Ugyanis nem feltétlenül portable a kód. Sajnos nem mindegyik fogadja el így ezt a formát. Az ilyen formával már kevésbé lehet gond:

  for(...)
  {
  if(...)
  ...;
  
  ...;
  }

• #4195 dabadab titán stranger28 #4193

  Új Válasz 2013-09-24 23:14:15 #4195

  Ú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

  

  dabadab

  titán

  válasz stranger28 #4193 üzenetére

  Par dolog:

  0. Az egesz cuccban egy darab komment sincs, pedig azt erdemes gyakorolni, mert kulon tudomany.

  1. "magic value"-ket a kodba irni nem tul jo gyakorlat, jobb konstanskent kezelni oket, illetve C-ben makrokent, szoval "#define MIN100_NOT_FOUND -1"

  2. A free()-ket erdemes pont forditott sorrendben meghivni, mint a malloc()-ot, egyreszt, mert ugy maradnak meg szepen a blokkok, masreszt meg lehet, hogy egy kesobb malloc()-olt strukturaban egy pointer egy elozoleg lefoglalt memoriateruletre mutat - es ha az elso free()-zed, a masodikrol meg azt gondolod, hogy az meg ervenyes adatokat tartalmaz, abbol baj lesz.

  3. A valtozoneveket (konstansokat, makrokat, file-okat, kommenteket) angolul tessek irni.

  4. A nemsi_index teljesen felesleges. Lehetne mondani, hogy azert van igy, hogy meglegyen a funkcionalis tagolas (beolvasas, feldolgozas, kiiras), de ez egyebkent sincs meg (amire teljesen elfogadhato magyarazat, hogy igy hatekonyabb a kod - leszamitva ezt a franya nemsi_indexet).

  5. Ha megis a tagolas mellett dontenel, akkor erdemes lenne azt harom fuggvenybe kirakni.

• #4194 greenity senior tag stranger28 #4193

  Új Válasz 2013-09-24 21:16:23 #4194

  Ú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

  

  greenity

  senior tag

  válasz stranger28 #4193 üzenetére

  Nekem tetszik, nálam elég úri lenne

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