1

在下面的代码中，我试图创建一个函数“patient_count”，它是“horse”、“pig”和“dog”类的朋友。我可以让功能成为 1 个班级的朋友，但不能成为所有 3 个班级的朋友。谁能告诉我我的错误是什么？

/*******************************************************\
* Veternarian Class Problem - I need a class for each   *
* of 3 animals. Horse, Pig and Dogs                     *
\*******************************************************/

#include <cstdlib>
#include <iostream>
#include <string>

const int HORSE_KENNEL = 100; // Number of horses we can store
const int PIG_KENNEL = 100; // Number of Pigs we can store
const int DOG_KENNEL = 100; // Number of Dogs we can store

/*******************************************************\
* Class horse                                           *
*                                                       *
* Member functions                                      *
* horse_count -- Keeps track of the number of horses    *
* add_horse -- Sends data into the object               *
* next_horse -- returns data from the object            *
\*******************************************************/

// Definition of the Class
class horse {

   private:
      int horse_count;              // Variable to keep track of data
      std::string horse_data[HORSE_KENNEL]; // A Place to put the data

      // Declarations for the method Prototypes
   public:            
      // Initialize 
      horse( );

      // A Function that accepts an argument but returns nothing
      void add_horse(const std::string new_horse_data);

      // This method returns the next Horse in the queue
      std::string next_horse( );

      friend int patient_count(horse);

};


/*******************************************************\
* Method Definition - Here we flush out the prototypes  *
* outlined in the last section                          *
\*******************************************************/
inline horse::horse( )
{
   for(int i = 0; i < HORSE_KENNEL; ++i){
      horse_data[i] = "Empty Spot";
   }
   horse_count = 0; // Zero the data count
}

/*******************************************************\
* horse::add_horse -- Send data to Object               *
\*******************************************************/
inline void horse::add_horse(const std::string new_horse_data)
{
   horse_data[horse_count] = new_horse_data;
   ++horse_count;
}

/*******************************************************\
* horse::next_horse - get data from object              *
\*******************************************************/

inline std::string horse::next_horse( )
{
   // this is specifically implementing a queue
   std::string current_horse = " ";
   int target_horse = 0;
   for(int i = 0;i < HORSE_KENNEL; ++i){
      if(horse_data[i] != "Empty Spot"){
         std::cout << "Horse Number " << i << " " << horse_data[i] << std::endl; 
      }
   }
   std::cout << "Select the horse you want: "; 
   std::cin >> target_horse;

   return (horse_data[target_horse]);
}

/*******************************************************\
* Class Pig                                             *
*                                                       *
* Member functions                                      *
* pig_count -- Keeps track of the number of pigs        *
* add_pig -- Sends data into the object                 *
* next_pig -- returns data from the object              *
\*******************************************************/

// Definition of the Class
class pig {

   private:
      int pig_count;              // Variable to keep track of data
      std::string pig_data[PIG_KENNEL]; // A Place to put the data

      // Declarations for the method Prototypes
   public:            
      // Initialize 
      pig( );

      // A Function that accepts an argument but returns nothing
      void add_pig(const std::string new_pig_data);

      // This method returns the next pig in the queue
      std::string next_pig( );

      friend pig patient_count(pig);
};


/*******************************************************\
* Method Definition - Here we flush out the prototypes  *
* outlined in the last section                          *
\*******************************************************/
inline pig::pig( )
{
   for(int i = 0; i < PIG_KENNEL; ++i){
      pig_data[i] = "Empty Spot";
   }
   pig_count = 0; // Zero the data count
}

/*******************************************************\
* pig::add_pig -- Send data to Object                   *
\*******************************************************/
inline void pig::add_pig(const std::string new_pig_data)
{
   pig_data[pig_count] = new_pig_data;
   ++pig_count;
}

/*******************************************************\
* pig::next_pig - get data from object                  *
\*******************************************************/

inline std::string pig::next_pig( )
{
   // this is specifically implementing a queue
   std::string current_pig = " ";
   int target_pig = 0;
   for(int i = 0;i < PIG_KENNEL; ++i){
      if(pig_data[i] != "Empty Spot"){
         std::cout << "pig Number " << i << " " << pig_data[i] << std::endl; 
      }
   }
   std::cout << "Select the pig you want: "; 
   std::cin >> target_pig;

   return (pig_data[target_pig]);
}

/*******************************************************\
* Class dog                                             *
*                                                       *
* Member functions                                      *
* dog_count -- Keeps track of the number of dogs        *
* data_to_object -- Sends data into the object          *
* data_from_object -- returns data from the object      *
\*******************************************************/

// Definition of the Class
class dog {

  private:
    int dog_count;              // Variable to keep track of data
    std::string dog_data[DOG_KENNEL]; // A Place to put the data

  // Declarations for the method Prototypes
  public:            
    // Initialize 
    dog( );

    // A Function that accepts an argument but returns nothing
    void add_dog(const std::string new_dog_data);

    // This method returns the next dog in the queue
    std::string next_dog( );

  friend dog patient_count(dog);
};


/*******************************************************\
* Method Definition - Here we flush out the prototypes  *
* outlined in the last section                          *
\*******************************************************/
inline dog::dog( )
{
   for(int i = 0; i < DOG_KENNEL; ++i){
      dog_data[i] = "Empty Spot";
   }
   dog_count = 0; // Zero the data count
}

/*******************************************************\
* dog::add_dog -- Send data to Object                   *
\*******************************************************/
inline void dog::add_dog(const std::string new_dog_data)
{
   dog_data[dog_count] = new_dog_data;
   ++dog_count;
}

/*******************************************************\
* dog::next_dog - get data from object                  *
\*******************************************************/

inline std::string dog::next_dog( )
{
   // this is specifically implementing a queue
   std::string current_dog = " ";
   int target_dog = 0;
   for(int i = 0;i < DOG_KENNEL; ++i){
      if(dog_data[i] != "Empty Spot"){
         std::cout << "dog Number " << i << " " << dog_data[i] << std::endl; 
      }
   }
   std::cout << "Select the dog you want: "; 
   std::cin >> target_dog;

   return (dog_data[target_dog]);
}

/**************************************************\
* This function is a friend of all the animal      *
* classes and returns the total of all animals     *
* PROBLEM ******* PROBLEM *********PROBLEM *********
* When I add the other 2 classes on the next line  *
* The program stops working                        *
\**************************************************/
// int patient_count(horse target_horse) //works
int patient_count(horse target_horse, pig target_pig, dog target_dog) // Nova
{
   //  int all_animals = target_horse.horse_count; //Works
   int all_animals = target_horse.horse_count + target_pig.pig_count + target_dog.dog_count; // Nova

   return (all_animals);
}
/**************************************************\
* The Class is defined above, this section is a    *
* Small testing harness to verify that the class   *
* is doing what it was designed to do              *
\**************************************************/
int main( )
{
   int total_animals;
   horse current_horse; // Create a instance

   // Send 3 values to the object
   current_horse.add_horse("Mr Ed, 10, Male");
   current_horse.add_horse("Lightning, 4, Female");
   current_horse.add_horse("Blitz, 7, Male");

   // Call for the return of the 3 values
   std::cout << "Selected Horse  ->" << current_horse.next_horse( ) << '\n';

   pig current_pig; // Create a instance

   // Send 3 values to the object
   current_pig.add_pig("Arnold, 4, Male");
   current_pig.add_pig("Babe, 2, Female");
   current_pig.add_pig("Killer, 7, Male");

   // Call for the return of the 3 values
   std::cout << "Selected Pig  ->" << current_pig.next_pig( ) << '\n';

   dog current_dog; // Create a instance

   // Send 3 values to the object
   current_dog.add_dog("Misty, 15, Female");
   current_dog.add_dog("Tristian, 12, Male");
   current_dog.add_dog("Tempest, 11, Female");

   // Call for the return of the 3 values
   std::cout << "Selected Dog  ->" << current_dog.next_dog( ) << '\n';


   // Now get the results from the friend function
   //  total_animals = patient_count(current_horse); // Works
   total_animals = patient_count(current_horse, current_pig, current_dog); // Nova
   std::cout << "Total Animals: " << total_animals << std::endl;
   return (0);
}
4

4 回答 4

5

You need to define appropriate friend function patient_count with appropriate signatures/declaration as you declare them for each of the classes.

Right now you have one patient_count function defined with signature:

int patient_count(horse target_horse, pig target_pig, dog target_dog);

The friend functions declared in your various classes are with signatures:

dog patient_count(dog);
int patient_count(horse);
pig patient_count(pig);

You need to define functions for each of these signatures. The function you defined with 3 input parameters is pretty much useless because it is not declared are friend of any of your classes.

Note that in C++, functions are overloaded(each function has separate existance) based on:

  • No of Arguments
  • Sequence of Arguments &
  • Type o Arguments

So each of those functions are separate and not the same and need a separate definition.

于 2011-07-01T19:25:09.357 回答
2

嗯......我认为有更简单的方法来处理这个：

class animal { 
    static int count;

    animal() { ++count; }
    ~animal() { --count; }
};

class horse : public animal { 
    // horse stuff
};

class pig : public animal { 
    // pig stuff here
};

class dog : public animal { 
    // dog stuff here
};

int patient_count() { return animal::count; }

除此之外，您的代码似乎有一个相当基本的问题：它（例如）将动物与动物集合混淆。你有很多东西，比如：

dog current_dog; // Create a instance

// Send 3 values to the object
current_dog.add_dog("Misty, 15, Female");
current_dog.add_dog("Tristian, 12, Male");
current_dog.add_dog("Tempest, 11, Female");

这是没有意义的。一只狗应该准确地代表：一只狗。它有一个名字，一个年龄，一种性别，等等。你上面的真的是三只狗，而不是一只。为了代表它们，你应该有一个狗的集合——最好是一个标准的集合std::vector，比如

dog dogs[10];   // an array of 10 dogs (none yet initialized though)

dogs[0] = dog("Misty, 15, female");
dogs[1] = dog("Tristian, 12, male");
dogs[2] = dog("Tempest, 11, female");

猪、牛、马等几乎相同：一个动物对象应该代表一种实际的动物。动物的集合与单个动物不同。但是请注意，上面的评论：10 条狗的数组就是 10 条狗（尽管它们都没有名字、年龄或性别，但我们已经定义了它们，因此它们都正式存在）。这意味着patient_count当您定义数组时，将报告存在 10 只狗，而不管包含有意义数据的数字是多少。这std::vector显然是一种更好的选择。如果您执行以下操作：

std::vector<dog> dogs;

dogs.push_back("Misty, 15, female");
dogs.push_back("Tristian, 12, male");
dogs.push_back("Tempest, 11, female");

此时，您已经创建并存储了 3 只狗，因此如果patient_count此时打印出来，它应该显示 3（代表创建/定义的实际狗）而不是 10（或其他）来表示潜在动物的数量，同时忽略包含有意义数据的数字。

于 2011-07-01T20:37:05.843 回答
1

In your classes, you declared them as friends of the following 3 functions(for the horse, pig, and dog class respectively)

int patient_count(horse);
pig patient_count(pig);
dog patient_count(dog);

None of those functions exist. The only function that exists is this one:

int patient_count(horse,pig,dog);

Replace your friend line in each of your classes with that declaration. You will need to forward declare pig and dog also.

于 2011-07-01T19:26:47.360 回答
0

因为

int patient_count(horse);
pig patient_count(pig);
dog patient_count(dog);

和

int patient_count(horse target_horse, pig target_pig, dog target_dog);

是患者计数的重载。最好的办法是创建一个具有这样的字段 animal_count 的基类动物

/*******************************************************\
* Veternarian Class Problem - I need a class for each   *
* of 3 animals. Horse, Pig and Dogs                     *
\*******************************************************/

#include <cstdlib>
#include <iostream>
#include <string>

const int HORSE_KENNEL = 100; // Number of horses we can store
const int PIG_KENNEL = 100; // Number of Pigs we can store
const int DOG_KENNEL = 100; // Number of Dogs we can store


class animal{
protected:
    int animal_count;
public:
    friend int patient_count(animal);
};



/*******************************************************\
* Class horse                                           *
*                                                       *
* Member functions                                      *
* animal_count -- Keeps track of the number of horses    *
* add_horse -- Sends data into the object               *
* next_horse -- returns data from the object            *
\*******************************************************/
// Definition of the Class
class horse : public animal {

  private:
    std::string horse_data[HORSE_KENNEL]; // A Place to put the data

  // Declarations for the method Prototypes
  public:            
    // Initialize 
    horse::horse( );

    // A Function that accepts an argument but returns nothing
    void add_horse(const std::string new_horse_data);

    // This method returns the next Horse in the queue
    std::string next_horse( );


};


/*******************************************************\
* Method Definition - Here we flush out the prototypes  *
* outlined in the last section                          *
\*******************************************************/
inline horse::horse( )
{
  for(int i = 0; i < HORSE_KENNEL; ++i){
    horse_data[i] = "Empty Spot";
  }
  animal_count = 0; // Zero the data count
}

/*******************************************************\
* horse::add_horse -- Send data to Object               *
\*******************************************************/
inline void horse::add_horse(const std::string new_horse_data)
{
  horse_data[animal_count] = new_horse_data;
  ++animal_count;
}

/*******************************************************\
* horse::next_horse - get data from object              *
\*******************************************************/

inline std::string horse::next_horse( )
{
  // this is specifically implementing a queue
  std::string current_horse = " ";
  int target_horse = 0;
  for(int i = 0;i < HORSE_KENNEL; ++i){
    if(horse_data[i] != "Empty Spot"){
      std::cout << "Horse Number " << i << " " << horse_data[i] << std::endl; 
    }
  }
  std::cout << "Select the horse you want: "; 
  std::cin >> target_horse;

return (horse_data[target_horse]);
}

/*******************************************************\
* Class Pig                                             *
*                                                       *
* Member functions                                      *
* animal_count -- Keeps track of the number of pigs        *
* add_pig -- Sends data into the object                 *
* next_pig -- returns data from the object              *
\*******************************************************/

// Definition of the Class
class pig :public animal{

  private:
     // Variable to keep track of data
    std::string pig_data[PIG_KENNEL]; // A Place to put the data

  // Declarations for the method Prototypes
  public:            
    // Initialize 
    pig::pig( );

    // A Function that accepts an argument but returns nothing
    void add_pig(const std::string new_pig_data);

    // This method returns the next pig in the queue
    std::string next_pig( );
};


/*******************************************************\
* Method Definition - Here we flush out the prototypes  *
* outlined in the last section                          *
\*******************************************************/
inline pig::pig( )
{
  for(int i = 0; i < PIG_KENNEL; ++i){
    pig_data[i] = "Empty Spot";
  }
  animal_count = 0; // Zero the data count
}

/*******************************************************\
* pig::add_pig -- Send data to Object                   *
\*******************************************************/
inline void pig::add_pig(const std::string new_pig_data)
{
  pig_data[animal_count] = new_pig_data;
  ++animal_count;
}

/*******************************************************\
* pig::next_pig - get data from object                  *
\*******************************************************/

inline std::string pig::next_pig( )
{
  // this is specifically implementing a queue
  std::string current_pig = " ";
  int target_pig = 0;
  for(int i = 0;i < PIG_KENNEL; ++i){
    if(pig_data[i] != "Empty Spot"){
      std::cout << "pig Number " << i << " " << pig_data[i] << std::endl; 
    }
  }
  std::cout << "Select the pig you want: "; 
  std::cin >> target_pig;

return (pig_data[target_pig]);
}

/*******************************************************\
* Class dog                                             *
*                                                       *
* Member functions                                      *
* animal_count -- Keeps track of the number of dogs        *
* data_to_object -- Sends data into the object          *
* data_from_object -- returns data from the object      *
\*******************************************************/

// Definition of the Class
class dog :public animal {

  private:
    std::string dog_data[DOG_KENNEL]; // A Place to put the data

  // Declarations for the method Prototypes
  public:            
    // Initialize 
    dog::dog( );

    // A Function that accepts an argument but returns nothing
    void add_dog(const std::string new_dog_data);

    // This method returns the next dog in the queue
    std::string next_dog( );


};


/*******************************************************\
* Method Definition - Here we flush out the prototypes  *
* outlined in the last section                          *
\*******************************************************/
inline dog::dog( )
{
  for(int i = 0; i < DOG_KENNEL; ++i){
    dog_data[i] = "Empty Spot";
  }
  animal_count = 0; // Zero the data count
}

/*******************************************************\
* dog::add_dog -- Send data to Object                   *
\*******************************************************/
inline void dog::add_dog(const std::string new_dog_data)
{
  dog_data[animal_count] = new_dog_data;
  ++animal_count;
}

/*******************************************************\
* dog::next_dog - get data from object                  *
\*******************************************************/

inline std::string dog::next_dog( )
{
  // this is specifically implementing a queue
  std::string current_dog = " ";
  int target_dog = 0;
  for(int i = 0;i < DOG_KENNEL; ++i){
    if(dog_data[i] != "Empty Spot"){
      std::cout << "dog Number " << i << " " << dog_data[i] << std::endl; 
    }
  }
  std::cout << "Select the dog you want: "; 
  std::cin >> target_dog;

return (dog_data[target_dog]);
}

/**************************************************\
* This function is a friend of all the animal      *
* classes and returns the total of all animals     *
* PROBLEM ******* PROBLEM *********PROBLEM *********
* When I add the other 2 classes on the next line  *
* The program stops working                        *
\**************************************************/
// int patient_count(horse target_horse) //works
int patient_count(animal target_animal) // Nova
{
    int all_animals = target_animal.animal_count;
    return (all_animals);
}
int patient_count(horse target_horse, pig target_pig, dog target_dog) // Nova
{
//  int all_animals = target_horse.animal_count; //Works
   int all_animals = patient_count(target_horse) + patient_count(target_pig) + patient_count(target_dog); // Nova

return (all_animals);
}
/**************************************************\
* The Class is defined above, this section is a    *
* Small testing harness to verify that the class   *
* is doing what it was designed to do              *
\**************************************************/
int main( )
{
  int total_animals;
  horse current_horse; // Create a instance

  // Send 3 values to the object
  current_horse.add_horse("Mr Ed, 10, Male");
  current_horse.add_horse("Lightning, 4, Female");
  current_horse.add_horse("Blitz, 7, Male");

  // Call for the return of the 3 values
  std::cout << "Selected Horse  ->" << current_horse.next_horse( ) << '\n';

  pig current_pig; // Create a instance

  // Send 3 values to the object
  current_pig.add_pig("Arnold, 4, Male");
  current_pig.add_pig("Babe, 2, Female");
  current_pig.add_pig("Killer, 7, Male");

  // Call for the return of the 3 values
  std::cout << "Selected Pig  ->" << current_pig.next_pig( ) << '\n';

  dog current_dog; // Create a instance

  // Send 3 values to the object
  current_dog.add_dog("Misty, 15, Female");
  current_dog.add_dog("Tristian, 12, Male");
  current_dog.add_dog("Tempest, 11, Female");

  // Call for the return of the 3 values
  std::cout << "Selected Dog  ->" << current_dog.next_dog( ) << '\n';


  // Now get the results from the friend function
//  total_animals = patient_count(current_horse); // Works
  total_animals = patient_count(current_horse, current_pig, current_dog); // Nova
  std::cout << "Total Animals: " << total_animals << std::endl;
return (0);
}
于 2011-07-01T19:44:20.847 回答