Благодаря за отговора! :)

Имам решение на задачата, което изглежда удовлетворява условията, но честно казано не ми харесва - не блести с оригиналност, нито с гъвкавост, трябва да се внимава при използването му заради подводни камъни в имплементацията, а за капак има и части, които се повтарят в разклонените класове...

От коментара ти разбирам, че може би по-добро решение ще е SequencePrinterToString да наследява директно SequencePrinter, като има вариант и да се "сприятелят" частично или напълно, а SequencePrinterToFile и SequencePrinterToConsole да наследяват SequencePrinterToString (и без друго в настоящата ми имплементация те го използват за създаване на стринга, който пък още по-добре ще е да се рализира с << operator overload). Ако ми остане време от другите задачи, може да се върна и да я преработя. 

Всъщност, най-големите ми проблеми с тази задача дойдоха заради изискването SequenceGenerator да бъде представен като референция в SequencePrinter класа, което направи невъзможно директното преизползване на един SequenceGenerator обект в няколко SequencePrinter обекта, както и е предпоставка за допускане на грешки. Определено не бих избрал този начин за имплементиране, ако трябва аз да правя дизайна с настоящите си знания, но както казах, има много неща, които все още не знам... :)

В тази задача има нещо като уловка и тя е именно този const SequenceGenerator & параметър. Ако полето ти в класа е референция, няма как да имаш setSequence() метод, защото след като една референция е инициализиране е невъзможно да я накараш да сочи към нещо друго - ако кажеш setSequence(someOtherGenerator), това няма да смени твоя printer да ползва друг generator, a просто ще презапише върху предишния generator този, който подаваш (т.е. все едно извън класа си присвоил на единия генератор другия).

Това, което аз бих направил в този случай, е да си направя полето в класа да е const SequenceGenerator *, вместо референция - понеже е pointer, мога да го карам да сочи към друго място, тоест мога правилно да имплементирам setSequence(). Също така, понеже имам pure-virtual клас, който трябва да наследя с няколко класа, които имат общо помежду си, ще си направя един общ базов клас за тях - SequencePrinterBase примерно:

class SequenceGeneratorBase : public SequenceGenerator 
{
protected:
    const SequenceGenetarot* m_sequence;
    SequenceGeneratorBase() :
        m_sequence(nullptr) {}
public: 
    virtual void setSequence(const SequenceGenerator& sequence)
    {
        this->m_sequence = &sequence;
    }
}

Оттук нататък наследяваш SequenceGeneratorBase с класовете дадени в задачата, а метода го викаш ето така:

SequenceGenerator * sequence = new SqrtGenerator(...);
...
SequencePrinter * printer = new SequencePrinterToString(...);
printer->setSequence(*sequence);

Но защо тогава да е референция, а не просто указател - само за да е уловка ли? Не. Понякога се ползва референция, за да е по-трудно да се допусне грешка с параметъра - ако е указател, лесно могат да ти подадат указател към произволна памет, cast-нат към съответния тип директно като викат метода. Когато е референция, на викащия му се налага поне да има някаква променлива предварително от някакъв тип. Разбира се, в C++ винаги може да ти подадат грешен параметър, но с този код ако някой неволно сгреши докато пише е малко по-вероятно да го хване компилатора. Не е напълно солиден подход и не се ползва много, но исках да ви срещна и с тази ситуация, защото в истинската работа би могло да ви се случи нещо подобно.

Поздрави,

Жоро

Благодаря за изчерпателния отговор!

С него плюс още едно преглеждане на видеото и демата от последната лекция, нещата почнаха да ми се нареждат.

Поздрави!

Здравей,

По-скоро е задължително да имплементираш print() - замисли се за варианта с конзолата. Ако имаш няколко SequencePrinterToConsole, те всичките принтират към една и съща конзола, не към различни конзоли, нали? Ако са няколко SequencePrinterToFile, сочещи към един и същи файл, ще пишат по един и същи файл. Е за string-а е същото, все към един и същи string принтираш ако направиш няколко инстанции, които сочат към него. Като цяло идеята на задачата е да свикнете да работите с класове, които имат достъп до обекти от друг клас в себе си

Ето една валидна ситуация, в която можеш да го ползваш - искаш да имплементираш функцията на google търсачката, която като ѝ напишеш "first 5 prime numbers" ти ги извежда (то в момента просто ти извежда най-related резултата, но да речем, че има нещо което го смята директно). Обаче искаш да работи и за sqrt, и за fibonacci, и т.н. - е тогава нещото към което печаташ е все едно и също, но си подготвил няколко различни принтера и пускаш този с правилната заредена sequence (за да може ако потребителя реши да смени нещото, което гледа, да можеш директно да извикаш някоя от другите инстанции и да му покажеш друг sequence, вместо да ги изчисляваш наново). Разбира се има и други начини да го постигнеш това, но в случая се иска да имплементираш нещо, което работи така. 

Ако искаш да си поиграеш с operator overloading, това което бих ти препоръчал е да напишеш operator<<(const SequenceGenerator& sequence), вътре в SequencePrinter (тоест SequencePrinter << SequenceGenerator). Като се изпълни този оператор, принтера ти взема sequence-а и го принтира директно в целта си, каквато е тя (за да можеш с един ред код да смениш sequence-а и да го изпечаташ, вместо да трябва първо да правиш setSequence и след това да викаш print())

Поздрави,

Жоро

Мерси за отговора. :)

Здравей,

Какво имаш предвид, че не тръгва? 

Принципно домашните е достатъчно да съдържат само .h и .cpp файлове (т.е. самия код). Оттам нататък тези файлове дали ще ги Add-нете във Visual Studio, в Code:Blocks, в Qt, в CLion или ще ги компилирате на ръка от конзолата, няма значение. Виж на края на лекцията за Code Organization как взехме SmartArray класа и го компилирахме под няколко различни IDE-та от споменатите.

Ако компилираш файлове от домашно, по описания начин, би следвало да ти се компилират вярно, независимо от IDE-то, което ползваш. Ако не се компилира изобщо, значи нещо кодът не е наред. Ако се компилира под едно IDE, но не и под друго, значи най-вероятно се ползват някакви неща, които не са в C++ стандарта, който компилатора на съответното IDE поддържа.

Поздрави,

Жоро

Ох.. прав си,всичко тръгва като изполвам само хедер и соурс файловете.Извинявам се за моето невнимание и отново неуместен въпрос.
Поздрави!

Е, няма проблем, нали затова е форумът.

Не е неуместен въпрос, просто е въпрос по материал, който сме покривали. Затова е хубаво да се опитваш да прегледаш в лекциите дали сме правили нещо подобно, когато те затрудни нещо, и ако и там не намериш, тогава да питаш - така ще е по-бързо за теб, а и учебният ефект ще е по-голям, защото сам ще си разрешил проблема.

Но това не означава, че не може да питаш като видиш, че губиш твърде много време върху нещо.