Создание функции, которая является другом для нескольких классов - PullRequest
1 голос
/ 01 июля 2011

В приведенном ниже коде я пытаюсь создать функцию «Patient_Count», которая является другом для классов «лошадь», «свинья» и «собака». Я могу заставить эту функцию быть другом с 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 ]

5 голосов
/ 01 июля 2011

Вам необходимо определить соответствующую функцию друга patient_count с соответствующими сигнатурами / декларацией, поскольку вы объявляете их для каждого из классов.

Сейчас у вас есть одна patient_count функция, определенная с подписью:

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

Функции друзей, объявленные в ваших различных классах, имеют подписи:

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

Вам необходимо определить функции для каждой из этих подписей.Функция, которую вы определили с 3 входными параметрами, в значительной степени бесполезна, потому что она не объявлена ​​как друг любого из ваших классов.

Обратите внимание, что в C ++ функции перегружены (каждая функция имеет отдельное существование) на основании:

  • Количество аргументов
  • Последовательность аргументов &
  • Тип o Аргументы

Таким образом, каждая из этих функций является отдельной, а не одинаковойи нужно отдельное определение.

2 голосов
/ 02 июля 2011

Хм ... Я думаю, что есть лот более простой способ справиться с этим:

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 (или что-либо другое) для представления числа потенциальных животных, при этом игнорируется число, содержащее значимые данные.

1 голос
/ 01 июля 2011

В ваших классах вы объявили их друзьями следующих 3 функций (для класса лошадь, свинья и собака соответственно)

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

Ни одна из этих функций не существует.Единственная существующая функция - эта:

int patient_count(horse,pig,dog);

Замените строку вашего друга в каждом из ваших классов этим объявлением.Вам нужно будет также заранее объявить свинью и собаку.

0 голосов
/ 01 июля 2011

Потому что

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);
}
Добро пожаловать на сайт PullRequest, где вы можете задавать вопросы и получать ответы от других членов сообщества.
...