Какие ошибки допущены при проектировании класса

@trev · Регистрация: 10.07.2014

Студворк — интернет-сервис помощи студентам

Подскажите пожалуйста, какие ошибки допущены при проектировании класса, если они имеются:

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
#include "stdafx.h"
#include <fstream>
 
class FileClient
{
protected:
    char* p;
public:
 
    virtual void connect() = 0;
    void close(){}
    virtual void ping()
    {
        printf("%s : %s", "__FUNCTION__", p);
    }
    FileClient()
    {
        p = new char[10];
        p[0] = '1';
    }
    ~FileClient()
    {
        delete p;
    }
};
class Client : public FileClient
{
    FILE* hFile;
public:
    virtual void connect()
    {
        hFile = fopen("d:\test.txt", "w");
    }
    void ping()
    {
        char arr[] = { 0, 0, 5, 56, 12, 32 };
        fwrite(arr, 1, sizeof(arr), hFile);
        FileClient::ping();
    }
    ~Client()
    {
        delete[] p;
        p = 0;
    }
};
 
int main()
{
    FileClient* client = new Client;
    client->connect();
    client->ping();
    delete client;
}

@gray_fox · 27.07.2014, 16:11

Не по теме:

Сообщение от -=ЮрА=-

- раскрой глаза

Что ж, видимо я не настолько advanced что бы постичь это... :cry:

Добавлено через 3 минуты
Тогда может кто-нибудь другой мне это объяснит? У кого-нибудь глаза раскрыты?

@-=ЮрА=- · 27.07.2014, 16:12

Не по теме:

Voivoid, дорогой я вижу на тебя лечение не действует, хорошо раз ты такой умник вот тебе линк
как исправишь пиши

Добавлено через 1 минуту

Не по теме:

Сообщение от gray_fox

Что ж, видимо я не настолько advanced что бы постичь это..

- а ты сам какого уровня чтобы тебе надо всё разжёвывать? У тебя выше перегруженный делит стоит, тебе мешает твой низкий скил или желание попытаться меня потролить?Напиали - если не сделать ресет по нудевому поинтеру можно подвесить несколько процессов к примеру. Хотя я хз, по твоим ответам можно понять что управлять потоками врядли умеешь. Ладно не буду метать бисер...

@Voivoid · 27.07.2014, 16:18

Где линк-то? Не вижу

@-=ЮрА=- · 27.07.2014, 16:19

Сообщение от Voivoid

Где линк-то? Не вижу

держи Цветные строки в CComboBox
Поглядим стоит ли с тобой вообще разговаривать

@DrOffset · 27.07.2014, 16:22

Сообщение от -=ЮрА=-

по моему ты сейчас кивнул в мою сторону.

Это ты зря так думаешь.

Собственно не понятно как твой код демонстрирует мою неправоту. Одно дело, когда логика программы каким-либо образом завязана на нулевой указатель, другое дело, когда нет. Да и ты не ответил, как именно такая проверка поможет найти битый пойнтер. Мы и с проверкой и без - все равно войдем в этот if, так как любой ненулевой указатель приводится к true. А последний твой код совсем о другом, как бы, ведь там есть else, который уже подразумевает какое-то кастомное действие, но речь шла не о том первоначально.

Очень много я встречал довольно таки опытных программеров, которые имели такую привычку, проверять на ноль перед delete, и никто так и не смог внятно объяснить зачем они это делают. Самый достойный вариант был о том, что в 89 году был компилятор, который такое не допускал, а вот привычка с тех пор осталась.

Собственно, 5.3.5/2.

In the ﬁrst alternative (delete object), the value of the operand of delete may be a null pointer
value, a pointer to a non-array object created by a previous new-expression, or a pointer to a subobject (1.8)
representing a base class of such an object (Clause 10). If not, the behavior is undeﬁned. In the second
alternative (delete array), the value of the operand of delete may be a null pointer value or a pointer
value that resulted from a previous array new-expression.

@-=ЮрА=- · 27.07.2014, 16:28

DrOffset, хорошо для тебя, я всегда проверяю поинтер и предусматриваю возможные варианты, более того рекомендую писать ветки if( pointer ){}ele{}. в большинстве проектов приходится перегружать и new и delete как для отладки так и для прочих нужд. Лучше всегда иметь вариант на случай нулевого поинтера и удалять лишь живое причём небитое. Я показл выше код пост 17(если вдруг забыл). Представь если ресет из моего кода не зануляет параметр а кидает ивенту аля ABNORMAL_TERMINATE, и потом представь что если бы этой ивенты не было, а чтобы облегчить абстракцию представь что cObj некие сущности процессов.

Не по теме:

Или ты просто потролить захотел, тогда тебе с Voivoid на излечение Цветные строки в CComboBox. Далее буду разговаривать лишь с теми кто хоть внятно ответить сможет что там подправить надо(мой ехе уже в теме)

@Voivoid · 27.07.2014, 16:30

Сообщение от -=ЮрА=-

Поглядим стоит ли с тобой вообще разговаривать

Не по теме:

Забавная логика. Как-бы невзначай перевести стрелки со знания C++ на MFC. Как будто мне все это нужно :rofl:

Tulosba · 27.07.2014, 16:31

Не по теме:

:popcorn:

@-=ЮрА=- · 27.07.2014, 16:33

Не по теме:

Сообщение от Voivoid

Забавная логика. Как-бы невзначай перевести стрелки со знания C++ на MFC. Как будто мне все это нужно

- просто напиши я не могу этого сделать, с тобой понятно уже всё до конца.

@Taatshi · 27.07.2014, 18:52

Комментарий администратора

Господа, оффтоп прекращаем.

Новые блоги и статьи Все статьи Все блоги /
Транслитерация текста с кириллицы на латиницу (вариант №2) Maks 13.11.2026 Обновленная версия ранее созданной обработки. Добавлено: - Проверка исходного текста на кириллицу, через обращение к стандартной функции "ТолькоКириллицаВСтроке" из общего модуля. . .	Часы с термометром и праздничным календарем yariko 12.11.2026 Сегодня мы рассмотрим как можно сделать отличные многофункциональные часы. Итак, поставим конкретную задачу: разработать часы с календарем, двумя термодатчиками, выводом поздравительных сообщений. . .	Транслитерация текста с кириллицы на латиницу (вариант №1) Maks 11.11.2026 Решил облегчить жизнь себе и коллегам админам при создании электронных почтовых ящиков для пользователей, в результате написал обработку "Транслитерация текста" с кириллицы на латиницу, а в ней. . .	Книги по геймдизайну, левел-дизайну, игровым механикам и проектированию игрового опыта 8Observer8 08.11.2026 Содержание блога \| English Version 1. 2003 - 06 - Designing Virtual Worlds (1st Edition) Автор: Richard A. Bartle \| Объем: 768 стр. Ссылки: Amazon . . .
Родители токсичные абьюзеры по отношению к своим детям ругают их оскорбляют не любят zorxor 08.11.2026 Жить в атмосфере постоянных оскорблений, критики и холодности со стороны родителей — это тяжелое и болезненное испытание, которое оставляет глубокие эмоциональные раны. Токсичное и абьюзивное. . .	Как только я захожу к себе в свою комнату я сразу же с себя снимаю свои трусы. Ответ от Джемени ИИ. zorxor 08.11.2026 Джемени, как только я захожу к себе в свою комнату я сразу же с себя снимаю свои трусы. Многие люди, возвращаясь домой, сразу переодеваются в максимально свободную одежду или вовсе предпочитают. . .	Паттерн очереди действий/команд (Action / Command Queue). Система очереди действий и кат-сцен с SDL3 Callbacks, cglm и OpenGL 8Observer8 07.11.2026 Содержание блога \| English Version Руководство для начинающих по реализации паттерна очереди действий/ команд (Action / Command Queue) на C++ (C++20) с использованием официального SDL3 Callbacks. . .	Проверка полного ФИО в типовом справочнике Maks 07.11.2026 Решение реализовано в конфигурации КА2, на примере справочника "КонтактныеЛицаПартнеров". Задача: Запрет инициалов. Не допускаеть использование сокращений вида "И. И. ", "И. И. ", "И И" и. . .

@Voivoid 710 / 284 / 16 Регистрация: 31.03.2013 Сообщений: 1,340
	27.07.2014, 16:18
	Где линк-то? Не вижу 0

@-=ЮрА=- 6614 / 4256 / 401 Регистрация: 08.08.2009 Сообщений: 10,325 Записей в блоге: 24
	27.07.2014, 16:28
	DrOffset, хорошо для тебя, я всегда проверяю поинтер и предусматриваю возможные варианты, более того рекомендую писать ветки if( pointer ){}ele{}. в большинстве проектов приходится перегружать и new и delete как для отладки так и для прочих нужд. Лучше всегда иметь вариант на случай нулевого поинтера и удалять лишь живое причём небитое. Я показл выше код пост 17(если вдруг забыл). Представь если ресет из моего кода не зануляет параметр а кидает ивенту аля ABNORMAL_TERMINATE, и потом представь что если бы этой ивенты не было, а чтобы облегчить абстракцию представь что cObj некие сущности процессов. Не по теме: Или ты просто потролить захотел, тогда тебе с Voivoid на излечение Цветные строки в CComboBox. Далее буду разговаривать лишь с теми кто хоть внятно ответить сможет что там подправить надо(мой ехе уже в теме) 0

Tulosba
	27.07.2014, 16:31
	Не по теме: :popcorn: 0