5

ここにかなりのコードをインライン化する必要があったようです。これを次のようなヘッダーファイルに完全に残すのは悪い設計慣行ではないかと思います。

#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

4 に答える 4

9

それが慰めであるならば、私は私のいくつかにはるかに多くのコードを持っています。そして、すべてのC ++標準ライブラリの実装、BoostおよびMicrosoft（たとえば、ATL）も同様です。

于 2010-04-07T19:21:12.413 に答える
6

多くの質問にオープンであると私が思う唯一の部分は、の関数の実装ですDirectoryIteratorImpl。これはテンプレートではないため、実際にはヘッダーに含める必要はなく、多少長いルーチン（「実際の」コンストラクターと増分）がいくつかあります。

残りはテンプレートであるか、またはどのような場合でもインラインにしたいような些細な関数で構成されています（たとえば、のメンバーFileData）。いずれにせよ、それらはヘッダーになります。

于 2010-04-07T19:37:00.083 に答える
1

ヘッダーの長さに関しては、ヘッダーファイルに必要な数のコードを含めることができます。トレードオフは、プログラムをビルドするたびに再コンパイルする必要のあるコードの量です。CPPファイルに配置されたコードは、オブジェクトファイルにコンパイルして、後続の各ビルドにリンクできます。

の各メソッド定義をファイルDirectoryIteratorImplに移動することをお勧めし.cppます。クラス定義内でメソッドをインラインで定義していない場合、そのメソッドがヘッダーファイルにインクルードされる理由はありません。

無関係なことは別として：書くことは避けてくださいinline DirectoryIteratorImpl();-実際にインライン関数をインラインで書くか、インラインとしてマークしないでください。C ++ FAQ Liteから：

通常、関数の定義（{...}の間の部分）をヘッダーファイルに配置することが不可欠です。インライン関数の定義を.cppファイルに入れ、それが他の.cppファイルから呼び出された場合、リンカーから「未解決の外部」エラーが発生します。

関数がヘッダーファイルに書き込むには「大きすぎる」場合、関数は大きすぎてインライン化できず、コンパイラーはインライン提案を無視する可能性があります。

于 2010-04-07T19:30:19.657 に答える
1

ここではWindows用にプログラミングしているようですが、Visual Studioを使用していると仮定しますか？

とにかく、ヘッダーにコードが多すぎるとは思いません。

これは主にトレードオフの問題です。

  • コンパイルが遅い（ただし、マルチコアとプリコンパイル済みヘッダーがあります）
  • より頻繁な再コンパイル（ここでも、マルチコア）
  • おそらくコードの膨張...

（私の意見では）厄介な唯一の点は最新のものです...そして私はここで助けが必要です：関数がインライン化されることを確信していますか、コンパイラーとリンカーがインライン化しないことを決定することは可能ではありませんそれらを通常の呼び出しに変換しますか？

率直に言って、私はそれについてあまり心配しません。多くのBoostライブラリは、統合が容易になるという理由だけで、テンプレート以外の部分でもヘッダーのみです（リンクは必要ありません）。

于 2010-04-08T06:39:21.917 に答える