1

book オブジェクトのリストを引数として受け取る関数を書いています。各 book オブジェクトには、プライベート データ メンバーの価格があります。この関数は、各本の価格を比較し、価格が最も高い本を返すことを想定しています。

//Client program
#include <iostream>
#include "Book.h"
#include "textbook.h"
#include "Name.h"
#include "unsorted.h"
using namespace std;

int main()
{

book b1("The Exception to the Rulers", "Amy", "Goodman", "Hyperion", 342, "1-4013-0131", 21.95,'N'); // this is the title, authors first & last name, publisher, number of pages, isbn number, price, and code.
book b2("Who moved my cheese", "Spencer", "Johnson", "Red Tree", 95, "0-399-14446-3", 19.99,  'H');
book b3("Hellbound Hearts", "Neil", "Gaiman", "Dark Harvest", 326, "978-1-4391-4090-1", 16.00, 'F');

UnsortedType L1; // creating a list "L1" with the default vaule lengh 0

L1.InsertItem(b1); // populating the list with the first book
L1.InsertItem(b2); // populating the list with the second book
L1.InsertItem(b3); // populating the list with the third book

主に、実際のリスト「L1」またはL1の内容を価格を比較する関数に渡す方法がよくわかりません。関数 getMostExpensive を呼び出すには、次のようなことを行うため、混乱していると思います。

L1.getMostExpensive();

しかし、L1 で関数を呼び出す場合、引数を渡す必要がありますか? そうでない場合、関数 getMostExpensive() 内のプライベート データ メンバーの価格にアクセスするにはどうすればよいですか?

4

1 に答える 1

0

priceの非公開メンバーである必要があるのはなぜbookですか? 「みんな」が本の価格を知ることができるように思えます...

本当に公開できるのなら、よりシンプルstd::vector<book>で無料の関数を使用してみませんgetMostExpensive()か?

#include <vector>

...

std::vector<book> L1; 

L1.push_back(b1); // include first book
L1.push_back(b2); // include second book
L1.push_back(b3); // include third book

...

// free function
book getMostExpensive(const std::vector<book>& b) {

    double maxPrice=0;
    unsigned int maxInd;
    for(unsigned int i=0; i<b.size(); ++i){
        if (b[i].price > maxPrice){
            maxInd = i;
            maxPrice = b[i].price;
        }
    }
    return b[maxInd];
}

を維持する必要がある場合は、をprice private作成できます。UnsortedTypefriendbook

class UnsortedType;
class book {
    ...
    friend class UnsortedType;
    ...
};

この場合L1、 のプライベートにアクセスできますbook。

ただし、私のデフォルトの経験則は、必要なときはいつでもfriend、設計に欠陥があるということです:p

getter/setter アプローチを使用することもできます。

class book 
{
private:
    double _price; // the actual price

public: 
    const double& price // read-only copy of price

    // constructor
    book(...);

    // price-setter
    void setPrice(double newPrice);

};

// constructor initializes const reference to private member _price
book::book(...) : price(_price) {...}

void book::setPrice(double newPrice) { _price = newPrice>=0.0?newPrice:0.0; }

...

int main(...){
    book b;
    ...
    double P = b.price;  // valid
    b.price = 56.8;      // NOT valid; compile-time error

    b.setPrice(56.8);    // valid; this is the only way to set the price

}
于 2012-11-08T16:58:05.147 に答える