Logo Questions Linux Laravel Mysql Ubuntu Git Menu
 

left-hand operand of comma has no effect?

I'm having some trouble with this warning message, it is implemented within a template container class

int k = 0, l = 0;
    for ( k =(index+1), l=0; k < sizeC, l < (sizeC-index); k++,l++){
        elements[k] = arryCpy[l];
    }
    delete[] arryCpy;

this is the warning i get

cont.h: In member function `void Container<T>::insert(T, int)':
cont.h:99: warning: left-hand operand of comma has no effect
cont.h: In member function `void Container<T>::insert(T, int) [with T = double]':
a5testing.cpp:21:   instantiated from here
cont.h:99: warning: left-hand operand of comma has no effect
cont.h: In member function `void Container<T>::insert(T, int) [with T = std::string]':
a5testing.cpp:28:   instantiated from here
cont.h:99: warning: left-hand operand of comma has no effect
>Exit code: 0
like image 307
silent Avatar asked May 15 '10 09:05

silent


1 Answers

The comma expression a,b,c,d,e is similar to

{
  a;
  b;
  c;
  d;
  return e;
}

therefore, k<sizeC, l<(sizeC - index) will only return l < (sizeC - index).

To combine conditionals, use && or ||.

k < sizeC && l < (sizeC-index)  // both must satisfy
k < sizeC || l < (sizeC-index)  // either one is fine.
like image 140
kennytm Avatar answered Oct 12 '22 08:10

kennytm