Написание классов - хорошая практика

Question

Первое, что я действительно не хотел публиковать на этом обмене кода стека, потому что это действительно небольшой объем кода, написанный примерно за 5 минут.

Я хочу спросить вас, допустим ли класс (мой первый в C++), который я написал. У меня на самом деле не было много кода C++, поэтому я не могу сравнивать это ни с чем.

Но я видел некоторые классы, реализующие только объявления функций, внутренняя часть этих функций была написана где-то еще в коде.

Я прошу вас о любых предложениях, если что-то не так. И почему они делают, как я описал в параграфе выше? Какой стиль кодирования лучше?

class File { 

private: 

    FILE *_handler; 
    char *_path; 
    long _size; 

    void setHandler(char *mode) 
    { 
     this->_handler = fopen(this->_path, mode); 
    } 

public: 

    File(char *path) 
    { 
     this->_path = path; 
    } 

    size_t read() 
    { 
     this->setHandler("r"); 
     char *buffer = (char*) malloc(sizeof(char)*this->_size); 

     return fread(buffer, 1, this->_size, this->_handler); 
    } 

    void write(char *data) 
    { 
     this->setHandler("w"); 
     fputs(data, this->_handler); 
    } 

    long size() 
    { 
     if(! sizeof(this->_size) > 0) 
     { 
      fseek(this->_handler, 0, SEEK_END); 
      this->_size = ftell(this->_handler); 
      rewind(this->_handler); 
     } 

     return this->_size; 
    } 

}; // End File 

Answer 1

Здесь есть технические проблемы и то, что я рассматриваю как фундаментальные проблемы с дизайном.

технический:

Сколько раз вы открыть файл? Сколько раз вы его закрываете? Посмотрите, что делают read() и write().

Где находится обработка ошибок? Что произойдет, если fopen() не удастся. Никогда не используйте возвращаемые значения, не проверяя их.

Фундаментальные проблемы проектирования:

Вы выделяете память, которая освобождает его? Как правило, это плохая идея отделить ответственность за выделение и освобождение. Люди C++, как правило, используют умные указатели, чтобы помочь с этим.

Что сделал бы ваш код при наличии действительно большого файла?

Самый фундаментальный из всех: ваш интерфейс - это «вы должны помнить об этом» интерфейсе. Что произойдет, если кто-то называет read(), не задумываясь сначала вызвать size()? Почему ваш вызывающий абонент должен это делать? Создайте свой интерфейс с целью сделать жизнь вашего звонящего простым.

Answer 2

Там нет необходимости использовать this, если нет никакой двусмысленности.

File(char *path) 
{ 
    _path = path; 
} 

То же самое с функциями вы можете оставить this

size_t read() 
{ 
    setHandler("r"); 
    char *buffer = (char*) malloc(sizeof(char)*_size); 

    return fread(buffer, 1, _size, _handler); 
} 

Исполнительные функции в объявлении класса имеет его использования, но это не является обязательным (если с помощью шаблонов), вы можете определить свои функции в качестве источника файл и включите в него заголовочный файл класса. Это сохраняет реализацию класса отдельно от интерфейса класса.

Представьте, что вы изменяете реализацию одной функции в файле заголовка, все файлы, включая этот заголовок, даже если они не используют эту функцию, нуждаются в перекомпиляции.

Поскольку вы используете C++, вам может понадобиться изучить объекты файла C++ (fstream, ifstream, ofstream и т. Д.).

И, наконец, я не вижу смысла обертывать такой файл, если только ваш класс не предоставляет дополнительную функциональность, все, что вы здесь делали, это изменение имени функций и создание другого уровня абстракции, который не принести много к столу.