Loading...
kaminka avatar kaminka 5 Точки

Task2. SequenceGenerator FullOPP

Здравейте, 

Уж като гледах лекцията и разбрах за pure-virtual methods, но нещо в това упражнение не разбирам как да се използва generateSequence() в класовете FibonacciGenerator и SqrtGenerator. 

Прави ли се някаква имплементация на generateSequence() в самия клас SequenceGenerator или само в другите?

И как се достъпват startInteger и endInteger от FibonacciGenerator и SqrtGenerator, ако те наследяват SequenceGenerator и следователно техните конструктори викат конструктора на SequenceGenerator?

Примерно как да използвам startInteger и endInteger, които се дават в SqrtGenerator(4,10) и отиват в конструктора на SequenceGenerator? После как да ги използвам за generateSequence()?

class SqrtGenerator : public SequenceGenerator{
public:
    SqrtGenerator(int startInteger, int endInteger):
        SequenceGenerator(startIndex,endIndex){
        }

    double* generateSequence(){
        int index = 0;
        std::stringstream ss;
        for(int i=startIndex; i<endIndex; i++){
            nums[index] = sqrt(i);
            ss << "Sqrt(" << i << ")";
            numNames[index] = ss.str();
            index++;
        }
        return nums;
    }
};

 

Тагове:
2
C++ Programming
georgi.stef.georgiev avatar georgi.stef.georgiev 921 Точки
Best Answer

Здравей,

На прав път си, просто всеки от наследяващите класове ще си има негови си полета startIndex и endIndex, които ще се инициализират съответно от конструктора на SqrtGenerator или FibonacciGenerator.

В SequenceGenerator няма нищо, освен virtual double* generateSequence() = 0;, няма и конструктор (т.е. автоматично му се създава default ctor, който не прави нищо). Така наследяващите класове няма нужда да викат базовия конструктор (тоест няма нужда да пишеш викането, защото като не го напишеш автоматично се прави SequenceGenerator() вместо ти да го викаш). Тоест конструкторът ти за SqrtGenerator ще иглежда така:

SqrtGenerator(int startIndex, int endIndex) : startIndex(startIndex), endIndex(endIndex) { }

и тези startIndex и endIndex са полета в SqrtGenerator, които можеш да ги ползваш навсякъде в SqrtGenerator. Същото нещо и за FibonacciGenerator.

Като ги напишеш двата генератора сигурно ще забележиш, че имаш доста повторен код межу тях - startIndex и endIndex полетата, някакво поле, в което пазиш sequence-а и т.н. - тогава можеш да вмъкнеш един общ базов клас за SqrtGenerator и FibonacciGenerator, които съдържа споделените неща, наследява SequenceGenerator и се наследява от Sqrt.. и Fibonacci.., но това е вече допълнителна стъпка

Поздрави,

Жоро

1
kaminka avatar kaminka 5 Точки

Мерси Жоро :)

0
Dimitar_Petkov_Petkov avatar Dimitar_Petkov_Petkov 169 Точки

Жоро, а грешно ли е само SequenceGenerator да има полета startIndex, endIndex, а наследяващите генератори да да имат само ctor, който "вика" ctor на SequenceGenerator-a  и метода GenerateSequence() който ползва наследените полета и генерира поредицата?

0
georgi.stef.georgiev avatar georgi.stef.georgiev 921 Точки

Не бих казал че е грешно, защото никъде в условието не пише, че това трябва да е pure-virtual class (т.е. ООП интерфейс). Аз така бих го направил и бих пренесъл общите неща в наследяващ го абстрактен клас, който вече да наследя с конкретните Sqrt и Fibonacci, но това е по-скоро въпрос на стил, не е толкова на добри и лоши практики (разбира се зависи от това какво се иска в проекта, но предвид, че в случая не е уточнено, може да се направи както прецените)

0
kaminka avatar kaminka 5 Точки

А защо става така че :

 std::cout << gen.getName(5) << " " << gen.getValue(5) << std::endl;

принтва:

Sqrt(4)Sqrt(5)Sqrt(6)Sqrt(7)Sqrt(8)Sqrt(9) 3

Ето къде се пълни array-а: 

double* SqrtGenerator::generateSequence(){
    int index = 0;
    std::stringstream ss;
    for(int i=startIndex; i<endIndex; i++){
        nums[index] = sqrt(i);
        ss << "Sqrt(" << i << ")";
        numNames[index] = ss.str();
        index++;
        ss.clear();
    }
    return nums;
}

Явно нещо този stringstream не се изчиства. 

0
georgi.stef.georgiev avatar georgi.stef.georgiev 921 Точки

.clear() не прави това, което мислиш :)

.clear() идва (наследява се) от stream класовете - бяхме споменавали, че cin.clear() изчиства error state-а, ако stream-а се намира в такъв. Тоест .clear() няма да ти опразни stringstream-а. За да го опразниш трябва да му кажеш .str("") - това ще му set-не празен string (или какъвто string сложиш в кавичките) - тук .str() метода има два варианта, този без параметри просто връща стойността, а този с параметър я сменя.

Но предвид, че не го ползваш този stringstream извън цикъла (и че трябва да се reset-ва на всяка итерация), защо го инициализираш извън него? Един принцип на качествения код е една променлива да се вижда само в блока, в който се ползва - ако се притесняваш за performance, не го мисли, компилаторът обикновено оптимизира такива неща. Ако го декларираш вътре в цикъла, кодът става по-ясен, защото няма да ти се налага да правиш това .str("").

Един съвет за C++ - винаги проверявай какво прави един метод, когато не си напълно сигурна, тук можеш да ги намериш разбираемо описани и с примери: http://www.cplusplus.com/reference. Именованията и като цяло ООП йерархията на стандартната библиотека са правени преди да се развият добре принципите за качествен ООП код и има много такива объркващи неща. Аз лично почти винаги проверявам някаква функция, дори за лекциите ви всички функции ги прочитам наново за да съм сигурен, че не ви подвеждам за нещо (а съм ги писал по 100 пъти сигурно).

Поздрави,

Жоро

0
kaminka avatar kaminka 5 Точки

Окии. Ще ги проверявам. Благодаря ти пак!

0
Можем ли да използваме бисквитки?
Ние използваме бисквитки и подобни технологии, за да предоставим нашите услуги. Можете да се съгласите с всички или част от тях.
Назад
Функционални
Използваме бисквитки и подобни технологии, за да предоставим нашите услуги. Използваме „сесийни“ бисквитки, за да Ви идентифицираме временно. Те се пазят само по време на активната употреба на услугите ни. След излизане от приложението, затваряне на браузъра или мобилното устройство, данните се трият. Използваме бисквитки, за да предоставим опцията „Запомни Ме“, която Ви позволява да използвате нашите услуги без да предоставяте потребителско име и парола. Допълнително е възможно да използваме бисквитки за да съхраняваме различни малки настройки, като избор на езика, позиции на менюта и персонализирано съдържание. Използваме бисквитки и за измерване на маркетинговите ни усилия.
Рекламни
Използваме бисквитки, за да измерваме маркетинг ефективността ни, броене на посещения, както и за проследяването дали дадено електронно писмо е било отворено.