По поводу попа) Можно было сделать bool Pop(int* const value) хотя пользоваться таким было бы менее удобно.
@sergbo5509
8 жыл бұрын
Не обижайтесь на критику. Критикуют, значит внимательно слушают)
@sergbo5509
8 жыл бұрын
Прикольный стек. Ну ладно написал он типа "переполнение стека", и не выделил память под стек, а дальше то что? Правильно, метод класса попытается туда что-то впихулить. Но даже если программа передумала использовать стек, то при вызове деструктора вызовется delete на несуществующем масиве. Брависимо. Ну не знаем мы трай кэтч, но присвоил бы указателю 0 что ли. И в деструкторе уже проверил бы нужно ли delete вызывать.
@sergbo5509
8 жыл бұрын
C const int в описании класса тоже обманул! Можно там написать const int _privateConstant. Но при этом инициализировать уже в определении конструктора. Например.book::book():_privateConstant(13421){}
@sergbo5509
8 жыл бұрын
Чувак обидно, много толкового говоришь, но и вместе с тем путаешь людей.
@sergbo5509
8 жыл бұрын
Еще раз тут. Нельзя возвращать ссылку на приватные поля класса!!! ЭТО НЕ БЕЗОПАСНО! Это убивает всю вашу инкапсуляцию на корню! Да по большому счету и на публичные тоже не желательно. В идиале публичными должны быть только член-функции.
@sergbo5509
8 жыл бұрын
char* GetNameBook() const - означает не то что данные в программе меняться не будут, а то что функция-член не будет изменять внутреннее состояние объекта класса!
Пікірлер: 7