Это слишком много кода для библиотеки только заголовка? - PullRequest
5 голосов
/ 07 апреля 2010

Похоже, мне пришлось вставить здесь немного кода.Мне интересно, если это плохая практика проектирования - оставлять это целиком в заголовочном файле, например:

#include <list>
#include <string>
#include <boost/noncopyable.hpp>
#include <boost/make_shared.hpp>
#include <boost/iterator/iterator_facade.hpp>
#include <Windows.h>
#include "../Exception.hpp"

namespace WindowsAPI { namespace FileSystem {

class NonRecursiveEnumeration;
class RecursiveEnumeration;
struct AllResults;
struct FilesOnly;

template <typename Filter_T = AllResults, typename Recurse_T = NonRecursiveEnumeration>
class DirectoryIterator;

template <typename Recurse_T>
struct FileData;

class NonRecursiveEnumeration : public boost::noncopyable
{
    WIN32_FIND_DATAW currentData;
    HANDLE hFind;
    std::wstring root;
public:
    NonRecursiveEnumeration() : hFind(INVALID_HANDLE_VALUE) {
    };
    NonRecursiveEnumeration(const std::wstring& pathSpec) {
        std::wstring::const_iterator lastSlash =
            std::find(pathSpec.rbegin(), pathSpec.rend(), L'\\').base();
        if (lastSlash != pathSpec.end())
            root.assign(pathSpec.begin(), lastSlash);
        hFind = FindFirstFileW(pathSpec.c_str(), &currentData);
        if (hFind == INVALID_HANDLE_VALUE)
            WindowsApiException::ThrowFromLastError();
        while (!wcscmp(currentData.cFileName, L".") || !wcscmp(currentData.cFileName, L"..")) {
            increment();
        }
    };
    void increment() {
        BOOL success =
            FindNextFile(hFind, &currentData);
        if (success)
            return;
        DWORD error = GetLastError();
        if (error == ERROR_NO_MORE_FILES) {
            FindClose(hFind);
            hFind = INVALID_HANDLE_VALUE;
        } else {
            WindowsApiException::Throw(error);
        }
    };
    ~NonRecursiveEnumeration() {
        if (hFind != INVALID_HANDLE_VALUE)
            FindClose(hFind);
    };
    bool equal(const NonRecursiveEnumeration& other) const {
        if (this == &other)
            return true;
        return hFind == other.hFind;
    };
    const std::wstring& GetPathRoot() const {
        return root;
    };
    const WIN32_FIND_DATAW& GetCurrentFindData() const {
        return currentData;
    };
};

//Not implemented yet
class RecursiveEnumeration : public boost::noncopyable
{
};

template <typename Recurse_T>
struct FileData //Serves as a proxy to the WIN32_FIND_DATA struture inside the iterator.
{
    const Recurse_T* impl;
    template <typename Filter_T, typename Recurse_T>
    FileData(const DirectoryIterator<Filter_T, Recurse_T>* parent) : impl(parent->impl.get()) {};
    DWORD GetAttributes() const {
        return impl->GetCurrentFindData().dwFileAttributes;
    };
    bool IsDirectory() const {
        return (GetAttributes() & FILE_ATTRIBUTE_DIRECTORY) != 0;
    };
    bool IsFile() const {
        return !IsDirectory();
    };
    bool IsArchive() const {
        return (GetAttributes() & FILE_ATTRIBUTE_ARCHIVE) != 0;
    };
    bool IsReadOnly() const {
        return (GetAttributes() & FILE_ATTRIBUTE_READONLY) != 0;
    };
    unsigned __int64 GetSize() const {
        ULARGE_INTEGER intValue;
        intValue.LowPart = impl.GetCurrentFindData().nFileSizeLow;
        intValue.HighPart = impl.GetCurrentFindData().nFileSizeHigh;
        return intValue.QuadPart;
    };
    std::wstring GetFolderPath() const {
        return impl->GetPathRoot();
    };
    std::wstring GetFileName() const {
        return impl->GetCurrentFindData().cFileName;
    };
    std::wstring GetFullFileName() const {
        return GetFolderPath() + GetFileName();
    };
    std::wstring GetShortFileName() const {
        return impl->GetCurrentFindData().cAlternateFileName;
    };
    FILETIME GetCreationTime() const {
        return impl->GetCurrentFindData().ftCreationTime;
    };
    FILETIME GetLastAccessTime() const {
        return impl->GetCurrentFindData().ftLastAccessTime;
    };
    FILETIME GetLastWriteTime() const {
        return impl->GetCurrentFindData().ftLastWriteTime;
    };
};

struct AllResults
{
    template <typename Recurse_T>
    bool operator()(const FileData<Recurse_T>&) {
        return true;
    };
}; 

struct FilesOnly
{
    template <typename Recurse_T>
    bool operator()(const FileData<Recurse_T>& arg) {
        return arg.IsFile();
    };
};

#pragma warning(push)
#pragma warning(disable: 4355)
template <typename Filter_T, typename Recurse_T>
class DirectoryIterator : public boost::iterator_facade<DirectoryIterator<Filter_T>, const FileData<Recurse_T>, std::input_iterator_tag>
{
    friend class boost::iterator_core_access;
    boost::shared_ptr<Recurse_T> impl;
    FileData<Recurse_T> derefData;
    Filter_T filter;
    void increment() {
        do {
            impl->increment();
        } while (! filter(derefData));
    };
    bool equal(const DirectoryIterator& other) const {
        return impl->equal(*other.impl);
    };
    const FileData<Recurse_T>& dereference() const {
        return derefData;
    };
public:
    typedef FileData<Recurse_T> DataType;
    friend struct DataType;
    DirectoryIterator(Filter_T functor = Filter_T()) :
        impl(boost::make_shared<Recurse_T>()),
        derefData(this),
        filter(functor) {
    };
    explicit DirectoryIterator(const std::wstring& pathSpec, Filter_T functor = Filter_T()) :
        impl(boost::make_shared<Recurse_T>(pathSpec)),
        derefData(this),
        filter(functor) {
    };
};
#pragma warning(pop)

}}

Ответы [ 4 ]

9 голосов
/ 07 апреля 2010

У меня гораздо больше кода в моем, если это утешает.как и все реализации стандартной библиотеки C ++, Boost и Microsoft (например, ATL).

6 голосов
/ 07 апреля 2010

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

1 голос
/ 08 апреля 2010

Кажется, вы здесь программируете для Windows. Предположим, вы используете Visual Studio?

В любом случае, я не думаю, что в заголовках слишком много кода.

Это в основном компромиссы:

  • медленная компиляция (но у нас есть многоядерные и предварительно скомпилированные заголовки)
  • более частая перекомпиляция (опять же, многоядерные)
  • возможно, код наворотов ...

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

Честно говоря, я бы не слишком волновался об этом. Некоторые библиотеки Boost предназначены только для заголовков, даже если они не являются шаблонными частями просто потому, что это облегчает интеграцию (связывание не требуется).

1 голос
/ 07 апреля 2010

Что касается длины заголовка, вы можете иметь столько кода, сколько захотите в заголовочных файлах. Компромисс - это объем кода, который должен быть перекомпилирован при каждой сборке вашей программы; код, помещенный в ваши файлы CPP, может быть скомпилирован в объектные файлы и связан в каждой последующей сборке.

Я бы предложил перенести каждое определение метода для DirectoryIteratorImpl в файл .cpp. Если вы не определяете встроенный метод внутри определения класса, нет причин для его включения в файл заголовка.

Вне зависимости: избегайте написания inline DirectoryIteratorImpl(); - на самом деле пишите встроенные функции как встроенные или не помечайте их как встроенные. Из C ++ FAQ Lite :

Обычно обязательно, чтобы определение функции (часть между {...}) быть помещен в заголовочный файл. Если вы поместите определение встроенной функции в .cpp файл, и если он вызывается из какого-либо другого файла .cpp, вы получите «неразрешенный внешний» ошибка от компоновщика.

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

...