问题
Im having some trouble understanding how and why this code works the way it does. My partner in this assignment finished this part and I cant get ahold of him to find out how and why this works. I've tried a few different things to understand it, but any help would be much appreciated. This code is using 2's complement and a 32-bit representation.
/*
* fitsBits - return 1 if x can be represented as an
* n-bit, two's complement integer.
* 1 <= n <= 32
* Examples: fitsBits(5,3) = 0, fitsBits(-4,3) = 1
* Legal ops: ! ~ & ^ | + << >>
* Max ops: 15
* Rating: 2
*/
int fitsBits(int x, int n) {
int r, c;
c = 33 + ~n;
r = !(((x << c)>>c)^x);
return r;
}
回答1:
c = 33 + ~n;
This calculates how many high order bits are remaining after using n
low order bits.
((x << c)>>c
This fills the high order bits with the same value as the sign bit of x
.
!(blah ^ x)
This is equivalent to
blah == x
回答2:
On a 2's-complement platform
-n
is equivalent to~n + 1
. For this reason,c = 33 + ~n
on such platform is actually equivalent toc = 32 - n
. Thisc
is intended to represent how many higher-order bits remain in a 32-bitint
value ifn
lower bits are occupied.Note two pieces of platform dependence present in this code: 2's-complement platform, 32-bit
int
type.Then
((x << c) >> c
is intended to sign-fill thosec
higher order bits. Sign-fill means that those values ofx
that have0
in bit-positionn - 1
, these higher-order bits have to be zeroed-out. But for those values ofx
that have1
in bit-positionn - 1
, these higher-order bits have to be filled with1
s. This is important to make the code work properly for negative values ofx
.This introduces another two pieces of platform dependence:
<<
operator that behaves nicely when shifting negative values or when1
is shifted into the sign bit (formally it is undefined behavior) and>>
operator that performs sign-extension when shifting negative values (formally it is implementation-defined)The rest is, as answered above, just a comparison with the original value of
x
:!(a ^ b)
is equivalent toa == b
. If the above transformations did not destroy the original value ofx
thenx
does indeed fit inton
lower bits of 2's-complement representation.
回答3:
Using the bitwise complement (unary ~) operator on a signed integer has implementation-defined and undefined aspects. In other words, this code isn't portable, even when you consider only two's complement implementations.
It is important to note that even two's complement representations in C may have trap representations. 6.2.6.2p2 even states this quite clearly:
If the sign bit is one, the value shall be modified in one of the following ways:
-- the corresponding value with sign bit 0 is negated (sign and magnitude);
-- the sign bit has the value -(2 M ) (two's complement );
-- the sign bit has the value -(2 M - 1) (ones' complement ).
Which of these applies is implementation-defined, as is whether the value with sign bit 1 and all value bits zero (for the first two), or with sign bit and all value bits 1 (for ones' complement), is a trap representation or a normal value.
The emphasis is mine. Using trap representations is undefined behaviour.
There are actual implementations that reserve that value as a trap representation in the default mode. The notable one I tend to cite is Unisys Clearpath Dordado on OS2200 (go to 2-29). Do note the date on that document; such implementations aren't necessarily ancient (hence the reason I cite this one).
According to 6.2.6.2p4, shifting negative values left is undefined behaviour, too. I haven't done a whole lot of research into what behaviours are out there in reality, but I would reasonably expect that there might be implementations that sign-extend, as well as implementations that don't. This would also be one way of forming the trap representations mentioned above, which are undefined in nature and thus undesirable. Theoretically (or perhaps some time in the distant or not-so-distant future), you might also face signals "corresponding to a computational exception" (that's a C standard category similar to that which SIGSEGV
falls into, corresponding to things like "division by zero") or otherwise erratic and/or undesirable behaviours...
In conclusion, the only reason the code in the question works is by coincidence that the decisions your implementation made happen to align in the right way. If you use the implementation I've listed, you'll probably find that this code doesn't work as expected for some values.
Such heavy wizardry (as it has been described in comments) isn't really necessary, and doesn't really look that optimal to me. If you want something that doesn't rely upon magic (e.g. something portable) to solve this problem consider using this (actually, this code will work for at least 1 <= n <= 64
):
#include <stdint.h>
int fits_bits(intmax_t x, unsigned int n) {
uintmax_t min = 1ULL << (n - 1),
max = min - 1;
return (x < 0) * min + x <= max;
}
来源:https://stackoverflow.com/questions/14792521/bitwise-operations-and-shifts