Logo Questions Linux Laravel Mysql Ubuntu Git Menu
 

memcmp vs multiple equality comparisons

Tags:

Precondition: Consider such a class or struct T, that for two objects a and b of type T

memcmp(&a, &b, sizeof(T)) == 0 

yields the same result as

a.member1 == b.member1 && a.member2 == b.member2 && ... 

(memberN is a non-static member variable of T).

Question: When should memcmp be used to compare a and b for equality, and when should the chained ==s be used?


Here's a simple example:

struct vector {     int x, y; }; 

To overload operator == for vector, there are two possibilities (if they're guaranteed to give the same result):

bool operator==(vector lhs, vector rhs) { return lhs.x == rhs.x && lhs.y == rhs.y; } 

or

bool operator==(vector lhs, vector rhs) { return memcmp(&lhs, &rhs, sizeof(vector)) == 0; } 

Now if a new member were to be added to vector, for example a z component:

  • If ==s were used to implement operator==, it would have to be modified.
  • If memcmp was used instead, operator== wouldn't have to be modified at all.

But I think using chained ==s conveys a clearer meaning. Although for a large T with many members memcmp is more tempting. Additionally, is there a performance improvement from using memcmp over ==s? Anything else to consider?

like image 336
emlai Avatar asked Mar 04 '15 15:03

emlai


People also ask

What is memcmp used for?

Description. The C library function int memcmp(const void *str1, const void *str2, size_t n)) compares the first n bytes of memory area str1 and memory area str2.

Does memcmp compare byte by byte?

The memcmp() built-in function compares the first count bytes of buf1 and buf2. The relation is determined by the sign of the difference between the values of the leftmost first pair of bytes that differ.

Can you compare structures in C?

C provides no language facilities to do this - you have to do it yourself and compare each structure member by member.


1 Answers

Regarding the precondition of memcmp yielding the same result as member-wise comparisons with ==, while this precondition is often fulfilled in practice, it's somewhat brittle.

Changing compilers or compiler options can in theory break that precondition. Of more concern, code maintenance (and 80% of all programming work is maintenance, IIRC) can break it by adding or removing members, making the class polymorphic, adding custom == overloads, etc. And as mentioned in one of the comments, the precondition can hold for static variables while it doesn't hold for automatic variables, and then maintenance work that creates non-static objects can do Bad Things™.

And regarding the question of whether to use memcmp or member-wise == to implement an == operator for the class, first, this is a false dichotomy, for those are not the only options.

For example, it can be less work and more maintainable to use automatic generation of relational operator overloads, in terms of a compare function. The std::string::compare function is an example of such a function.

Secondly, the answer to what implementation to choose depends strongly on what you consider important, e.g.:

  • should one seek to maximize runtime efficiency, or

  • should one seek to create clearest code, or

  • should one seek the most terse, fastest to write code, or

  • should one seek to make the class most safe to use, or

  • something else, perhaps?

Generating relational operators.

You may have heard of CRTP, the Curiously Recurring Template Pattern. As I recall it was invented to deal with the requirement of generating relational operator overloads. I may possibly be conflating that with something else, though, but anyway:

template< class Derived > struct Relops_from_compare {     friend     auto operator!=( const Derived& a, const Derived& b )         -> bool     { return compare( a, b ) != 0; }      friend     auto operator<( const Derived& a, const Derived& b )         -> bool     { return compare( a, b ) < 0; }      friend     auto operator<=( const Derived& a, const Derived& b )         -> bool     { return compare( a, b ) <= 0; }      friend     auto operator==( const Derived& a, const Derived& b )         -> bool     { return compare( a, b ) == 0; }      friend     auto operator>=( const Derived& a, const Derived& b )         -> bool     { return compare( a, b ) >= 0; }      friend     auto operator>( const Derived& a, const Derived& b )         -> bool     { return compare( a, b ) > 0; } }; 

Given the above support, we can investigate the options available for your question.

Implementation A: comparison by subtraction.

This is a class providing a full set of relational operators without using either memcmp or ==:

struct Vector     : Relops_from_compare< Vector > {     int x, y, z;      // This implementation assumes no overflow occurs.     friend     auto compare( const Vector& a, const Vector& b )         -> int     {         if( const auto r = a.x - b.x ) { return r; }         if( const auto r = a.y - b.y ) { return r; }         return a.z - b.z;     }      Vector( const int _x, const int _y, const int _z )         : x( _x ), y( _y ), z( _z )     {} }; 

Implementation B: comparison via memcmp.

This is the same class implemented using memcmp; I think you'll agree that this code scales better and is simpler:

struct Vector     : Relops_from_compare< Vector > {     int x, y, z;      // This implementation requires that there is no padding.     // Also, it doesn't deal with negative numbers for < or >.     friend     auto compare( const Vector& a, const Vector& b )         -> int     {         static_assert( sizeof( Vector ) == 3*sizeof( x ), "!" );         return memcmp( &a, &b, sizeof( Vector ) );     }      Vector( const int _x, const int _y, const int _z )         : x( _x ), y( _y ), z( _z )     {} }; 

Implementation C: comparison member by member.

This is an implementation using member-wise comparisons. It doesn't impose any special requirements or assumptions. But it's more source code.

struct Vector     : Relops_from_compare< Vector > {     int x, y, z;      friend     auto compare( const Vector& a, const Vector& b )         -> int     {         if( a.x < b.x ) { return -1; }         if( a.x > b.x ) { return +1; }         if( a.y < b.y ) { return -1; }         if( a.y > b.y ) { return +1; }         if( a.z < b.z ) { return -1; }         if( a.z > b.z ) { return +1; }         return 0;     }      Vector( const int _x, const int _y, const int _z )         : x( _x ), y( _y ), z( _z )     {} }; 

Implementation D: compare in terms of relational operators.

This is an implementation sort of reversing the natural order of things, by implementing compare in terms of < and ==, which are provided directly and implemented in terms of std::tuple comparisons (using std::tie).

struct Vector {     int x, y, z;      friend     auto operator<( const Vector& a, const Vector& b )         -> bool     {         using std::tie;         return tie( a.x, a.y, a.z ) < tie( b.x, b.y, b.z );     }      friend     auto operator==( const Vector& a, const Vector& b )         -> bool     {         using std::tie;         return tie( a.x, a.y, a.z ) == tie( b.x, b.y, b.z );     }      friend     auto compare( const Vector& a, const Vector& b )         -> int     {         return (a < b? -1 : a == b? 0 : +1);     }      Vector( const int _x, const int _y, const int _z )         : x( _x ), y( _y ), z( _z )     {} }; 

As given, client code using e.g. > needs a using namespace std::rel_ops;.

Alternatives include adding all other operators to the above (much more code), or using a CRTP operator generation scheme that implements the other operators in terms of < and = (possibly inefficiently).

Implementation E: comparision by manual use of < and ==.

This implementation is the result not applying any abstraction, just banging away at the keyboard and writing directly what the machine should do:

struct Vector {     int x, y, z;      friend     auto operator<( const Vector& a, const Vector& b )         -> bool     {         return (             a.x < b.x ||             a.x == b.x && (                 a.y < b.y ||                 a.y == b.y && (                     a.z < b.z                     )                 )             );     }      friend     auto operator==( const Vector& a, const Vector& b )         -> bool     {         return             a.x == b.x &&             a.y == b.y &&             a.z == b.z;     }      friend     auto compare( const Vector& a, const Vector& b )         -> int     {         return (a < b? -1 : a == b? 0 : +1);     }      Vector( const int _x, const int _y, const int _z )         : x( _x ), y( _y ), z( _z )     {} }; 

What to choose.

Considering the list of possible aspects to value most, like safety, clarity, efficiency, shortness, evaluate each approach above.

Then choose the one that to you is clearly best, or one of the approaches that seem about equally best.

Guidance: For safety you would not want to choose approach A, subtraction, since it relies on an assumption about the values. Note that also option B, memcmp, is unsafe as an implementation for the general case, but can do well for just == and !=. For efficiency you should better MEASURE, with relevant compiler options and environment, and remember Donald Knuth's adage: “premature optimization is the root of all evil” (i.e. spending time on that may be counter-productive).

like image 85
Cheers and hth. - Alf Avatar answered Oct 02 '22 06:10

Cheers and hth. - Alf