0

我了解我的代码遇到的问题,但尝试其他人建议的一些事情并不能解决我的错误。

这是我的错误消息:

In file included from proj07.driver.cpp:4:
proj07.string.cpp:10: error: redefinition of 'String::String()'
/user/cse232/Projects/project07.string.h:25: error: 'String::String()' previously defined here
proj07.string.cpp: In constructor 'String::String(const char*)':
proj07.string.cpp:19: error: expected primary-expression before 'char'
proj07.string.cpp:19: error: expected `;' before 'char'
make: *** [proj07.driver.o] Error 1

这是我担心的一个:

proj07.string.cpp:10: error: redefinition of 'String::String()'
/user/cse232/Projects/project07.string.h:25: error: 'String::String()' previously defined here

我知道我应该定义一次接口文件,但我不确定要更改什么,因为我尝试的一切都会带来更多问题。

我的驱动程序(proj07.driver.cpp):

using namespace std;

#include <iostream>
#include "proj07.string.cpp"

int main()
{
  const char string1[] = {'a', 'b', 'c', 'd', 'e', 'f'};
  String test1();
  String test2(string1);
}

我的支持文件(proj07.support.cpp):

/* Implementation file for type "String" */

using namespace std;

#include <iostream>
#include "/user/cse232/Projects/project07.string.h"

/*Default constructor*/

String::String()
{
  Capacity = 0;
  Length = 0;
  Mem = NULL;
}

String::String( const char[] )
{
  cout << char.length[]; //Function not implemented yet
}

我的生成文件:

#
# Project 07
#

proj07.exe: proj07.driver.o proj07.string.o
        g++ -o proj07.exe proj.driver. proj07.string.o

proj07.driver.o: proj07.driver.cpp ~cse232/Projects/project07.string.h
        g++ -Wall -c proj07.driver.cpp

proj07.string.o: proj07.string.cpp ~cse232/Projects/project07.string.h
        g++ -Wall -c proj07.string.cpp

clean:
        rm -f proj07.*.o proj07.exe

还有头文件——这个包含了很多我还没有做的类函数。它是不可改变的。

/******************************************************************************
   Project #7 -- Interface file for type "String"
******************************************************************************/

#ifndef STRING_
#define STRING_

using namespace std;

#include <iostream>

class String
{
  private:

    unsigned Capacity;  // Number of memory locations reserved
    unsigned Length;    // Number of memory locations in use
    char * Mem;         // Pointer to memory to hold characters

  public:

    // Construct empty string
    //
    String()
    {
      Capacity = 0;
      Length = 0;
      Mem = NULL;
    }

    // Reset string to empty
    //
    void reset() { Length = 0; }

    // Return string capacity and length
    //
    unsigned capacity() const { return Capacity; }
    unsigned length() const { return Length; }

    // Return string status
    //
    bool empty() const { return Length == 0; }

    // Return reference to element I
    //
    char& operator[]( unsigned I ) { return Mem[I]; }

    // Return constant reference to element I
    //
    const char& operator[]( unsigned I ) const { return Mem[I]; }

    // Destroy string
    //
    ~String();

    // Construct string by copying existing string
    //
    String( const String& );

    // Construct string by copying C-style character string
    //
    String( const char[] );

    // Copy string into the current string
    //
    String& operator=( const String& );                                 

    // Append string to the current string
    //
    String& operator+=( const String& );
};

// Return string which is the concatenation of two strings
//
String operator+( const String&, const String& );

// Compare two strings (equality and relational operators)
//
bool operator==( const String&, const String& );
bool operator< ( const String&, const String& );

// Output string to stream
//
ostream& operator<<( ostream&, const String& );

// Input string from stream
//
istream& operator>>( istream&, String& );

#endif

我知道这个问题与我的#include 语句有关,但我对要更改哪些语句感到困惑。如果有人能告诉我我做错了什么,我将不胜感激。谢谢!

4

3 回答 3

6
#include "proj07.string.cpp"

您需要包含头文件,而不是 .cpp 文件。如果在多个 .cpp 文件中包含可能包含定义的 .cpp 文件,那么在构建时,您会遇到多个定义错误(因为在不同的 .cpp 文件中定义了多次)。

于 2010-11-03T21:50:43.717 回答
4

除了James McNellis所说的之外,您还在两个不同的地方定义了默认构造函数:

project07.string.h

class String 
{ 
  private: 

    unsigned Capacity;  // Number of memory locations reserved 
    unsigned Length;    // Number of memory locations in use 
    char * Mem;         // Pointer to memory to hold characters 

  public: 

    // Construct empty string 
    // 
    String() 
    { 
      Capacity = 0; 
      Length = 0; 
      Mem = NULL; 
    }
    /* ... */ 
};

并在proj07.support.cpp

#include "/user/cse232/Projects/project07.string.h" 

/*Default constructor*/

String::String() 
{ 
  Capacity = 0; 
  Length = 0; 
  Mem = NULL; 
} 

由于您包含string.h在 中support.cpp,编译器会看到 的默认构造函数的两种不同实现String,这违反了单一定义规则。


一些与您的问题无关的评论:

我知道你不能改变project07.string.h,但它真的不应该有using namespace std这样的东西。它使任何包含的文件project07.string.h都将整个std命名空间转储到其中,从而增加了其中一个标识符与其发生冲突的可能性。

此外,类声明似乎只需要ostreamand的前向声明istream。在这种情况下,您只需要#include <iosfwd>,而不是整个<iostream>.

于 2010-11-03T21:51:46.563 回答
1

正如消息所述,您的String构造函数定义了两次。一旦进入类定义:

class String {
    // Construct empty string
    //
    String()
    {
         Capacity = 0;
         Length = 0;
         Mem = NULL;
    }
};

并且一旦在string.cpp文件中,例如:

String::String()
{
    Capacity = 0;
    Length = 0;
    Mem = NULL;
}

此外,您可能希望将其包含string.h在您的主程序中,main.cpp否则您将在当前的 makefile 中出现链接器错误。

于 2010-11-03T21:52:27.987 回答