Программа перевода из 10 в 2 и 8 систему счисления(С++)?

Question

[Sazoks]

Всем привет. Мне 16, 4 месяца изучаю С++ и нарыл интересную задачку. Нужно написать программу перевода числ из 10 в 2 и 8 СС, подразумевается, что все числа положительные. Весьма тривиальная задача, как я понял. Вот что сделал я. Прошу более опытных программистов оценить код и сказать, что не так и как можно лучше. Полностью открыт к ОБЪЕКТИВНОЙ критике.
Заранее все спасибо и добра вам!

#include <iostream>
#define DEBUG
using namespace std;

void double_sys(int num)
{
	int count = 1;
	int copy_num = num;
	while (copy_num != 1)
	{
		count++;
		copy_num /= 2;
	}
	int *arr = new int[count];
	for (int i = (count - 1); i > 0; i--)
	{
		arr[i] = num % 2;
		num /= 2;
	}
	arr[0] = (int)(num);
	for (int i = 0; i < count; i++)
		cout << arr[i];
	cout << endl;
	delete[] arr;
}

void eight_sys(int num)
{
	int count = 1;
	int copy_num = num;
	while (copy_num > 7)
	{
		count++;
		copy_num /= 8;
	}
	int *arr = new int[count];
	for (int i = (count - 1); i > 0; i--)
	{
		arr[i] = num % 8;
		num /= 8;
	}
	arr[0] = (int)(num);
	for (int i = 0; i < count; i++)
		cout << arr[i];
	cout << endl;
	delete[] arr;
}

int main()
{
	setlocale(LC_ALL, "ru");

	int num;
	cout << "=====================================================\n";
	cout << "| Перевод числа из десятичной системы счисления(СС) |\n| в двоичную и восьмеричную                         |";
	cout << "\n=====================================================\n\n";

	cout << "Введите число: "; cin >> num;

	cout << "В 2-й: "; double_sys(num);
	cout << "В 8-й: "; eight_sys(num);
	system("pause");
	return 0;
}

Comments

[longclaps]

11 вопросов в анамнезе, первый задан 27 окт. 2018 - а вставлять код текстом так и не научился.
Весьма тривиальная задача, хехе.

[Sazoks]

longclaps, и зачем ты написал? Хех, так и знал, что найдется хотя бы один ОСОБЕННЫЙ, который не увидит слово "ОБЪЕКТИВНОЙ". Не понимаю таких людей.. Скорее всего, ты либо не доволен своей жизнью, либо доволен, но тебе она досталась не легко, и теперь ты возомнил себя покорителем мира сего и самым умным, остряк........

[longclaps]

Sazoks, твой код - говно. Я хотел привести его к божескому виду, но не тут-то было. Вот за этим и написал. Теперь понимаешь?
"ОСОБЕННЫЙ", "ОБЪЕКТИВНОЙ" - больше капслока, больше ада )
зы Вставишь код - исправлю.

[jcmvbkbc]

Sazoks, он правильно тебе написал: ты объективно нарушаешь пункт 3.8 правил.

[Sazoks]

longclaps, Все понял, извини, что быканул так..

[Sazoks]

longclaps, Скажи пжлст в чем именно код говно и как можно было проще эту задачу решить?

Answer 1

[longclaps]

подразумевается, что все числа положительные

используй типизацию, а не подразумевай

double_sys, eight_sys - почти 100% дублирование кода

int *arr = new int[count];

в этой задаче максимально необходимый размер строки можно прикинуть и использовать статическую переменную

#include <iostream>

using namespace std;

void to_any_number_system(unsigned num, unsigned radix) {
    // соблюдение условия 2 <= radix <= 16 на совести вызывающего
    const char digits[] = "0123456789abcdef";
    char buf[sizeof(num) * 8], *p = buf; // двоичное представление num влезет
    do *p++ = digits[num % radix]; while (num /= radix);
    do cout << *--p; while (p > buf);
    cout << endl;
}

int main() {
    unsigned n = 0xffffffff; // чтобы мало не казалось )
    for (unsigned r = 2; r <= 16; r++) {
        cout << (r > 9 ? "" : " ") << r << '\t';
        to_any_number_system(n, r);
    }
}

Comments

[Sazoks]

Спасибо большое!
Вопрос один, сколько вы программируете и что нужно сделать, чтобы так же эффективно решать задачи? Все, что я решаю, мне кажется убогими костылями..

[longclaps]

Sazoks, я-то давно начал, но всем этим трюкам ты выучишься быстро )

[Sazoks]

Блин, я сейчас разбирал ваш код, и по-моему это гениально!
Я бы до такого не додумался, по крайней мере на текущем этапе своего развития, как программиста. Спасибо за пример КАЧЕСТВЕННОГО кода))) Это реально круто