「代入の左オペランドとして左辺値が必要」というエラーを解決したい

Question

キーボードの入力より「野菜の名前」、「野菜の値段」「消費税込みの値段」を半角スペース区切りで表示するようなプログラムを作成したいと考えています。今のところ「野菜の名前」、「野菜の値段」を表示する処理は書けたのですが、「消費税込みの値段」を表示する処理が書けていません。

#include <cmath>
#include <iostream>
#define number_of_vegetable 4

class Vegetable {
	private:
		std::string name;
		int price;
	
	public:
		void putname(std::string s) {
			name = s;
		}

	public:
		std::string getname() {
			return name;
		}

	public:
		void putprice(int n) {
			price = n;
		}
	
	public:
		int getprice() {
			return price;
	  }
	
	public: 
		void getpricewithtax(int m) {
			std::round(price*1.1) = m;
		}
		
	public:
		int getpricewithtax() {
			return std::round(price*1.1);
		}	
};

int main() {
	Vegetable item[number_of_vegetable];
	
	for (int i = 0; i < number_of_vegetable; i++) {
		std::string s;
		int n;
		
		std::cin >> s;
		std::cin >> n;
		
		item[i].putname(s);
		item[i].putprice(n);
	}
	
	for (int i = 0; i < number_of_vegetable; i++) {
		std::cout << item[i].getname() << " ";
		std::cout << item[i].getprice() << " ";
		std::cout << item[i].getpricewithtax() << std::endl;
	}
};

このようなコードを書いたところ、
std::round(price*1.1) = m;に対して

lvalue required as left operand of assignment（代入の左オペランドとして左辺値が必要）

というエラーが表示されました。
このエラーが表示されなくなるにはどうしたら良いでしょうか？
分かる方がいましたらご教授していただきたく存じます。

Comments

[fana]

> このエラーが表示されなくなるにはどうしたら良いでしょうか？

どうすれば良いのか？　と問うのであれば，
std::round(price*1.1) = m;
という問題の記述で「何をしたいのか？」っていう意図をまず説明すべきでは．
すなわち，
void getpricewithtax(int m)
っていうメソッドってのはそもそも何をするものなの？　っていう．
引数 m が何なのかもわからんし．
（少なくともこのメソッドは提示された main 関数からは使われてない．そもそも要らない物なのであれば，このメソッドを削除すればそれで終了だと思うけど．）

[退会済みユーザー]

消費税を含めた値段を四捨五入して表示する処理を書きたいと考えています。
書き方がわからないためとりあえず
public:
void putprice(int n) {
price = n;
}

public:
int getprice() {
return price;
}
を真似して書きました。

[BeatStar]

とりあえず基礎からやった方がいいです。
今の質問では「空を飛ぼうと腕をパタパタと動かしてみましたがなぜか飛びません。鳥はこれで飛ぶのに」みたいな頓珍漢な質問です。
「人間は鳥と違って空は飛ばない」ことと「飛ぶには浮力と前進することが必要」といった基礎知識が必要です。
飛行機を作ったライト兄弟らのような人達はでたらめにやっているのではなく、既存の理論や関連知識から理論展開しています。

それを「基礎やるのだりー」とかでやらないのならどんな説明をしても無駄です。まずは基礎からやりましょう。

なにごとも基礎の積み重ねです。

[BeatStar]

今回のは基礎ができていないことが原因です。

[ikadzuchi]

「代入の左オペランドとして必要な左辺値」は誤訳しています。正しくは「代入の左オペランドとして左辺値が必要」です。

Answer 1

#include <cmath>
#include <iostream>
#define number_of_vegetable 4

class Vegetable {
private:
  std::string name;
  int price;
public:
  void putname(std::string s) { name = s; }
  std::string getname() const { return name; }
  void putprice(int n) { price = n; }
  int getprice() const { return price; }
  int getpricewithtax() const {  return std::round(price*1.1); }
};

int main() {
  Vegetable item[number_of_vegetable];

  for (int i = 0; i < number_of_vegetable; i++) {
    std::string s;
    int n;
    std::cin >> s >> n;
    item[i].putname(s);
    item[i].putprice(n);
  }

  for ( const Vegetable& veg : item ) {
    std::cout << veg.getname() << " "
              << veg.getprice() << " "
              << veg.getpricewithtax() << std::endl;
  }
}

Answer 2

getpricewithtaxメソッドは戻り値がvoidなんでそういうエラー出てますね

---

まったく質問が変わってるやんｗ
ってことで、

std::round(price*1.1) = m;に対して

これ、明らかに間違ってますが、
この文ではなにをするのか説明できるでしょうか

Comments

[退会済みユーザー]

消費税を含めた値段を四捨五入して表示する処理を書きたいと考えています。

[退会済みユーザー]

どうすれば消費税を含めた値段を四捨五入して表示できるかがわかりません。

[y_waiwai]

この行では表示はしてませんよ

Answer 3

このような方法でもできました。

#include <iostream>
#define number_of_vegetable 4

class Vegetable {
	private:
		std::string name;
		int price;
	
	public:
		void putname(std::string s) {
			name = s;
		}

	public:
		std::string getname() {
			return name;
		}

	public:
		void putprice(int n) {
			price = n;
		}
	
	public:
		int getprice() {
			return price;
	  }
	
	public: 
		void putpricewithtax() {
			std::round(price*1.1);
		}
		
	public:
		int getpricewithtax() {
			return std::round(price*1.1);
		}	
};

int main() {
	Vegetable item[number_of_vegetable];
	
	for (int i = 0; i < number_of_vegetable; i++) {
		std::string s;
		int n;
		
		std::cin >> s;
		std::cin >> n;
		
		item[i].putname(s);
		item[i].putprice(n);
	}
	
	for (int i = 0; i < number_of_vegetable; i++) {
		std::cout << item[i].getname() << " ";
		std::cout << item[i].getprice() << " ";
		std::cout << item[i].getpricewithtax() << std::endl;
	}
};

Comments

[BeatStar]

確かにエラーメッセージは出ないでしょうけど、仕様には沿ってないと思いますよ。
そもそもstd::round(...)とは何者でしょうか？
プログラミングって単にエラーメッセージが出なければいいというものではありません。
ちゃんとアプリとかとして使えなければ話になりません。

[退会済みユーザー]

貴重なアドバイスありがとうございます。以後気をつけます。

[fana]

> putpricewithtax()

いやだからこのメソッド自体が丸ごと要らないんじゃないの？
使わないし意味のある機能も持たないメソッドを書いとく意義は何なの？