ラジオを保持する倉庫のクラスを作成する

Question

"ラジオを格納する倉庫、 テレビ、およびコンピュータ用のクラスを作成します。項目なしの 倉庫を開始するコンストラクタを指定します。倉庫に各項目の数量を格納するインスタンス変数を含めます。各項目の 在庫に追加する方法と、倉庫の内容を表示する方法を含めます。 Mainメソッドでテストし、2つのウェアハウスを作成します。それぞれ 表示各倉庫の最終内容に項目を追加します。」

using System; 
public class warehouse 
{ 
    int radios, televisions, computers; 

    public warehouse() 
    { 
     radios = televisions = computers = 0; 
    } 

    public void addRadioStock() 
    { 
     Console.WriteLine("Please enter the number of radios in stock."); 
     string radiosInput = Console.ReadLine(); 
     Convert.ToInt16(radiosInput); 
    } 

    public void addTelevisionsStock() 
    { 
     Console.WriteLine("Please enter the number of televisions in stock."); 
     string televisionsInput = Console.ReadLine(); 
     Convert.ToInt16(televisionsInput); 
    } 

    public void addComputersStock() 
    { 
     Console.WriteLine("Please enter the number of computers in stock."); 
     string computersInput = Console.ReadLine(); 
     Convert.ToInt16(computersInput); 
    } 

    public void Display() 
    { 

     Console.WriteLine(" There are " + radios + " radios in stock."); 
     Console.WriteLine(" There are " + televisions + " televisions in stock."); 
     Console.WriteLine(" There are " + computers + " computers in stock."); 
     Console.WriteLine(); 
    } 

    public static void Main() 
    { 
     warehouse warehouse1 = new warehouse(); 
     warehouse warehouse2 = new warehouse(); 
     warehouse1.addRadioStock(); 
     warehouse1.addTelevisionsStock(); 
     warehouse1.addComputersStock(); 
     warehouse2.addRadioStock(); 
     warehouse2.addTelevisionsStock(); 
     warehouse2.addComputersStock(); 
     Console.WriteLine("Warehouse 1:"); 
     warehouse1.Display(); 
     Console.WriteLine("Warehouse 2:"); 
     warehouse2.Display(); 
    } 
} 

私はあなたの機能の

Answer 1

非をやって何が間違って出力を取得することができません、intに変換すると、値を代入した後、クラスのメンバーに送ってください。例えば

変更この：これに

public void addRadioStock() 
{ 
    Console.WriteLine("Please enter the number of radios in stock."); 
    string radiosInput = Console.ReadLine(); 
    Convert.ToInt16(radiosInput); 
} 

：

また

public void addRadioStock() 
{ 
    Console.WriteLine("Please enter the number of radios in stock."); 
    string radiosInput = Console.ReadLine(); 
    radios = Convert.ToInt16(radiosInput); 
} 

私はrecomment：C＃でint.TryParse

チェックnaming conventionsの使用で

1. ルック。そして、もっと全体的な外観のためにthis

Answer 2

あなたの問題はここにある：

Convert.ToInt16(computersInput); 

Convert.ToInt16は、入力を解析し、番号を返します。いずれの場合も、返された番号は何にも割り当てられません。