Замечания по Лабораторной работе №1 "radix"

[AniSkyWorker]

• [ ] вместо#define MIN_NOTATION 2 следует использовать const unigned MIN_NOTATION = 2;
• [ ] вместо unsigned int следует писать просто - unsigned, если спецификаторы не указаны, компилятор автоматически будет отображать в int
• [ ] вас не смущают функции int CharToInt(char ch) и char DigitToChar(char ch), огромные switch-case может заменить пара строчек, исследуйте и замените, добавив пару условий

int digit = chr - '0';
int digit = chr - 'A' + 10;

char chr = num + '0';
char chr = num + 'A' - 10;

• [ ] можно объединить

        if (!found)
        {
            for (unsigned int j = 0; j < radix; ++j)
            {
                if (value[i] == DIGITS[j])
                {
                    found = true;
                }
            }
        }
        if (!found)
        {
            return false;
        }

• [ ] вместо const string & DIGITS = "0123456789ABCDEFGHIJKLMNOPQRSTUVWXYZ"; и прохода по строке используйте условия вида if (chr >= '0' && chr <= '9')
• [ ] зачем это делать в цикле если проверяется 1ый элемент, который не изменяется. вынесите наверх

bool found = false;
        if (i == 0)
        {
            if (value[0] == '-')
            {
                found = true;
            }
        }

• [ ] у вас излишняя функция ConvertFromDecTo, модернизируйте алгоритм функции ConvertFromTo, чтобы переводить обратно в 10ую систему счисления и используйте ее

[AniSkyWorker]

• [ ] в тестах проверьте и все возможные ошибочные состояния, у вас их предусмотрено достаточно, а вот их тестирования нет
• [ ] следует загружать файлы необходимые для инициализации проекта (vcxproj) в репозиторий, иначе возникает много неудобств при запуске ваших лаб

[AniSkyWorker]

• [ ] int numX10 = 0; нигде не используется
• [ ] для возведения в степень есть функция pow, используйте ее. выпилить функцию Power
• [ ] не нужно писать свою функцию умножения. не нужно проверять все числа, нужно предусмотреть отдельные способы обработки максимальных и минимальных чисел на платформе, например переводить их по разрядам

[AniSkyWorker]

k = 0.3

[alexey-malov]

• [ ] Если путь к проекту содержит пробелы, то тесты запускаться не будут

[alexey-malov]

• [ ] Программа должна воспринимать как строчные, так и заглавные буквы

[alexey-malov]

//Функция переводит строку в целочисленное число
int StrToInt(const string &str, bool &wasError)
{
    int result = 0;
    for (unsigned i = 0; i < str.length(); ++i)
    {
        int digit = CharToDigit(str[i]);
        if (result > (INT_MAX - digit) / 10)
        {
            wasError = true;
            return 0;
        }
        result = result * 10 + digit;
    }
    return result;
}

• [ ] Без лишней нужды велосипедостроением лучше не заниматься. Преобразовать строку в число можно и при помощи стандартной функции, например, stoi. Либо в своей функции реализуйте поддержку отрицательных чисел

[alexey-malov]

• [ ] Программа должна уметь переводить число INT_MIN (-2^31)

[alexey-malov]

//Ставит указанный массив задом-наперед
void ReverseString(string &str)
{
    char tmpCh = '0';
    for (unsigned i = 0; i < (unsigned)(str.length() / 2); ++i)
    {
        tmpCh = str[i];
        str[i] = str[str.length() - i - 1];
        str[str.length() - i - 1] = tmpCh;
    }
}

• [ ] Комментарий упоминает массив, хотя функция принимает строку.
• [ ] Используйте алгоритм std::reverse

[alexey-malov]

//Перевод числа в строку
string IntToStr(int number)
{
    string result;
    while (number != 0)
    {
        result += DigitToChar(number % 10);
        number /= 10;
    }
    ReverseString(result);
    return result;
}

Очередной велосипед, не поддерживающий отрицательные числа

[alexey-malov]

radix.exe 10 10 0

• [ ] выдает некорректный результат. Написать тест для проверки этой ситуации и исправить

[alexey-malov]

//Функция переводит строку из системы счисления, задаваемой параметром radix, в десятичную
int ConvertToDec(const string &str, unsigned radix, bool &isNegative, bool &wasError)

• [ ] Функция не переводит из строки в десятичную систему счисления
• [ ] Зачем параметр isNegative?

[alexey-malov]

• [ ] Программа должна также принимать унарный +

[alexey-malov]

• [ ] Результат работы программы - не только ее код возврата, но и вывод в output. Его надо в тестах также проверять.

[alexey-malov]

• [ ] Обойтись без возведения в степень

[alexey-malov]

k = 0,5