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

Покритикуйте пожалуйста программу - C++

Восстановить пароль Регистрация
Другие темы раздела
C++ Поиск элемента в списке. http://www.cyberforum.ru/cpp-beginners/thread362908.html
Есть список, который содержит объекты класа type. Мне нужно найти в этом списке объект, который будет отвечать некоторым условиям. Пробую реализовать это с помощью рекурсии type GetUnit(std::list<type*> List, std::list<type*>::iterator itr) { if ((*itr)) { if <условие> return itr; if (itr != List.end()) {
C++ Предварительное объявление Добрый вечер, Есть файл file.h и file.cpp, в file.cpp я делаю подключение #include <boost/signal.hpp>, не подключаю в хэдер file.h умышленно, потому что этот хэдер подключается в много других файлов, проблема состоит в том чтобы в file.h объявить указатель на сигнал Signal *errorSignal; пробовал сделать предварительное объявление в file.h так: namespace boost { class signal<void... http://www.cyberforum.ru/cpp-beginners/thread362902.html
C++ Введите с клавиатуры данные о температуре воздуха за месяц. Определите, сколько раз за месяц была самая низкая температура?
Собственно, проходим циклы, возникли проблемы с этой задачей. Нуждаюсь в вашей помощи.
getline C++
Я считываю число из входного потока, но дальше из входного потока уже не могу считать строку. Его надо очищать что ли? #include <stdafx.h> #include <iostream> #include <locale.h> #include <string> using namespace std; int main()
C++ С чего начать программирование? http://www.cyberforum.ru/cpp-beginners/thread362877.html
Не нашёл тему, уж извините. Проблема в том, что: Я не знаю программирования. Впринципе. Мне посоветовали С++. Как думаете? Стоит ли начинать именно с него или с чего то другого?
C++ Быстрое чтение и запись файлов Вопрос такой: имеется файл input.txt следующего вида: 5 12 6 7 44 2 1 2 3 4 5 2 343 634 54 1 123 2 345 56 56 11111 2222 333 44 5 подробнее

Показать сообщение отдельно
rangerx
1908 / 1517 / 139
Регистрация: 31.05.2009
Сообщений: 2,876
08.10.2011, 13:05     Покритикуйте пожалуйста программу
То, что касается класса:
1) Все сеттеры не мешало бы переименовать из add_x в set_x.
2) Все геттеры должны быть константными методами т.е.
C++
1
char* get_x() const;
и раз уж поля класса, значения которых они возвращают, имеют тип char* - возвращать const char*.
C++
1
const char* get_x() const;
3) Не может существовать объект класса Student без установленных полей. У данного класса должен быть конструктор с параметрами.
4) Во всех сеттерах перед использованием new память, что была выделена ранее, не освобождается. Как результат, утечка.
5) Отсутствует деструктор, в котором должны освобождаться выделенные ресурсы. При уничтожении объекта, опять же, утечка памяти.
6)
Цитата Сообщение от MrGluck Посмотреть сообщение
surname = new char (strlen(y) - 1);
это выделение памяти под 1 элемент типа char и инициализация его числом strlen(y) - 1. Правильный вариант должен выглядеть так
C++
1
surname = new char[ strlen(y) + 1];
7) Дублирование кода в виде
C++
1
2
surname = new char (strlen(y) - 1);
strcpy(surname, y);
8) Нет реализации конструктора копий и оператора присваивания.
9) Все сеттеры в качестве параметра должны принимать const char*, а не char*.
10) Булевым переменным принято передавать значения true\false, а не 1\0.
11) Не подключён <cstring> хотя используется функция strcpy.
 
Текущее время: 12:01. Часовой пояс GMT +3.
КиберФорум - форум программистов, компьютерный форум, программирование
Powered by vBulletin® Version 3.8.9
Copyright ©2000 - 2016, vBulletin Solutions, Inc.
Рейтинг@Mail.ru