Реализация интерфейса[РЕШЕНО]

[Даниил]

Да, конечно. Вот, в первозданном виде.

[m_ax]

Да, конечно. Вот, в первозданном виде.

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

[Igors]

А с чего вы собственно взяли, что архитектура кривая?

Ну вообще-то кривая Это называется "мнимая общность". Неск классов наследуются от Number, и это можно как-то оправдать, мол, "все эти классы - числа". Но если посмотреть а что же общего у этих классов - то выясняется что аж ничего. Нет ни одного метода который общий и имеет смысл напр как для PNumber так и для ComplexNumber. "Все они имеют оператор +", ну так в Assistant есть десятки (или больше) классов с таким оператором, но на этом основании они не наследуются от одного базового.

Но не стоит это воспринимать так трагически как делает m_ax  Это просто учебный пример, вполне возможен случай когда классы действительно имеют достаточно общего, виртуалов и сделать еще и операторы виртуальными может быть вполне разумно.

Да, конечно. Вот, в первозданном виде.

Ой    Так у Вас "течет" память. Если оператор вернул класс по значению, то он автоматычно удалится когда станет ненужным. Но если оператор вернет ссылку - удалять ее никто не будет, и память (которую Вы выделили через new) не будет освобождена.

[m_ax]

Но не стоит это воспринимать так трагически как делает m_ax   Это просто учебный пример, вполне возможен случай когда классы действительно имеют достаточно общего, виртуалов и сделать еще и операторы виртуальными может быть вполне разумно.

Нет, я не драматизирую)
Кривость архитектуры здесь не в том, что они наследуются от "мнимой общности".. Фиг с ней, пусть наследуются - это как раз и даст им возможность полиморфизма.
Дело то в другом. Они пытаются забивать гвозди утюгом, в уверенности, что утюг для этого и создан.
Уже сам факт того, как они объявляют операторы+ - и др:

Код

C++ (Qt)
class Number
{
public:
 // ...
    virtual Number& operator +(Number &n) = 0;
    virtual Number& operator -(Number &n) = 0;
    virtual Number& operator /(Number &n) = 0;
    virtual Number& operator *(Number &n) = 0;
//...   
};
 

говорит о том, что у автора в голове полное не понимание основ перегрузки операторов.. И авторитетность таких людей, которые ещё преподают c++, у меня под большим-большим сомнением.
В этом и проблема)

[Igors]

говорит о том, что у автора в голове полное не понимание основ перегрузки операторов.. И авторитетность таких людей, которые ещё преподают c++, у меня под большим-большим сомнением.
В этом и проблема)

Как легко критиковать других Но такая критика только сваливает тему во флуд - и ничего больше

[m_ax]

говорит о том, что у автора в голове полное не понимание основ перегрузки операторов.. И авторитетность таких людей, которые ещё преподают c++, у меня под большим-большим сомнением.
В этом и проблема)

Как легко критиковать других Но такая критика только сваливает тему во флуд - и ничего больше

Нееет, мухи отдельно - котлеты отдельно)
Одно дело, когда такие архитектурные решения, в процессе обучения вояет сам обучающийся (что вполне нормально: все мы учимся на ошибках), а другое, когда такие решения навязывают, выдавая за правильные, преподы.
Так вот потихоньку и разваливается наше классическое образование.. Вы меня понимаете?   

[Даниил]

Так вот потихоньку и разваливается наше классическое образование.. Вы меня понимаете?   

Милости просим в министерство образования с инновационными идеями.

[m_ax]

Так вот потихоньку и разваливается наше классическое образование.. Вы меня понимаете?   

Милости просим в министерство образования с инновационными идеями.

Вообще то, вы в этом должны быть больше заинтересованы, не находите?

[Igors]

Так вот потихоньку и разваливается наше классическое образование..

При всем желании никак не могу назвать свое образование классическим. Нет, ну формально я программирование изучал, но тогда рулили транзисторы и операционные усилители, а этот курс АЯиП никаким авторитетом не пользовался. Лектор был рыжий и один раз даже пришел на лекцию с яйцами (в смысле с лотком), всякому было ясно что программист - женщина и скоро уйдет в декрет.

Так что по сравнению с тем что было - не так уж плохо, чего Вы напустились? И вообще - хватит порожняк гонять с учебным примером,  для Вас есть напр такое

http://www.prog.org.ru/index.php?topic=21481.msg148890#msg148890

А то повыучивали тут паттерны всякие..

[Даниил]

Вообще то, вы в этом должны быть больше заинтересованы, не находите?

Дабы вы успокоились, скажу вам, что преподаватель выразил такую же идею - что именно перегружать операторы (+,-, /,*) не самая лучшая идея для данного решения. Но задумка была именно в dynamic_cast и кривизна решения тут мера необходимая. Просто для того, чтобы студенты думали головой и пытались найти лучшее решение в проигранной ситуации.
А так да, он всерьез задумался о том, чтобы переписать методички.

[m_ax]

Вообще то, вы в этом должны быть больше заинтересованы, не находите?

Дабы вы успокоились, скажу вам, что преподаватель выразил такую же идею - что именно перегружать операторы (+,-, /,*) не самая лучшая идея для данного решения. Но задумка была именно в dynamic_cast и кривизна решения тут мера необходимая. Просто для того, чтобы студенты думали головой и пытались найти лучшее решение в проигранной ситуации.
А так да, он всерьез задумался о том, чтобы переписать методички.

Идея тут не столько в dynamic_cast, а в том, чтобы на примере чисел продемонстрировать полиморфное поведение.
И этого эффекта можно добиться. И будет у вас возможность делать, например так:

Код

C++ (Qt)
Number *n1 = new TANumber;
Number *n2 = new TANumber;
 
*n1 = *n1 + *n2;
*n1 = *n1 - *n2;
*n1 = (*n1) * (*n2);
// ...
// и даже так:
TANumber n1;
TANumber n2;
 
n1 = n1 + n2;
n1 = n1 - n2;
// ...
 

 
Просто нужно вынести операторы + - * / из класса, а не пытаться делать их его членами.

ЗЫ Ну вы меня успокоили) Преподу привет от Russia Qt Forum)
ЗЫЗЫ Мы следим за качеством методичек 

[Tonal]

...
И пользоваться только AllNumber.

Ну "пользоваться только" - это несолидно.

ИМХО, в программировании нет аргумента «несолидно».
Есть лучше/хуже соответствует задаче, например.

А почему не так

Код

C++ (Qt)
struct CFractNumber : public CBaseNumber {
  CFractNumber & operator + (const CBaseNumber & other) const 
  {
    assert(dynamoc_cast <const CFactNumber *> &other) != 0);
    CFractNumber & result = mBuf[mBufCount];
    result = *this;
    mBufCount = (mBufCount + 1) % BUF_SIZE;
    return result += other;
  }
 
  private:
    static int mBufCount = 0;
    static CFractNumber mBuf[BUF_SIZE];
 
    enum {
      BUF_SIZE = 256,
    };
};
 

Потому что не контролируется переполнение

Код

C++ (Qt)
CFractNumber fun1(const CFractNumber& init, int cnt) {
  CFractNumber& res = init + init;
  for (int i = 0; i < cnt - 1; ++i)
    res = res + init;
  return res;
}
// или чуть подругому:
CFractNumber fun2(const CFractNumber& init, int cnt) {
  const CFractNumber& first = init + init;
  CFractNumber res = first;
  for (int i = 0; i < cnt - 1; ++i)
    res = res + init + first;
  return res;
}
//И где-то в коде:
CFractNumber b = fun1(a, 1000);
CFractNumber d = fun2(c, 1000);
 

По сути отличается от моего кода только распределением памяти и контролем за ней.
Ежели можешь рассчитать верхний предел одновременного количества временных объектов, и приделать гарантию, что они не пересекутся, то можно и так.

[Igors]

Потому что не контролируется переполнение

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