Форум программистов, компьютерный форум, киберфорум
Наши страницы

С++ для начинающих

Войти
Регистрация
Восстановить пароль
 
zloy_developer
Программист БГУИР
291 / 60 / 2
Регистрация: 05.12.2011
Сообщений: 375
Записей в блоге: 3
#1

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

10.05.2013, 15:34. Просмотров 878. Ответов 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;
}
Спасибо!
0
Надоела реклама? Зарегистрируйтесь и она исчезнет полностью.
Similar
Эксперт
41792 / 34177 / 6122
Регистрация: 12.04.2006
Сообщений: 57,940
10.05.2013, 15:34
Здравствуйте! Я подобрал для вас темы с ответами на вопрос Недостатки данного кода С++ (C++):

Объясните смысл данного кода - C++
Такой пример: int array; for_each(array, array + 5, (int &amp;p){p = 0;}); cout &lt;&lt; array; Результат такой:0, 0, 0, 0, 0 А как...

Пояснить значение данного кода - C++
int dRow = (row != 0 &amp;&amp; arr == (value - 1)) ? -1 : \ (row != (n - 1) &amp;&amp; arr == (value - 1)) ? 1 : 0;

Прошу пояснить тонкости данного кода - C++
#include &lt;math.h&gt; &lt;...&gt; class CAccumulator { private: float value; static int lastId; int myId; public: CAccumulator( const...

Нарисовать блок схему для данного кода - C++
#include &quot;stdafx.h&quot; #include &lt;iostream&gt; using namespace std; int main() { float cost; cout &lt;&lt; &quot;Cost (rur/kg): &quot;;...

Помогите составить блок-схему для данного кода - C++
Помогите пожалуйста нарисовать алгоритм для этой программы: #include &lt;iostream&gt; #include &lt;conio.h&gt; #include &lt;cstring&gt; #include...

Каким будет результат компиляции и выполнения данного кода? - C++
void main(void) { char alphabet ; int index; for (letter = &quot;A&quot;, index = 0; letter &lt;= &quot;Я&quot;; letter++, index++) alphabet = letter;...

1
NoMasters
Псевдослучайный
1764 / 1107 / 73
Регистрация: 13.09.2011
Сообщений: 3,143
10.05.2013, 16:26 #2
Стоит проверять значение, возвращаемое fopen и закрывать файл после окончания работы с ним. А в деструкторе delete нужен со скобками, массив как-никак.

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

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

Не по теме:

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

1
MoreAnswers
Эксперт
37091 / 29110 / 5898
Регистрация: 17.06.2006
Сообщений: 43,301
10.05.2013, 16:26
Привет! Вот еще темы с ответами:

Каким будет результат компиляции и выполнения данного кода? - C++
struct employee { char name ; int office_number; }; using namespace std; void main(void) { employee *worker; worker = new...

Будет ли утечка памяти при выполнении данного кода? - C++
собстна, сабж. char* cpy(char* source, int pos, int kolv) { char* dist = new char; for(int i=pos; i &lt; pos+kolv; i++) dist =...

Не могу понять смысл данного задания, и написание его кода - C++
В одномерном массиве все отрицательные элементы переместить в начало массива, а остальные — в конец с сохранением порядка следования....

В чем причина неработоспособности данного кода расчета значения формулы - C++
Задание При выполнении, 1 выполняет сразу два действия место одного и т.д., не хочет работать уже и условия менял. В другой...


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

Или воспользуйтесь поиском по форуму:
2
Ответ Создать тему
Опции темы

КиберФорум - форум программистов, компьютерный форум, программирование
Powered by vBulletin® Version 3.8.9
Copyright ©2000 - 2017, vBulletin Solutions, Inc.
Рейтинг@Mail.ru