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 回答