Побитовое кодирование

[JeyKip]

Помогите, пожалуйста, решить задачу на С++:
"Декодировать последовательность битов по следующему правилу:
00 - #
01 - @
10 - $
11 - &

Каждые 2 бита заменять соответствующим символом. Получаемые символы последовательности выводиить на печать."

Ввод числа в 10СС и вывод его в 2СС я сделал:

#include <iostream>
using namespace std;
void main ()
{
	setlocale(0,"");
	int a,i;
	cout<<"Введите число\n";
	cin>>a;
for (i=0; i<8; i++)
{
	if (a&0x80) cout<<"1";
	else cout<<"0";
	a<<=1;
	
}

А дальше не знаю...КТО ОТЗОВЕТСЯ, ЗАРАНЕЕ ОГРОМНОЕ СПАСИБО!!!

[Гром]

Во-первых, здесь предполагается, что размер int равен одному биту, что вообще говоря не гарантировано. Я бы предложил использовать

while (a)
{
if (a & 1) cout << "1";
else cout << "0";
a >>= 1;
}

И, например,

string s;
while (a)
{
b = (a & 3);
a >>= 1;
switch (b)
{
case 0: s += "#"; break;
//...
}
}

а затем вывести строку в обратном порядке (честно говоря, давно не имел дело с битовыми операторами, мог что-то напутать, но суть вы, надеюсь, уловили!)
Да, и, конечно, в первом примере опять же у меня получилось, что число пишется задом наперед, ну да мне лень исправлять...

[rangerx]

Сообщение от JeyKip

Помогите, пожалуйста, решить задачу на С++:
"Декодировать последовательность битов по следующему правилу:
00 - #
01 - @
10 - $
11 - &

Каждые 2 бита заменять соответствующим символом. Получаемые символы последовательности выводиить на печать."

#include <iostream>

int main()
{
    const int SIZE = 4;
    const char symbols[SIZE] = {'#', '@', '$', '&'};

    unsigned int val = 0;
    std::cout << "Input value: ", std::cin >> val;

    //32 бита взято в надежде, что так оно и есть )))
    unsigned int index = 0;
    for (int i = 31;  i > 0 ; i-=2)
    {
        if( (val & (1 << i)) ) index |= (1 << 1);
        if( (val & (1 << i-1)) ) index |= (1 << 0);

        std::cout << symbols[index] << ' ';
        index = 0;
    }
    return 0;
}

[Somebody]

if( (val & (1 << i)) ) index |= (1 << 1);
if( (val & (1 << i-1)) ) index |= (1 << 0);

index = val >> i & 3;

//32 бита взято в надежде, что так оно и есть )))

Не есть хорошо.

[JeyKip]

Всем, кто принял участие в решении этой задачи, еще раз огромное спасибо...Ниже представлен код этой программы в рабочем варианте..!! Может быть, кому-то пригодится:

#include <iostream>
using namespace std;
void main ()
{
	setlocale(0,"");
	int b,a,c=128,d=64;
	cout<<"Введите число\n";
	cin>>a;
	cout<<"Представление числа в 2-СС:\n";
	for (b=128; b>0; b/=2)
	{
		if (a&b) cout<<"1";
		else cout<<"0";
	}
cout<<"\n";
cout<<"Символьное кодирование:\n";
for (b=0; b<4; b++,a<<=2)

	{
				if (((a&c)!=128)&&((a&d)!=64))
			cout<<"#";
		else
	{
		if (((a&c)!=128)&&(a&d))
			cout<<"@";
		else
{
		if ((a&c)&&((a&d)!=64))
			cout<<"$";
		else

	{
		if ((a&c)&&(a&d))
			cout<<"&";
	}
		
}
		}}
for (b=0; b<2; b++)
		cin.get();
	}

[Гром]

М, небольшие замечания:
Объявляйте константы соответствующим образом:
const int c = 128;
const int d = 64;
И желательно отдельно, а то пока я нашел, где же они определены...

Еще - поскольку (a & c) == 128 либо (a & c) == 0, и аналогично с (a & d), то проще и нагляднее будет

int e = (a & c) + (a & d);
switch(e)
{
case 0: cout << "#"; break;
case 64: cout << "@"; break;
case 128: cout << "$"; break;
case 192: cout << "&"; break;
}

[rangerx]

index = val >> i & 3;

Да, так по идее должно работать(не проверял). Просто лень было думать, написал, что первое пришло в голову )

Не есть хорошо.

Никто и не говорит, что это хорошо. Исправить несложно.