Форум программистов, компьютерный форум CyberForum.ru

Недостатки данного кода С++ - C++

Восстановить пароль Регистрация
 
zloy_developer
Программист БГУИР
 Аватар для zloy_developer
291 / 60 / 2
Регистрация: 05.12.2011
Сообщений: 369
Записей в блоге: 3
10.05.2013, 15:34     Недостатки данного кода С++ #1
Здравствуйте, Уважаемые Форумчане! Подскажите, пожалуйста.
Что не так в этом коде? Перечислите недостатки, которые вы видите. Предложите свой вариант рефакторинга.
C++
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
#include <stdio.h>
 
class Feature
{
public:
        enum FeatureType {eUnknown, eCircle, eTriangle, eSquare};
 
        Feature() : type(eUnknown), points(0) {       }
 
        ~Feature()
        {
               if (points)
                       delete points;
        }
 
        bool isValid() 
        {
               return type != eUnknown;
        }
 
        bool read(FILE* file)
        {              
               if (fread(&type, sizeof(FeatureType), 1, file) != sizeof(FeatureType))
                       return false;
               short n = 0;
               switch (type) 
               {
               case eCircle: n = 3; break;
               case eTriangle: n = 6; break;
               case eSquare: n = 8; break;
               default: type = eUnknown; return false;
               }
               points = new double[n];
               if (!points)
                       return false;
               return fread(&points, sizeof(double), n, file) == n*sizeof(double);
        }
        void draw()
        {
               switch (type)
               {
               case eCircle: drawCircle(points[0], points[1], points[2]); break;
               case eTriangle: drawPoligon(points, 6); break;
               case eSquare: drawPoligon(points, 8); break;
               }
        }
 
protected:
        void drawCircle(double centerX, double centerY, double radius);
        void drawPoligon(double* points, int size);
 
        double* points;
        FeatureType type;              
};
 
int main(int argc, char* argv[])
{
        Feature feature;
        FILE* file = fopen("features.dat", "r");
        feature.read(file);
        if (!feature.isValid())
               return 1;
        return 0;
}
Спасибо!
Similar
Эксперт
41792 / 34177 / 6122
Регистрация: 12.04.2006
Сообщений: 57,940
10.05.2013, 15:34     Недостатки данного кода С++
Посмотрите здесь:

C++ [C++]Объясните смысл данного кода
C++ Будет ли утечка памяти при выполнении данного кода?
C++ Прошу пояснить тонкости данного кода
Поразрядная сортировка и его недостатки C++
Каким будет результат компиляции и выполнения данного кода? C++
C++ Каким будет результат компиляции и выполнения данного кода?
QUICKsort и MERGEsort недостатки и преимущества C++
Помогите составить блок-схему для данного кода C++

Искать еще темы с ответами

Или воспользуйтесь поиском по форуму:
После регистрации реклама в сообщениях будет скрыта и будут доступны все возможности форума.
NoMasters
Псевдослучайный
1737 / 1080 / 69
Регистрация: 13.09.2011
Сообщений: 3,093
10.05.2013, 16:26     Недостатки данного кода С++ #2
Стоит проверять значение, возвращаемое fopen и закрывать файл после окончания работы с ним. А в деструкторе delete нужен со скобками, массив как-никак.

Добавлено через 1 минуту
+инициализировать points стоит не целочисленным нулём, а соответствующим типу.

Добавлено через 27 минут
А ещё при повторном вызове read утекает память... И sizeof для enum зависит от реализации и возможных значений.

Не по теме:

С++ позволяет сделать бесконечное число ошибок на ровном месте

Yandex
Объявления
10.05.2013, 16:26     Недостатки данного кода С++
Ответ Создать тему
Опции темы

Текущее время: 00:55. Часовой пояс GMT +3.
КиберФорум - форум программистов, компьютерный форум, программирование
Powered by vBulletin® Version 3.8.9
Copyright ©2000 - 2016, vBulletin Solutions, Inc.
Рейтинг@Mail.ru