Информация об изменениях

Сообщение Re: Оцените качество кода на С++ от 18.09.2014 20:30

Изменено 18.09.2014 20:31 vpchelko

Здравствуйте, GhostCoders, Вы писали:
Меня очень смущают "наполнители"
GC>reference CSewingDlg::DrawEllipse(reference element)
GC>{
GC>    EllipseParam epEllipseParam;
GC>    if (GetObjParam(element, &epEllipseParam, sizeof(EllipseParam), ALLPARAM))
GC>    {
 GC>        double x = epEllipseParam.xc,
GC>               y = epEllipseParam.yc;
GC>        double angle = epEllipseParam.ang;
GC>        double a = epEllipseParam.a;
GC>        double b = epEllipseParam.b;

GC>        const double perimeter =  ksGetCurvePerimeter (element, ST_MIX_MM); 

GC>        epEllipseParam.style = 1;

GC>        return ksEllipse(&epEllipseParam);
GC>    }
GC>    return 0;
GC>}

Таких "наполнителей" в коде дохрена.

А также куча сгенерированного кода. Свой и генерируемый код — нужно разделять, ибо поддержка IDE может перестанет работать.

Да и нотации типа m_, в новом коде я не приемлю, уже есть нормальные IDE.
Re: Оцените качество кода на С++
Здравствуйте, GhostCoders, Вы писали:
Меня очень смущают "наполнители"
GC>reference CSewingDlg::DrawEllipse(reference element)
GC>{
GC>    EllipseParam epEllipseParam;
GC>    if (GetObjParam(element, &epEllipseParam, sizeof(EllipseParam), ALLPARAM))
GC>    {
 GC>        double x = epEllipseParam.xc,
GC>               y = epEllipseParam.yc;
GC>        double angle = epEllipseParam.ang;
GC>        double a = epEllipseParam.a;
GC>        double b = epEllipseParam.b;

GC>        const double perimeter =  ksGetCurvePerimeter (element, ST_MIX_MM); 

GC>        epEllipseParam.style = 1;

GC>        return ksEllipse(&epEllipseParam);
GC>    }
GC>    return 0;
GC>}

Таких "наполнителей" в коде дохрена.

А также куча сгенерированного кода. Свой и генерируемый код — нужно разделять, ибо поддержка IDE может перестанет работать. Я бы на вашем месте генерируемый код тут не постил.

Да и нотации типа m_, в новом коде я не приемлю, уже есть нормальные IDE.