新・明解 C言語 入門編 演習7-4を解けなくて困っています。どなたかご教授よろしくお願いいたします。

Question

前提・実現したいこと

新・明解 C言語 入門編というテキストを使って勉強をしています。
演習7-4を解いていますが、なかなか上手く行きません。

演習7-4 符号無し整数xの第posビットを、1にした値を返す関数set、0にした値を返す関数reset、反転した値を返す関数inverseを作成せよ。
unsigned set(unsigned x, int pos) {/* … /}
unsigned set(unsigned x, int pos) {/ … /}
unsigned reset(unsigned x, int pos) {/ … */}

■■な機能を実装中に以下のエラーメッセージが発生しました。

発生している問題・エラーメッセージ

結果1
整数を入力して下さい : 10
何ビット目を変更しますか? : 1
元の数                : 00000000000000000000000000001010
1ビット目を1にした数  : 00000000000000000000000000001011
1ビット目を0にした数  : 00000000000000000000000000001000
1ビット目を反転した数 : 00000000000000000000000000001011

結果2
整数を入力して下さい : 20
何ビット目を変更しますか? : 1
元の数                : 00000000000000000000000000010100
1ビット目を1にした数  : 00000000000000000000000000010111
1ビット目を0にした数  : 00000000000000000000000000010100
1ビット目を反転した数 : 00000000000000000000000000010001

…

該当のソースコード

#include <stdio.h>

void printbits(unsigned x)
{
	int i;

	for (i = (8 * sizeof(unsigned)) - 1; i >= 0; i--)
	{
		if ((x >> i) & 1)			/* 2進数表示するために計算。 */
		{
			putchar('1');			/* 1と表示させる */
		}
		else
		{
			putchar('0');			/* 0と表示させる */
		}
	}
}

unsigned lrotate(unsigned x, int n)
{
	int bits = (8 * sizeof(unsigned));		/* 8ビット * sizeof(符号無し整数) */
							/* 今回のsizeof関数は型を返すため */
	return (n ? (x << n) | (x >> (bits - n)) : x);
}

unsigned set(unsigned x, int pos)
{
	return (x - 1 | lrotate(1U, pos));			/* xビット目を1に変更する */
}

unsigned reset(unsigned x, int pos)
{
	return (x & ~ lrotate(1U, pos));			/* xビット目を0に変更する */
}

unsigned inverse(unsigned x, int pos)
{
	return (x - 1 ^ lrotate(1U, pos));			/* xビット目を反転させる */
}

int main()
{
	unsigned x;
	int n;

	printf("整数を入力して下さい : ");	scanf("%u", &x);
	printf("何ビット目を変更しますか? : ");	scanf("%d", &n);

	printf("元の数                : ");	printbits(x);		putchar('\n');
	printf("%dビット目を1にした数  : ", n);	printbits(set(x, n));	putchar('\n');
	printf("%dビット目を0にした数  : ", n);	printbits(reset(x, n));	putchar('\n');
	printf("%dビット目を反転した数 : ", n);	printbits(inverse(x, n));	putchar('\n');

	return 0;
}

試したこと

setを設定するコードですが、return (x | lrotate(1U, pos));にするとどうしても「x+1」ビット目を変更してしまいます。
return (x - 1 | lrotate(1U, pos));にしましたが、reset等の出力値がおかしくなります。
ダメもとでreturn ((x - 1) | lrotate(1U, pos));も試してみましたが、ダメでした。

補足情報（FW/ツールのバージョンなど）

Comments

[dodox86]

> ダメもとでreturn ((x - 1) | lrotate(1U, pos));も試してみましたが、

こう、求める結果がでるまでコードをいじくり回すのではなく、コードの動きを頭の中で・あるいは頭の中の動きをコードに反映させて、動きを追いましょう。

> return (n ? (x << n) | (x >> (bits - n)) : x);

例えばこういったコードのひとつひとつを分解し、自分で説明でき、それぞれの結果は想定通りですかと問い直してもよいです。（質問文中に説明を書いて、と言っている訳ではないです）

Answer 1

以下のようにしてください。
set,reset,inverseを修正しました。
lrotateに誤りがあるのでlrotateは使用していません。

unsigned set(unsigned x, int pos)
{
    return (x | ( 1 << pos));                   /* posビット目を1に変更する */
}

unsigned reset(unsigned x, int pos)
{
    return (x & ~(1 << pos));                   /* posビット目を0に変更する */
}

unsigned inverse(unsigned x, int pos)
{
    return (x  ^ (1 << pos));                   /* posビット目を反転させる */
}

Comments

[Bulls]

すごくヒントになりました。
ありがとうございました。

Answer 2

＃define set( x, pos)  ((x)|(1<<(pos)))
＃define reset( x, pos)  ((x)&~(1<<(pos))) 

あえてコピペできないようにしてます。
なにしてるか理解しましょう