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

• Nyiscsák aktív tag

  #2557

  #2553 Sk8erPeter

  2011-05-11 19:30:54

  Új Válasz

  Ú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

  

  Nyiscsák

  aktív tag

  #2553 Sk8erPeter

  #2553 Sk8erPeter

  Ezt a
  system("PAUSE");
  baromságot nagyon gyorsan felejtsd el (egyszer össze kéne számolni, hányszor hangzott el ez a topikban).
  Gondolom valami gyökér tanár mutatta ezt a szarságot, vagy valami gány fórumon szedted össze.
  Ott van helyette pl. a getchar();

  Ezenfelül jobb lenne, ha azonnal visszatérnél, amint egyezést találtál egy korábbi számmal (minek végigmenni rajta teljesen?).
  if (pTomb[j]==pTomb[i])
  ez a feltételvizsgálat nálad legalább egyszer mindenképp igaz lesz, amikor önmagával hasonlítod az elemet, felesleges ezt vizsgálni.
  Szóval így is lehet:
  int tobbszor( int* pTomb, int size )
  {
  int i, j;
  
  for ( i = 0; i < size; i++ ){
  for ( j = 0; j < size; j++){
  if (i!=j && pTomb[j]==pTomb[i])
  return 1;
  }
  }
  return 0;
  }

  Sajnos a systemes hülyeséget a tanár verte belénk a DevC++ használatakor, de igyekszem felejteni. A feladat egyszerűsítéséért pedig köszönet! Így sokkal átláthatóbb az egész.

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