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

• kingabo őstag

  #2845

  #2839 chabeee

  2011-11-07 17:17:38

  Ú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

  

  kingabo

  őstag

  #2839 chabeee

  #2839 chabeee

  Sziasztok! Lenne egy ilyen függvényem:
  void masol (int** t1, int** t2, int** t3, const int n1, const int n2)
  {
  int i,j, counter = 0;
  
  for(i=0;i<n1;i++) {
  counter++;
  }
  for(i=0;i<n2;i++) {
  counter++;
  }
  
  *(t3) = (int*)malloc(sizeof(int)*counter);
  for(i=0;i<n1;i++) {
  *(*(t3)+i) = *(*(t1)+i);
  }
  
  for(i=n1, j=0; i<counter,j<n2; i++, j++) {
  *(*(t3)+i) = *(*(t2)+j);
  }
  
  for(i=0;i<counter;i++) {
  printf("3.%d: %d\n", i+1, *(*(t3)+i));
  }
  free(*(t3));
  }

  annyit csinál hogy a t1 és a t2 dinamikus tömb elemeit belemásolja egy t3 nevű dinamikus tömbbe, viszont warningel ennél a sornál:

  for(i=n1, j=0; i<counter,j<n2; i++, j++) {

  ezt írja: ../main.c:113:26: warning: left-hand operand of comma expression has no effect [-Wunused-value]

  ha van valakinek valami ötlete, hogy miért írja ezt, annak megköszönném a segítséget!

  üdv csabi

  Ezekkel mit akartál megvalósítani?
  for(i=0;i<n1;i++) {
  counter++;
  }
  for(i=0;i<n2;i++) {
  counter++;
  }

  Szerintem ez ugyanaz és nagyságrendekkel gyorsabb:

  counter = n1 + n2;

  A t3-at, ha csak a kiíratáshoz használod, akkor fölösleges a másolás. Ha ez egy visszatérő paraméter akarna lenni, akkor az fv végén lévő free(*(t3)); törli és ha ezután használni szerednéd (a törölt elemeket), csúnya dolgok történ(het)nek.

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