Форум программистов, компьютерный форум, киберфорум
С++ для начинающих
Войти
Регистрация
Восстановить пароль
Блоги Сообщество Поиск Заказать работу  
 
Рейтинг 4.55/11: Рейтинг темы: голосов - 11, средняя оценка - 4.55
Инженер - программист МАЗ
 Аватар для zloy_developer
293 / 64 / 2
Регистрация: 05.12.2011
Сообщений: 392
Записей в блоге: 3

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

10.05.2013, 15:34. Показов 2308. Ответов 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
cpp_developer
Эксперт
20123 / 5690 / 1417
Регистрация: 09.04.2010
Сообщений: 22,546
Блог
10.05.2013, 15:34
Ответы с готовыми решениями:

Прошу найти недостатки моего кода
Суть приложения: выбираем количество примеров и сложность и начинаем считать представляемые примеры. За каждый ответ дают очки (В...

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

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

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

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

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

Не по теме:

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

1
Надоела реклама? Зарегистрируйтесь и она исчезнет полностью.
raxper
Эксперт
30234 / 6612 / 1498
Регистрация: 28.12.2010
Сообщений: 21,154
Блог
10.05.2013, 16:26
Помогаю со студенческими работами здесь

Проблемы/дедлоки данного кода
Всем привет, такой вопрос. Работаю с signalR в .net core , ее архитектура сильно похожа на webApi , так что думаю поймут и те кто никогда...

Описать работу данного кода
Нашёл я код, который мне нужен, но не понимаю как тут что работает - в библиотеках algorithm, iterator, не сильно шарю. Вот код; ...

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

Параллельное выполнение данного кода на 4-х ядрах
Необходимо, чтобы данный участок кода выполнялся параллельно на 4 ядрах. Как это сделать? Я сделал следующим образом, но у меня всё...

Как улучшить просыпаемость данного кода?
К пину INT подключена солнечная панель. Иногда устройство засыпает и не просыпается. Код тут Может использовать компаратор?...


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

Или воспользуйтесь поиском по форуму:
2
Ответ Создать тему
Новые блоги и статьи
Использование SDL3-callbacks вместо функции main() на Android, Desktop и WebAssembly
8Observer8 24.01.2026
Если вы откроете примеры для начинающих на официальном репозитории SDL3 в папке: examples, то вы увидите, что все примеры используют следующие четыре обязательные функции, а привычная функция main(). . .
моя боль
iceja 24.01.2026
Выложила интерполяцию кубическими сплайнами www. iceja. net REST сервисы временно не работают, только через Web. Написала за 56 рабочих часов этот сайт с нуля. При помощи perplexity. ai PRO , при. . .
Модель сукцессии микоризы
anaschu 24.01.2026
Решили писать научную статью с неким РОманом
http://iceja.net/ математические сервисы
iceja 20.01.2026
Обновила свой сайт http:/ / iceja. net/ , приделала Fast Fourier Transform экстраполяцию сигналов. Однако предсказывает далеко не каждый сигнал (см ограничения http:/ / iceja. net/ fourier/ docs ). Также. . .
http://iceja.net/ сервер решения полиномов
iceja 18.01.2026
Выкатила http:/ / iceja. net/ сервер решения полиномов (находит действительные корни полиномов методом Штурма). На сайте документация по API, но скажу прямо VPS слабенький и 200 000 полиномов. . .
Расчёт переходных процессов в цепи постоянного тока
igorrr37 16.01.2026
/ * Дана цепь(не выше 3-го порядка) постоянного тока с элементами R, L, C, k(ключ), U, E, J. Программа находит переходные токи и напряжения на элементах схемы классическим методом(1 и 2 з-ны. . .
Восстановить юзерскрипты Greasemonkey из бэкапа браузера
damix 15.01.2026
Если восстановить из бэкапа профиль Firefox после переустановки винды, то список юзерскриптов в Greasemonkey будет пустым. Но восстановить их можно так. Для этого понадобится консольная утилита. . .
Сукцессия микоризы: основная теория в виде двух уравнений.
anaschu 11.01.2026
https:/ / rutube. ru/ video/ 7a537f578d808e67a3c6fd818a44a5c4/
КиберФорум - форум программистов, компьютерный форум, программирование
Powered by vBulletin
Copyright ©2000 - 2026, CyberForum.ru