Замечания по программе Radix

[alexey-malov]

1>c:\teaching\ips\2018\oop\smirnov\cpp_oop_practice\lab_1\task_2\task_2.cpp(77): warning C4267: 'initializing': conversion from 'size_t' to 'int', possible loss of data
1>c:\teaching\ips\2018\oop\smirnov\cpp_oop_practice\lab_1\task_2\task_2.cpp(88): warning C4244: 'argument': conversion from 'double' to 'const unsigned int', possible loss of data

• [x] Для индексации контейнеров STL следует использовать size_t
• [x] Избавиться от pow

[alexey-malov]

unsigned int GetSum(const unsigned int a, const unsigned b)
{
    if (a >= numeric_limits<unsigned int>::max() - b)
    {
        throw overflow_error("Input number overflow");
        return 0;
    }

• [x] Инструкция, следующая за throw выполнена не будет

[alexey-malov]

// Будет ли переполнение при умножении беззнаковых чисел а и b
if (a == 0 || (std::numeric_limits<unsigned>::max() / a >= b))
{
  // a * b не вызовет переполнения
}
else
{
   // переполнение   
}

[alexey-malov]

• [x] При переводе нуля из любой системы счисления в любую программа не выводит ничего. Воспроизвести в тестах, исправить проблему

[alexey-malov]

• [x] При переводе FFFFFFFF из 16 в 10 программа сообщает о переполнении, хотя FFFFFFFE - не сообщает. Исправить проблему (должно переводиться нормально)

[Romasmi]

При входном числе FFFFFFFF (4294967295) программа выдаст ошибку, т.к. numeric_limits::max() = 4294967295 (Проверено выводов в output инструкцией cout << numeric_limits::max() << endl;)

[alexey-malov]

    if (str.length() <= 0)
    {
        wasError = true;
        return 0;
    }

• [x] Метод length() возвращает значение типа size_t, которое не может быть меньше 0. Кроме того, даже если бы он возвращал знаковый тип, то в нормально спроектированном классе строки отрицательная длина была бы невозможной

[alexey-malov]

unsigned int GetMul(const unsigned int a, const unsigned b)
{
    if (a == 0 || ((numeric_limits<unsigned int>::max() / b + 1) >  a))

• [x] Если b == 1, программа неверно отрепортит переполнение. +1 - это костыль.

• [x] Должен быть тест на преобразование максимального числа

[alexey-malov]

        unsigned int currentDischargeNumber = LetterToNumber(str[i]);

• [x] Разряд (числа) - digit. Discharge - электрический разряд

[alexey-malov]

    return InInterval(letter, '0', '9') ? static_cast<int>(letter - '0') : static_cast<int>(letter) - 55;

• [x] 55 - магическое число. Выразите его через символ 'A' и число 10

const char AMinus10 = 'A' - 10;

• [x] AMinus10 -> переименовать. Фактически - это основа для буквенных разрядов

[alexey-malov]

        unsigned int currentDischargeNumber = LetterToNumber(str[i]);
        if (currentDischargeNumber > radix)
        {
            wasError = true;
            return 0;
        }

• [x] Ошибка в сравнении

[alexey-malov]

• [x] Функции IsValidated, Convert выглядят тяжёлыми для восприятия

[alexey-malov]

• [x] Функции, выполняющие вычисления, не должны выводить в cout ничего. Раз уж используете исключения, то и код обработки ошибок разместитье в одном месте внутри main

[Romasmi]

Fixed by commit https://github.com/Romasmi/cpp_oop_practice/commit/95c9f55c466bac6616b498d881d2029726163fd9

[alexey-malov]

bool ValidStringNumber(string number)
{
    try
    {
        int tempNumber = stoi(number);
    }
    catch (const exception& e)
    {
        return false;
    }
}

• [x] Не во всех случаях функция возвращает значения.

[alexey-malov]

• [x] Программа должна компилироваться без предупреждений компилятора во всех конфигурациях и платформах.