Проблемы с классом

[Drakonkn]

Написал класс телевизор.

class TVSet
{
public:
      TVSet(int channel,string name);
     ~TVSet(void);
     int GetChannel(void);
     int SetChannelinN(int N);
     void PowerON();
     void PowerOFF();
     string GetName();
     int NextChannel();
     int PrewChannel();
};

Преподаватель выложил данный код в общий доступ (для нашей группы) с комментарием: Кто заметит несуразность в объявлении, получит +1 к баллам на девятой неделе.

Объясните что он имел ввиду. Хотелось бы самому исправиться, а не ждать пока кто-то ответит

[Rififi]

Drakonkn

Объясните что он имел ввиду.

Это обращение к женской половине группы.
Те девушки, которые в положении, получат некие баллы на 9 неделе беременности (хотя, с другой стороны, это уже вовсе и не девушки, а каГбэ настоящие женщины. Вот и первая несуразность(((: )

Но основная проблема - телевизор скорее всего китайский, позволяет наладить производство дешевых, а фактически - бесплатных копий, что ведет к повышению уровня безработицы, подрыву и экономики и дестабилизации внутреннего рынка товаров народного потребления. Возможно, это секретная программа КНР по ослаблению стран - вероятных противников

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

[pproger]

PrevChannel написано с ошибкой, несуразность налицо

[Drakonkn]

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

Drakonkn

Объясните что он имел ввиду.

Это обращение к женской половине группы.
Те девушки, которые в положении, получат некие баллы на 9 неделе беременности (хотя, с другой стороны, это уже вовсе и не девушки, а каГбэ настоящие женщины. Вот и первая несуразность(((: )

Но основная проблема - телевизор скорее всего китайский, позволяет наладить производство дешевых, а фактически - бесплатных копий, что ведет к повышению уровня безработицы, подрыву и экономики и дестабилизации внутреннего рынка товаров народного потребления. Возможно, это секретная программа КНР по ослаблению стран - вероятных противников

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

Не очень понял где тут шутка, а где намёк на проблему
По поводу порно канала, вовсе нет: при выключении сохраняется последний включенный канал (как раз это было одним из критериев данной программы)

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

[Drakonkn]

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

PrevChannel написано с ошибкой, несуразность налицо

Вы конечно правы, но сомневаюсь, что он имел ввиду именно это. Потому что смотрел лабы сокурсников, там вообще нечто феноменальное с именами переменных. Кривая транслитерация + неверно переведенные слова = имя переменной. Однако никому из них он замечаний не делал.

[ArmanPrestige]

Напиши потом, что имел в виду препод твой. Интересно

[Stilet]

ИМХО несуразность в том что класс не имеет тела, и методы не абстрактные.

[sVasilich]

А что за int channel в конструкторе? По идее, здесь должно быть что-то типа maxChannel, которая задаёт максимальное количество каналов (чтобы int NextChannel(); имел какую-то верхнюю границу). А при создании номер канала инициализируется нулём.

Возможно ещё
int NextChannel();
int PrewChannel();

можно сделать void. В реальном телевизоре такие методы переключают каналы (есть же кнопочка на пульте).

Но эти все замечания скорее ещё один вариант реализации, можно же и так (смотря какая задача).

[Drakonkn]

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

ИМХО несуразность в том что класс не имеет тела, и методы не абстрактные.

реализация в отдельном файле

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

А что за int channel в конструкторе? По идее, здесь должно быть что-то типа maxChannel, которая задаёт максимальное количество каналов (чтобы int NextChannel(); имел какую-то верхнюю границу). А при создании номер канала инициализируется нулём.

Возможно ещё
int NextChannel();
int PrewChannel();

можно сделать void. В реальном телевизоре такие методы переключают каналы (есть же кнопочка на пульте).

Но эти все замечания скорее ещё один вариант реализации, можно же и так (смотря какая задача).

int channel - номер канала при инициализации, типа на пульте нажимаешь на циферку и ТВ включается и у тебя сразу активируется тот канал, на который ты нажал) На счёт максимального канала - верно подметил, отсюда же вытекает проблема отрицательного канала. Конечно я недосмотрел, но это тривиальный баг. Думаю он не стал бы акцентировать на этом внимание всей группы.

[sVasilich]

Дык у тебя операция включения отдельная есть. По идее, конструктор создаёт новый телевизор, а PoverOn - кнопка на пульте.

И в текущей реализации получаем: ты первый раз включаешь телевизор заданным каналом, потом используя функцию PoverOn(при последующих включениях) загружаешь канал, который последним смотрели.. А куда делась кнопочка на пульте, с каналом, на который я нажимал в первый раз?) Которая:

типа на пульте нажимаешь на циферку и ТВ включается и у тебя сразу активируется тот канал, на который ты нажал)

Т.е. выходит, я его включаю каждый раз новым пультом))