OO编程实践之“同步文件夹”——实现(1)

标题写的是设计，其实这里应该是写测试。刚才开始很难写出测试来，因为传统编程思维的影响。如果是以前，这段代码也许很快就能写出个能用的版本。但现在我需要提高的我代码质量，不能永远停留在那个脚步。

根据前面的需求，首先想到的是，我如何判断source中的文件比dest中的新。噢，我们只需比较最后的修改时间就行了。

于是我写了关于修改时间的简单测试

    FileModifyTime source(std::time(NULL));

    FileModifyTime dest(std::time(NULL)-1000);

    CPPUNIT_ASSERT(source.isNewerThan(dest));

    CPPUNIT_ASSERT(!source.isNewerThan(source));

    CPPUNIT_ASSERT(!dest.isNewerThan(source));

　　接着，我就写实现代码了。

    class FileModifyTime

    {

    public:

        FileModifyTime(time_t modifyTime) : itsModifyTime(modifyTime){}

        bool isNewerThan(const FileModifyTime& fi){ return itsModifyTime > fi.itsModifyTime;}

    private:

        time_t itsModifyTime;

    };

运行测试，通过了。太简单了，离我实际应用太远了。

对于第一次尝试TDD开发，我真不知道如何来写测试。考虑到测试一般而言，是给定一系列输入，然后查看输出是否与期望的相符合。在这个小程序中，我的输入是什么呢？我的输出又是什么呢？于是进入了漫长的思考中...

考虑这个程序的核心作用在source中找出已经改动的文件或者文件夹，那么测试的输入应该是各种文件信息的输入，输出是应该是是否需要更新的bool值，于是测试代码大概是这样：

Folder source("sourceFolder");

Folder dest("destFolder");

SynchronousSystem SS(&source,&dest);



CPPUNIT_ASSERT(SS.isNeedUpdate("File1"));

CPPUNIT_ASSERT(SS.isNeedUpdate("File2") == false);

这一步可能跨度有点大，因为我需要写2个类。

先写个空类能保持编译通过。

class Folder

{

public:

    Folder(const std::string& path){}

};

class Folder;

class SynchronousSystem

{

public:

    SynchronousSystem(Folder* source,Folder* dest){}

    bool isNeedUpdate(const std::string& filename)

    {

        return true;

    }

};

编译通过，但测试不通过。引文这里的我直接返回了true。为了使其通过测试，我修改了isNeedUpdate函数

    bool isNeedUpdate(const std::string& filename)

    {

        if(filename == "File1")

            return true;

        else

            return false;

    }

测试通过了，但isNeedUpdate的作用应该是判断文件修改时间，这个需要从source和dest中读取文件信息，于是继续编写测试代码：

    Folder source("sourceFolder");

    Folder dest("destFolder");



    CPPUNIT_ASSERT(source.getFileModifyTime("File1").isNewerThan(dest.getFileModifyTime("File1"));

    CPPUNIT_ASSERT(source.getFileModifyTime("File2").isNewerThan(dest.getFileModifyTime("File2") == false);

于是在Folder中需要增加成员函数getFileModifyTime

    FileModifyTime getFileModifyTime(const std::string& filename)

    {

        return FileModifyTime(std::time(NULL));

    }

为了使测试通过，我们需要在Folder中做一些工作。

class Folder

{

public:

    Folder(const std::string& path):itsPath(path){}

    FileModifyTime getFileModifyTime(const std::string& filename)

    {

        if (filename == "File1" && itsPath == "destFolder")

            return FileModifyTime(std::time(NULL)-1000);

        if (filename == "File2" && itsPath == "sourceFolder")

            return FileModifyTime(std::time(NULL)-1000);

        return FileModifyTime(std::time(NULL));

    }

private:

    std::string itsPath;

};

这样所有测试都通过了，再回过头来看SynchronousSystem，它的isNeedUpdate函数应该调用Folder看查询文件的修改时间，类SynchronousSystem修改为

class SynchronousSystem

{

public:

    SynchronousSystem(Folder* source,Folder* dest):itsSourceFolder(source),itsDestFolder(dest){}

    bool isNeedUpdate(const std::string& filename)

    {

        return itsSourceFolder->getFileModifyTime(filename).isNewerThan(itsDestFolder->getFileModifyTime(filename));

    }

private:

    Folder* itsSourceFolder;

    Folder* itsDestFolder;

};

一切很好，所有测试都通过了。

 

这样看来，一个功能1差不多完成了，考虑功能2的情况，如果dest中的文件不存在，那么isNeedUpdate中应该返回true。测试代码：

    Folder source("sourceFolder");

    Folder dest("destFolder");

    SynchronousSystem SS(&source,&dest);

    CPPUNIT_ASSERT(SS.isNeedUpdate("FileNotExist"));

测试不通过，我们更新isNeedUpdate函数

    bool isNeedUpdate(const std::string& filename)

    {

        if(!itsDestFolder->isExist(filename))

            return true;

        return itsSourceFolder->getFileModifyTime(filename).isNewerThan(itsDestFolder->getFileModifyTime(filename));

    }

类Folder中增加

    bool isExist(const std::string& filename)

    {

        if(filename == "FileNotExist")

            return false;

        else

            return true;

    }

测试通过了。