2010-07-17 135 views
0

我正在尝试做一个屏幕抓取任务。我的cpp工作,但我不知道如何整合我的单元测试。我试图对文件的有效性做了布尔检查单元测试,但它给我这个错误:C++ CppUnit测试(CPPUNIT_ASSERT)

error: cannot call member function 'bool ScreenScrape::getFile()' without object 

screenscrape.cpp:

#include "screenscrape.h" 

using namespace std; 

int main() 
{ 
    ScreenScrape ss; 
    int choice; 

    ... 
    ... 

    ss.matchPatternTest(); 
} 

screenscrape.h:

class ScreenScrape 
{ 
    public: 
    ScreenScrape(); 
     void parserTest(int choice); 
     void matchPatternTest(); 
     void setIndexValue(string data, string IndexName); 
     void setIndexChange(string data); 
     void setIndexPercent(string data); 
     void setIndexDate(string data); 
     bool getFile(); 
    private: 
     string IndexName; 
     string IndexValue; 
     string IndexChange; 
     string IndexPercent; 
     string IndexVID; 
     string IndexCID; 
     string IndexPID; 
     string IndexDate; 
}; 

bool ScreenScrape::getFile() 
{ 
    string file1 = "yahoofinance.htm"; 
    char* file2 = new char [file1.size()+1]; // parse file for c string conversion 
    strcpy(file2, file1.c_str());    // converts to c string 

    ifstream fin; 

    fin.open(file2); 
    if(fin.good()) 
     return true; 
    else 
     return false; 
} 

screenscrapetest.cpp:

#include "screenscrapetest.h" 
#include "screenscrape.h" 

CPPUNIT_TEST_SUITE_REGISTRATION (ScreenScrapeTest); 

void ScreenScrapeTest::fileTest() 
{ 
    CPPUNIT_ASSERT(ScreenScrape::getFile()); // test file validity 
} 

screenscrapetest.h:

#ifndef _SCREENSCRAPETEST_H 
#define _SCREENSCRAPETEST_H 
#include <cppunit/TestCase.h> 
#include <cppunit/extensions/HelperMacros.h> 
#include "screenscrape.h" 

class ScreenScrapeTest : public CppUnit::TestFixture 
{ 
    CPPUNIT_TEST_SUITE (ScreenScrapeTest); 
    CPPUNIT_TEST (fileTest); 
    CPPUNIT_TEST_SUITE_END(); 
    public: 
     void fileTest(); 
}; 
#endif 

我试图宣布 “屏幕抓取SS;”下screenscrapetest.h,使用对象(SS)来调用的GetFile(),但它给我这个错误的倍数:

/home/user/NetBeansProjects/Assignment1/screenscrape.h:259: multiple definition of `ScreenScrape::getFile()' 

我只想与单元测试,以检查文件的有效性。任何帮助将不胜感激。 在此先感谢!

问候, 华莱士

回答

2

bool ScreenScrape::getFile()不是static,所以不能被称为静态函数。您需要(a)将其声明为static或(b)创建ScreenScrape的实例并从中调用getFile()。看看代码,这并不是很明显,为什么这个函数是类的一个方法,但它可能还处于开发的早期阶段。它也可以被重构以去除大量的冗余代码:

bool ScreenScrape::getFile() 
{ 
    std::ifstream fin("yahoofinance.htm"); 
    return fin.good(); 
} 

不要忘记你的包括警卫screenscrape.h

#ifndef SCREENSCRAPE_H 
#define SCREENSCRAPE_H 
// Class declaration here... 
#endif//ndef SCREENSCRAPE_H 

,并考虑getFile实施移动到CPP源文件。这两个步骤将防止您获得“多个声明”错误。

这将修复您的编译错误,但检查文件有效性不是单元测试的责任。单元测试不应该与文件系统交互。

0

如果你将要调用ScreenScrape::getfile()而非ss.getfile(),然后getfile()需求被定义为静态的。你得到的错误是因为需要在特定对象上调用非静态方法。

很难找到定义ScreenScrape对象的版本的错误,然后使用它来调用getfile();你显然没有包含所有相关的代码,因为你的screenscrape.h文件没有259行,并且你还没有显示修改过的代码,你在其中“使用一个对象(ss)来调用getFile()”。