質問をすることでしか得られない、回答やアドバイスがある。

15分調べてもわからないことは、質問しよう!

新規登録して質問してみよう
ただいま回答率
85.48%
C#

C#はマルチパラダイムプログラミング言語の1つで、命令形・宣言型・関数型・ジェネリック型・コンポーネント指向・オブジェクティブ指向のプログラミング開発すべてに対応しています。

Q&A

解決済

1回答

1980閲覧

なぜ、クラスの変数定義はprivateでもよいのに、メソッドだとエラーが出るのですか?

toeic

総合スコア7

C#

C#はマルチパラダイムプログラミング言語の1つで、命令形・宣言型・関数型・ジェネリック型・コンポーネント指向・オブジェクティブ指向のプログラミング開発すべてに対応しています。

0グッド

0クリップ

投稿2020/04/11 09:34

理解したいこと

1円玉~500円玉をカウントするプログラムを組んでいます。
それを組む中で、クラスのメソッドをprivateにしたら、エラーが発生しました。publicにするとエラーが消えます。
privateだと、インスタンス化したら使えないのか?と思ったのですが、
その前に変数を定義する際は、privateなのに、エラーが発生しません。
この差は何なのでしょうか?
ソースコードの16行目「private void AddCoins(int kind, int count)」でエラーが出ます。
9行目の「private int yen500 = 0;」では、エラーが出ません。

発生している問題・エラーメッセージ

エラー CS0122 'CoinCase.AddCoins(int, int)' はアクセスできない保護レベルになっています (78行目)

該当のソースコード

C#

1using System; 2using System.Collections.Generic; 3using System.Text; 4 5namespace ConsoleApp1 6{ 7 class CoinCase 8 { 9 private int yen500 = 0; 10 private int yen100 = 0; 11 private int yen50 = 0; 12 private int yen10 = 0; 13 private int yen5 = 0; 14 private int yen1 = 0; 15 16 private void AddCoins(int kind, int count) 17 { 18 switch (kind) 19 { 20 case 500: 21 yen500 += count; 22 break; 23 case 100: 24 yen100 += count; 25 break; 26 case 50: 27 yen50 += count; 28 break; 29 case 10: 30 yen10 += count; 31 break; 32 case 5: 33 yen5 += count; 34 break; 35 case 1: 36 yen1 += count; 37 break; 38 } 39 } 40 public int GetCount(int kind) 41 { 42 switch (kind) 43 { 44 case 500: 45 return yen500; 46 case 100: 47 return yen100; 48 case 50: 49 return yen50; 50 case 10: 51 return yen10; 52 case 5: 53 return yen5; 54 case 1: 55 return yen1; 56 } 57 return 0; 58 } 59 public int GetAmount() 60 { 61 return ((yen500 * 500) + (yen100 * 100) + (yen50 * 50) + (yen10 * 10) + (yen5 * 5) + (yen1 * 1)); 62 } 63 } 64 65 class Excute 66 { 67 static void Main() 68 { 69 CoinCase task1 = new CoinCase(); 70 for (int i = 1; i <= 10; i++) 71 { 72 Console.WriteLine("{0}回目",i); 73 Console.WriteLine("コインの種類は?"); 74 75 int kind = int.Parse(Console.ReadLine()); 76 Console.WriteLine("枚数は?"); 77 int count = int.Parse(Console.ReadLine()); 78 task1.AddCoins(kind, count); 79 } 80 Console.WriteLine("500円は{0}枚", task1.GetCount(500)); 81 Console.WriteLine("100円は{0}枚", task1.GetCount(100)); 82 Console.WriteLine("50円は{0}枚", task1.GetCount(50)); 83 Console.WriteLine("10円は{0}枚", task1.GetCount(10)); 84 Console.WriteLine("5円は{0}枚", task1.GetCount(5)); 85 Console.WriteLine("1円は{0}枚", task1.GetCount(1)); 86 Console.WriteLine("総額" + task1.GetAmount()); 87 } 88 } 89}

気になる質問をクリップする

クリップした質問は、後からいつでもMYページで確認できます。

またクリップした質問に回答があった際、通知やメールを受け取ることができます。

バッドをするには、ログインかつ

こちらの条件を満たす必要があります。

guest

回答1

0

ベストアンサー

publicはクラス外から呼べますが、privateは呼べません。
これは関数でも変数でも同じです。
(他にもprotectedというものもありますが、詳細は自分で調べてください)

投稿2020/04/11 09:38

fiveHundred

総合スコア9803

バッドをするには、ログインかつ

こちらの条件を満たす必要があります。

toeic

2020/04/11 10:32

AddCoinsメソッドは、作成したインスタンスから呼ぶのでpublicなのは分かりました。 その前の変数定義 private int yen500 = 0; がprivateでよいのは、同じCoinCaseクラス内で使用されているからとの理解でよいでしょうか? 'yen500'もAddCoinsメソッドを通じて使われるので、publicで定義する必要があるのかと思ってましたがそのようなものでしょうか?
fiveHundred

2020/04/11 11:06

その認識であっております。 むしろ、オブジェクト指向にあるカプセル化という概念から、'yen500'のようなものはprivateにすることのほうが多いですね。
toeic

2020/04/11 11:12

なるほど、勉強になりました。 ありがとうございます!
guest

あなたの回答

tips

太字

斜体

打ち消し線

見出し

引用テキストの挿入

コードの挿入

リンクの挿入

リストの挿入

番号リストの挿入

表の挿入

水平線の挿入

プレビュー

15分調べてもわからないことは
teratailで質問しよう!

ただいまの回答率
85.48%

質問をまとめることで
思考を整理して素早く解決

テンプレート機能で
簡単に質問をまとめる

質問する

関連した質問