Замечания по bin2dec

[alexey-malov]

• [x] Программа не поддерживает число из 32 единичек (2^32-1). Доработать тест, который бы проверял корректность преобразования данного числа. Число взять из интернет или вручную посчитать на калькуляторе

[alexey-malov]

• [x] Выделить функцию Bin2Dec для преобразования строки в число
• [x] эта функция не должна выводить ничего в output. Об ошибке должна сообщать через код возврата или аргумент, переданный по ссылке, или использовать механизм исключений

[alexey-malov]

• [x] oVerflow, а не owerflow

[alexey-malov]

        if (inStr[i] == '1')
        {
            if (owerflow)
            {
                cout << "Error! Result owerflow!" << endl;
                return 1;
            }
            result += multiplier;
        }
        else
        if (inStr[i] != '0')
        {
            cout << "Invalid argument type!" << endl
                << "Usage: bin2dec.exe <binary number>" << endl;
            return 1;
        }

• [x] else if лучше писать в одной строчке

[alexey-malov]

    for (int i = inStrSize - 1; i >= 0; --i)

1>c:\teaching\2016\ips\lihachev\oop2016\bin2dec\main.cpp(22): warning C4267: 'initializing': conversion from 'size_t' to 'int', possible loss of data

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

[alexey-malov]

{
    auto inStrSize = binString.size();
    if (inStrSize > MAX_SIZE_LENGTH)
    {
        errorMessage = errorMessage.append("Invalid arguments length!\nMax numbers length must be ")
            .append(to_string(MAX_SIZE_LENGTH)).append(", but found ").append(to_string(inStrSize));
        return ERROR_STATE;
    }

• [ ] код плохо читается

[alexey-malov]

int ERROR_STATE = -1;
int MAX_SIZE_LENGTH = 32;

int Bin2dec(string const & binString, unsigned long int & result, string & errorMessage)
{
    auto inStrSize = binString.size();
    if (inStrSize > MAX_SIZE_LENGTH)

• [ ] используйте константы из файла climits для определения количества битов

sizeof(type) * CHAR_BIT

[alexey-malov]

• [ ] Используйте bool для сообщения об успешности конвертирования