Базовый тип чисел

@BTR · Регистрация: 01.11.2010

Author24 — интернет-сервис помощи студентам

Всем привет, вот в вузе дали задачку, написать полноценный базовый числовой тип, то есть как тип string, только с числами, что бы я мог туда писать и дробные и целые числа и выводились они таким образом

C++

int main() {
setlocale(LC_ALL, "Russian");
 
    num n = 17;
    num m = 5.5;
    cout << n << endl << m << endl;
 
system("pause >> void");
return 0;
}

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

num.h

C++

#ifndef NUM_H
#define NUM_H
 
class num {
    int i;
    double d;
    char dummy;
 
public:
    num();
    num(int _i);
    num(double _d);
    num(const num &);
 
    num & operator += (const num &);
    num & operator -= (const num &);
    num & operator *= (const num &);
    num & operator /= (const num &);
    num   operator %  (const num &);
    num & operator = (const num &);
    friend std::ostream & operator << (std::ostream &, const num &);
    friend num operator + (const num &, const num &);
    friend num operator - (const num &, const num &);
    friend num operator * (const num &, const num &);
    friend num operator / (const num &, const num &);
    friend bool operator >(const num &, const num &);
    friend bool operator <(const num &, const num &);
    friend bool operator <=(const num &, const num &);
    friend bool operator >=(const num &, const num &);
 
};
 
template<class T> T s(const T &n, int m);
 
#endif

И Вот он файл с определением класса, надеюсь у вас хватит терпения дочитать его
num.cpp

C++

#include <iostream>
#include "num.h"
 
template<class T>
T s(const T &n, int m) {
    T nn = n;
    for(int i = 1; i < m; ++i) {
        nn *= n;
    }
    return nn;
}
 
num::num()
    : i(0), d(0)
{
    dummy = '0';
}
 
num::num(int _i)
    : i(_i), d(0)
{
    dummy = 'i';
}
 
num::num(double _d)
    : i(0), d(_d)
{
    dummy = 'd';
}
 
num::num(const num &copy)
{
    i = copy.i;
    d = copy.d;
 
    if(i != 0)
        dummy = 'i';
    else
        dummy = 'd';
}
 
 
num & num::operator += (const num &n)
{
    if(n.dummy == 'i' && dummy == 'i')
        i += n.i;
    if(n.dummy == 'd' && dummy == 'd')
        d += n.d;
    if(n.dummy == 'i' && dummy == 'd')
        d += static_cast<double>(n.i);
    if(n.dummy == 'd' && dummy == 'i')
    {
        d = static_cast<double>(i); i = 0;
        d += n.d;
        dummy = 'd';
    }
 
    return *this;
}
 
num & num::operator -= (const num &n)
{
    if(n.dummy == 'i' && dummy == 'i')
        i -= n.i;
    if(n.dummy == 'd' && dummy == 'd')
        d -= n.d;
    if(n.dummy == 'i' && dummy == 'd')
        d -= static_cast<double>(n.i);
    if(n.dummy == 'd' && dummy == 'i')
    {
        d = static_cast<double>(i); i = 0;
        d -= n.d;
        dummy = 'd';
    }
 
    return *this;
}
 
num & num::operator = (const num &n)
{
    if(n.dummy == 'i')
    {
        i = n.i;
        dummy = 'i';
    }
 
    if(n.dummy == 'd')
    {
        d = n.d;
        dummy = 'd';
    }
 
    return *this;
}
 
num & num::operator *= (const num &n)
{
    if(n.dummy == 'i' && dummy == 'i')
        i *= n.i;
    if(n.dummy == 'd' && dummy == 'd')
        d *= n.d;
    if(n.dummy == 'i' && dummy == 'd')
        d *= static_cast<double>(n.i);
    if(n.dummy == 'd' && dummy == 'i')
    {
        d = static_cast<double>(i); i = 0;
        d *= n.d;
        dummy = 'd';
    }
 
    return *this;
}
 
num & num::operator /= (const num &n)
{
    if(n.dummy == 'i' && dummy == 'i')
    {
        // Прикидываемся дробным числом
        d = static_cast<double>(i); i = 0;
        dummy = 'd';
    }
 
    if(n.dummy == 'd' && dummy == 'd')
        d /= n.d;
 
    if(n.dummy == 'i' && dummy == 'd')
        d /= static_cast<double>(n.i);
 
    if(n.dummy == 'd' && dummy == 'i')
    {
        d = static_cast<double>(i); i = 0;
        d /= n.d;
        dummy = 'd';
    }
 
    return *this;
}
 
num num::operator % (const num &n)
{
    if(n.dummy == 'i' && dummy == 'i')
        return static_cast<num>(i % n.i);
 
    if(n.dummy == 'd' && dummy == 'd')
    {
        num yourself = n;
        num self = d;
        yourself.i = static_cast<int>(yourself.d); yourself.d = 0;
        self.i = static_cast<int>(self.d); self.d = 0;
        return static_cast<num>(self.i % yourself.i);
    }
 
    if(n.dummy == 'i' && dummy == 'd')
    {
        num self = d;
        self.i = static_cast<int>(self.d); self.d = 0;
        return static_cast<num>(self.i % n.i);
    }
 
    if(n.dummy == 'd' && dummy == 'i')
    {
        num yourself = n;
        yourself.i = static_cast<int>(yourself.d); yourself.d = 0;
        return static_cast<num>(i % yourself.i);
    }
}
 
num operator * (const num &n, const num &m)
{
    num nc = n;
    num mc = m;
    return nc *= mc;
}
 
num operator / (const num &n, const num &m)
{
    num nc = n;
    num mc = m;
    return nc /= mc;
}
 
 
num operator + (const num &n, const num &m)
{
    num nc = n;
    num mc = m;
    return nc += mc;
}
 
num operator - (const num &n, const num &m)
{
    num nc = n;
    num mc = m;
    return nc -= mc;
}
 
std::ostream & operator << (std::ostream &os, const num &n)
{
    if(n.dummy == 'i')
        os << n.i;
    else if(n.dummy == 'd')
        os << n.d;
    else
        os << 0;
 
    return os;
}
 
bool operator > (const num &n, const num &m)
{
    if(n.dummy == 'i' && m.dummy == 'i')
        return n.i > m.i;
    if(n.dummy == 'd' && m.dummy == 'd')
        return n.d > m.d;
    if(n.dummy == 'd' && m.dummy == 'i')
        return n.d > static_cast<double>(m.i);
    if(n.dummy == 'i' && m.dummy == 'd')
        return static_cast<double>(n.i) > m.d;
}
 
bool operator < (const num &n, const num &m)
{
    return !(n > m);
}
 
bool operator >= (const num &n, const num &m)
{
    if(n.dummy == 'i' && m.dummy == 'i')
        return n.i >= m.i;
    if(n.dummy == 'd' && m.dummy == 'd')
        return n.d >= m.d;
    if(n.dummy == 'd' && m.dummy == 'i')
        return n.d >= static_cast<double>(m.i);
    if(n.dummy == 'i' && m.dummy == 'd')
        return static_cast<double>(n.i) >= m.d;
}
 
bool operator <= (const num &n, const num &m)
{
    if(n.dummy == 'i' && m.dummy == 'i')
        return n.i <= m.i;
    if(n.dummy == 'd' && m.dummy == 'd')
        return n.d <= m.d;
    if(n.dummy == 'd' && m.dummy == 'i')
        return n.d <= static_cast<double>(m.i);
    if(n.dummy == 'i' && m.dummy == 'd')
        return static_cast<double>(n.i) <= m.d;
}

Добавлено через 2 минуты
поле dummy я создал для того, что бы заносить в него либо символ i, что будет говорить что мы целое число, либо d, что говорит, что мы дробное число, это нужно для вывода в сout, что бы он определил, какое поле возвращать, d или i

Добавлено через 38 секунд
сейчас допишу еще оператор сравнения, еще просьба будет, выскажите свое мнение, может ли быть полезен данный класс?

@BTR · 12.03.2012, 16:16 **[ТС]**

а вообще польза от класса есть какая-нибудь, помимо опыта, который я получу?

Добавлено через 1 минуту
Я ничего серьезного пока на С++ не писал, просто не знаю, используется в реальных проектах что-то подобное? В php знаю, что универсальное средство приветствуется

@fasked · 12.03.2012, 16:17

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

а вообще польза от класса есть какая-нибудь, помимо опыта, который я получу?

Пользы от самого класса вряд ли будет. По сути в большинстве задач он бесполезен, а в оставшейся части дублирует функционал больших библиотек, как boost или Qt (а уж лучше использовать их, чем писать самому подобное). Обычно такие классы называются "variant".

@gooseim · 12.03.2012, 16:24

Надо в конструкторе инициализировать один из двух. Конструктор по-умолчанию не нужен. Не надо ничего здесь сравнивать, достаточно будет switch по type.

Я не говорил, что нужен конструктор по умолчанию.
Хорошо, напишите тоже самое с помощью switch и посмотрим чей код будет короче и красивее. То, что вы предлагаете это детский сад.

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

вообще польза от класса есть какая-нибудь, помимо опыта, который я получу

Я писал однажды подобный класс, который мне пригодился. Но он по сути был просто оберткой над boost::any.

@ForEveR · 12.03.2012, 16:58

а вообще польза от класса есть какая-нибудь, помимо опыта, который я получу?

Тоже дело было. Писал обертку над any для сериализации.

@fasked · 12.03.2012, 17:05

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

Хорошо, напишите тоже самое с помощью switch и посмотрим чей код будет короче и красивее. То, что вы предлагаете это детский сад.

C++

#include <stdexcept>
#include <iostream>
 
using namespace std;
 
class NumericBase
{
private:
   union
   {
      int integer;
      double real;
   } value;
 
   enum
   {
      INTEGER,
      REAL
   } type;
 
public:
   NumericBase(int n) : type(INTEGER)
   {
      value.integer = n;
   }
   
   NumericBase(double n) : type(REAL)
   {
      value.real = n;
   }
   
   NumericBase(const NumericBase &n) : type(n.type)
   {
      value = value;
   }
   
   NumericBase &operator=(const NumericBase &n)
   {
      if (this != &n)
      {
         type = n.type;          
         value = n.value;
      }
            
      return *this;
   }
   
   NumericBase &operator=(const int n)
   {
      type = INTEGER;
      value.integer = n;
      
      return *this;
   }
   
   NumericBase &operator=(const double n)
   {
      type = REAL;
      value.real = n;
      
      return *this;
   }
   
   NumericBase &operator+=(const NumericBase &n)
   {
      if (type != n.type)
         throw std::logic_error("Different types");
      
      switch (type)
      {
      case INTEGER:
         value.integer += n.value.integer;
         break;
      case REAL:
         value.real += n.value.real;
         break;
      }
      
      return *this;
   }
   
   friend ostream &operator<<(ostream &out, const NumericBase &n)
   {
      out << (n.type == INTEGER ? n.value.integer : n.value.real);
      return out;
   }
};
 
int main()
{
   NumericBase a = 10, b = 15;
   a += b;
   
   cout << a << " " << b << endl;
   
   NumericBase c = 1.5, d = 3.5;
   c += d;
   
   cout << c << " " << d << endl;
}

@gooseim · 12.03.2012, 17:38

fasked, это конечно хорошо, но у автора по задумке было конвертирования из одного типа в другой, если они не совпадают (operator+=).

@fasked · 12.03.2012, 17:56

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

это конечно хорошо, но у автора по задумке было конвертирования из одного типа в другой, если они не совпадают (operator+=)

Не беда.

C++

private:
   union value_type
   {
      int integer;
      double real;
   } value;
 
public:
   NumericBase &operator+=(const NumericBase &n)
   {
      value_type v = convert_if_need(n);
 
      switch (type)
      {
      case INTEGER:
         value.integer += v.integer;
         break;
      case REAL:
         value.real += v.real;
         break;
      }
      
      return *this;
   }
 
private:
   value_type convert_if_need(const NumericBase &n)
   {
      value_type v = n.value;
      
      if (type != n.type)
      {
         v.real = (n.type == INTEGER ? n.value.integer : n.value.real);
         type = REAL;
      }
      
      return v;
   }

C++

int main()
{
   NumericBase a = 10, b = 15;
   a += b;
   
   cout << a << " " << b << endl;
   
   NumericBase c = 1.5, d = 3.5;
   c += a;
   d += b;
   
   cout << c << " " << d << endl;
}

Код

25 15
26.5 18.5

@gooseim · 12.03.2012, 18:17

fasked, теперь проверьте такой код:

C++

NumericBase a = 10, b = 15.0;
   a += b;
   
   cout << a << " " << b << endl;

@fasked · 13.03.2012, 01:08

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

теперь проверьте такой код:

C++

   value_type convert_if_need(const NumericBase &n)
   {
      value_type v = n.value;
      
      if (type != n.type)
      {
         v.real = (n.type == INTEGER ? n.value.integer : n.value.real);     
         
         if (type != REAL)
            value.real = value.integer;
 
         type = REAL;
      }
      
      return v;
   }

Я не проверял не прошлый код, ни это исправление, так что возможно и оно работать не будет. Но по логике должно. Я ставил своей целью передать идею о том, что битовые операции лишние в данном случае, а не написать полностью работоспособный код. Хотя это и звучит как отмазка

Я не отрицаю косяк. Просто считаю, что битовая арифметика не нужна здесь. И на мой взгляд, этот код ничем не хуже Вашего.

@gooseim · 13.03.2012, 01:12

fasked, я просто хотел сказать то, что если бы вы исправили все ошибки, то ваш код бы стал слишком длинным и сложным по сравнению с моим. Поэтому я считаю обратное. Тут можно было бы обойтись без битовых операций, код не намного бы стал длиннее (хотя с ними все равно красивее на мой взгляд), но switch, на котором вы настаивали - не вариант.

@fasked · 13.03.2012, 10:31

Не по теме:

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

switch, на котором вы настаивали - не вариант.

Изначально я, кстати, не имел в виду именно switch как ключевое слово языка. А просто, как переключатель :D

@BTR 8 / 8 / 1 Регистрация: 01.11.2010 Сообщений: 157
	12.03.2012, 16:16 [ТС]	21
	а вообще польза от класса есть какая-нибудь, помимо опыта, который я получу? Добавлено через 1 минуту Я ничего серьезного пока на С++ не писал, просто не знаю, используется в реальных проектах что-то подобное? В php знаю, что универсальное средство приветствуется 0

@fasked 5043 / 2622 / 241 Регистрация: 07.10.2009 Сообщений: 4,310 Записей в блоге: 1
	12.03.2012, 16:17	22
	Сообщение от BTR а вообще польза от класса есть какая-нибудь, помимо опыта, который я получу? Пользы от самого класса вряд ли будет. По сути в большинстве задач он бесполезен, а в оставшейся части дублирует функционал больших библиотек, как boost или Qt (а уж лучше использовать их, чем писать самому подобное). Обычно такие классы называются "variant". 0

@gooseim 516 / 421 / 92 Регистрация: 23.09.2010 Сообщений: 1,165
	12.03.2012, 16:24	23
	Надо в конструкторе инициализировать один из двух. Конструктор по-умолчанию не нужен. Не надо ничего здесь сравнивать, достаточно будет switch по type. Я не говорил, что нужен конструктор по умолчанию. Хорошо, напишите тоже самое с помощью switch и посмотрим чей код будет короче и красивее. То, что вы предлагаете это детский сад. Сообщение от BTR вообще польза от класса есть какая-нибудь, помимо опыта, который я получу Я писал однажды подобный класс, который мне пригодился. Но он по сути был просто оберткой над boost::any. 0

@gooseim 516 / 421 / 92 Регистрация: 23.09.2010 Сообщений: 1,165
	13.03.2012, 01:12	30
	fasked, я просто хотел сказать то, что если бы вы исправили все ошибки, то ваш код бы стал слишком длинным и сложным по сравнению с моим. Поэтому я считаю обратное. Тут можно было бы обойтись без битовых операций, код не намного бы стал длиннее (хотя с ними все равно красивее на мой взгляд), но switch, на котором вы настаивали - не вариант. 0

@ForEveR В астрале 8049 / 4806 / 655 Регистрация: 24.06.2010 Сообщений: 10,562
	12.03.2012, 16:58	24
	а вообще польза от класса есть какая-нибудь, помимо опыта, который я получу? Тоже дело было. Писал обертку над any для сериализации. 0

@gooseim 516 / 421 / 92 Регистрация: 23.09.2010 Сообщений: 1,165
	12.03.2012, 17:38	26
	fasked, это конечно хорошо, но у автора по задумке было конвертирования из одного типа в другой, если они не совпадают (operator+=). 0

@fasked
	13.03.2012, 10:31 Базовый тип чисел #31
	Не по теме: Сообщение от gooseim switch, на котором вы настаивали - не вариант. Изначально я, кстати, не имел в виду именно switch как ключевое слово языка. А просто, как переключатель :D 0