Версия для печати темы Нажмите сюда для просмотра этой темы в оригинальном формате

Форум программистов > C/C++: Общие вопросы > Конструктор производного класса

Автор: KasMP 15.7.2009, 12:14

Приветствую вас, мои самые-самые  ! Предполагается такая структура классов и их наследников: Figure Point Line FilledFigure Ellipse Rectangle Сейчас важны только Figure и Point. Код class Figure { protected:     char    type; // принимает значения от "0" до "4" (ничего, точка, линия, эллипс, прямоугольник)     color    fpen; // объявлено выше: struct color {short r, g, g;}; public:     bool ValidType (char i);     Figure (ifstream *f, char ttype);     Figure (char i, color j);     // другие конструкторы     // функции доступа     virtual void Draw (HWND hWnd);     virtual bool PtInFigure (POINT pt); }; Код class Point : public Figure { protected:     POINT    p; public:     Point (ifstream *f): Figure(ifstream *f,'1');     // другие конструкторы     // функции доступа     void Draw (HWND hWnd);     bool PtInFigure (POINT pt); }; Предполагается, что точка задана в файле примерно так: Цитата point [три числа rgb] [два числа координат x и y] С конструктором фигуры все очевидно (но приведу его на всякий случай - вдруг ключ к разгадке проблем именно в нем): Код Figure :: Figure (ifstream *f, char ttype) {         *f >> fpen.r >> fpen.g >> fpen.b;         type = ttype;}; С конструктором точки все тоже очевидно, но почему-то не работает: Код Point :: Point (ifstream *f): Figure(ifstream *f,'1') {*f >> p.x >> p.y;}; Т.е. получается так: мы узнали, что впереди точка; сначала вызвался конструктор Фигуры и прочитал цвет пера; потом вызвался конструктор производного класса Точка и прочитал координаты. Но VS 2008 Express Edition совсем не думает, что все правильно и просто: Цитата(Ошибки) 1> ... error C2144: syntax error : 'std::ifstream' should be preceded by ')' 1> ... error C2612: trailing 'type' illegal in base/member initializer list 1> ... error C2082: redefinition of formal parameter 'f' 1> ... error C2059: syntax error : 'constant' 1> ... error C2059: syntax error : ')' 1> ... error C2143: syntax error : missing ';' before '{' Все ошибки относятся к строке с конструктором Точки: Код Point :: Point (ifstream *f): Figure(ifstream *f,'1') {*f >> p.x >> p.y;}; Я совсем не понимаю, что не так и что можно с этим сделать . Помогите, пожалуйста  ...

Автор: mes 15.7.2009, 12:21

Цитата(KasMP @  15.7.2009,  11:14 )  Point (ifstream *f): Figure(ifstream *f,'1');  в объявлении конструктура список инициализации лишний. P.S. привет.. давно тебя не было видно  Добавлено @ 12:23 Цитата(KasMP @  15.7.2009,  11:14 )     char    type; // принимает значения от "0" до "4" (ничего, точка, линия, эллипс, прямоугольник) не очень хорошое решение, чтоб объект базового класса знал о возможных наследниках. Добавлено @ 12:24 P.S.. я так понял Вы теперь наконец то дошли до классов  Добавлено @ 12:33 Цитата(KasMP @  15.7.2009,  11:14 ) # Figure     * Point     * Line     * FilledFigure           o Ellipse           o Rectangle я надеюсь, что Point (:Figure) не используется как примитив Point для остальных фигур   в том смысле, что точка-фигура и геометрическая точка на плоскости два разных объекта.

Автор: azesmcar 15.7.2009, 12:50

Цитата(mes @  15.7.2009,  12:21 )  в объявлении конструктура список инициализации лишний. Почему? В списке инициализации вызван конструктор базового класса, другое дело, что он вызван неправильно. Код Point (ifstream *f): Figure(f,'1'); //ifstream * вот это лишнее, это вызов а не объявление. Цитата(mes @  15.7.2009,  12:21 ) не очень хорошое решение, чтоб объект базового класса знал о возможных наследниках.

Автор: mes 15.7.2009, 12:57

Цитата(azesmcar @  15.7.2009,  11:50 ) Почему? потому что это декларация, а не определение конструктора. Цитата(azesmcar @  15.7.2009,  11:50 ) другое дело, что он вызван неправильно. кстати да, в определении конструктора список инициализации вызывается неверно.

Автор: Леопольд 15.7.2009, 12:59

Цитата(KasMP @ 15.7.2009,  12:14) Код class Point : public Figure { ... public:     Point (ifstream *f): Figure(ifstream *f,'1'); ... }; Здесь синтаксическая ошибка. Правильно вот так class Point : public Figure { ... public:     Point (ifstream *f): Figure(ifstream *f,'1'); ... };

Автор: azesmcar 15.7.2009, 13:00

Цитата(mes @  15.7.2009,  12:57 ) потому что это декларация, а не определение конструктора.   не заметил даже, а где определение? В любом случае в списке инициализации ошибка, в определении он нужен, но не в таком виде.

Автор: Леопольд 15.7.2009, 13:01

Цитата(azesmcar @ 15.7.2009,  12:50) Цитата(mes @  15.7.2009,  12:21 )  в объявлении конструктура список инициализации лишний. Почему? В списке инициализации вызван конструктор базового класса, другое дело, что он вызван неправильно. Я опоздал

Автор: mes 15.7.2009, 13:05

Цитата(Леопольд @  15.7.2009,  12:01 ) Я опоздал smile  и {} забыл   Лучше поправить чтоб не путать топик-стартера

Автор: Леопольд 15.7.2009, 14:07

Цитата(mes @ 15.7.2009,  13:05) Цитата(Леопольд @  15.7.2009,  12:01 ) Я опоздал smile  и {} забыл   Лучше поправить чтоб не путать топик-стартера  Я и сам забыл что список инициализации надо писать в определении конструктора, а в объявлении нельзя... Т.е. правильно вот так: Код struct Class {     int i;     Class(void); }; Class::Class(void):i(0){     std::cout<<"Class::Class(void)"<<std::endl; } Или так: Код struct Class {     int i;     Class(void):i(0){         std::cout<<"Class::Class(void)"<<std::endl;     } };

Автор: KasMP 16.7.2009, 10:21

Цитата(azesmcar @  15.7.2009,  12:50 ) Почему? В списке инициализации вызван конструктор базового класса, другое дело, что он вызван неправильно. Цитата(mes @  15.7.2009,  12:57 ) кстати да, в определении конструктора список инициализации вызывается неверно. Цитата(azesmcar @  15.7.2009,  12:50 ) Point (ifstream *f): Figure(f,'1'); //ifstream * вот это лишнее, это вызов а не объявление. Спасибо вам огромное! Я бы еще долго искала эту незаметную ошибку... Глупее не придумать  ! Цитата(Леопольд @  15.7.2009,  12:59 ) public:     Point (ifstream *f): Figure(ifstream *f,'1');   Цитата(azesmcar @  15.7.2009,  12:50 ) Цитата(mes @  15.7.2009,  12:21 ) не очень хорошое решение, чтоб объект базового класса знал о возможных наследниках.       Я думаю, я совсем уберу этот член: по типу переменной и так понятно, к какому классу-наследнику относится объект - зачем дублировать еще раз   ? Цитата(mes @  15.7.2009,  13:05 ) и {} забыл smile  Лучше поправить чтоб не путать топик-стартера  Да ладно, с ошибками даже лучше: человек больше вдумывается, больше понимает, больше пишет сам, ...  . Цитата(mes @  15.7.2009,  12:21 ) P.S. привет.. давно тебя не было видно   Неправда  ! Я http://forum.vingrad.ru/act-Search/CODE/show/searchid-230bfc2f8c6438475cacf2e9a01bc531/search_in-posts/result_type/posts/flag/search/index.html  ... А ты здесь уже совсем освоился   . Цитата(mes @  15.7.2009,  12:21 ) P.S.. я так понял Вы теперь наконец то дошли до классов  Мы давно до них дошли  (и ответы на свои вопросы у меня получалось находить  ), а наследование зачем-то пропустили... Вот я и занимаюсь им сама (с вашей помощью  ). Цитата(mes @  15.7.2009,  12:21 ) я надеюсь, что Point (:Figure) не используется как примитив Point для остальных фигур smile  в том смысле, что точка-фигура и геометрическая точка на плоскости два разных объекта. Нет, конечно  ! Как тебе вообще такая мысль в голову пришла      ???

Автор: mes 16.7.2009, 10:56

Цитата(KasMP @  16.7.2009,  09:21 ) Неправда smile ! Я здесь smile ... ну так это больше там, чем здесь    я в ту "даль" редко заглядываю, поэтому и не замечал  Цитата(KasMP @  16.7.2009,  09:21 ) Я думаю, я совсем уберу этот член: по типу переменной и так понятно, к какому классу-наследнику относится объект - зачем дублировать еще раз smile smile ?      Цитата(KasMP @  16.7.2009,  09:21 ) а наследование зачем-то пропустили... Вот я и занимаюсь им сама (с вашей помощью smile ).    Цитата(KasMP @  16.7.2009,  09:21 ) Цитата я надеюсь, что Point (:Figure) не используется как примитив Point для остальных фигур smile  в том смысле, что точка-фигура и геометрическая точка на плоскости два разных объект а. Нет, конечно smile ! Как тебе вообще такая мысль в голову пришла smile  smile  smile ???  посмотрел повнимательней, действительно нет : Цитата(KasMP @  15.7.2009,  11:14 ) class Point : public Figure { protected:     POINT    p; приношу извинения за ложную тревогу.     просто это самая распространенная ошибка при  составлении иерархии классов фигур. кстати заглавными буквами в C++ принято именовать макросы, и некрасиво смотрится когда также называют типы или переменные.

Автор: KasMP 16.7.2009, 11:49

Цитата(mes @  16.7.2009,  10:56 ) ну так это больше там, чем здесь  smile  я в ту "даль" редко заглядываю, поэтому и не замечал smile Заглядывай туда чаще - там тоже интересно  . Цитата(mes @  16.7.2009,  10:56 ) посмотрел повнимательней, действительно нет : Цитата(mes @  16.7.2009,  10:56 ) кстати заглавными буквами в C++ принято именовать макросы, и некрасиво смотрится когда также называют типы или переменные.  Заглавными буквами в WinAPI обозначаются именно типы, крепко-крепко связанные с этой самой WinAPI: HWND, HDC, RECT, HGDIOBJ, HPEN, HBRUSH, HFONT, COLORREF и, конечно же, POINT   .

Автор: mes 16.7.2009, 12:13

Цитата(KasMP @  16.7.2009,  10:49 ) smile Заглавными буквами в WinAPI обозначаются именно типы, крепко-крепко связанные с этой самой WinAPI: HWND, HDC, RECT, HGDIOBJ, HPEN, HBRUSH, HFONT, COLORREF и, конечно же, POINT smile smile .  ах да.. забыл про эти "пережитки" .. в .Net  они уже называются "по-человечески": http://msdn.microsoft.com/ru-ru/library/system.drawing.aspx P.S. ox чувствую, что достанется мне сейчас за эти  "пережитки", "по-человечески" и за оскорбление религиозных чувств.  будем надеяться, что пронесет..

Автор: KasMP 16.7.2009, 13:20

Имхо логично придерживаться правил, связанных именно с тем, где ты сейчас, а не правил какого-то мифического пространства  . Цитата(mes @  16.7.2009,  12:13 ) P.S. ox чувствую, что достанется мне сейчас за эти  "пережитки", "по-человечески" и за оскорбление религиозных чувств.  будем надеяться, что пронесет.. Конечно, достанется  : разве можно использовать такое тяжелое оружие как .Net, знакомый мне только понаслышке  , нагрузить все это неопровержимой ссылкой     и в итоге оставить меня совсем без возможности что-то возразить   (и еще все сложные слова взять в кавычки    ).

Автор: mes 16.7.2009, 13:24

Цитата(KasMP @  16.7.2009,  12:20 ) Имхо логично придерживаться правил, связанных именно с тем, где ты сейчас, а не правил какого-то мифического пространства smile . Ну да никуда не деться, если работать с WinAPI придется подстраиваться под их среду.

Powered by Invision Power Board (http://www.invisionboard.com)
© Invision Power Services (http://www.invisionpower.com)