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

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

ただいまの
回答率

90.50%

  • C#

    9059questions

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

  • WPF

    827questions

    Windows Presentation Foundation (WPF) は、魅力的な外観のユーザー エクスペリエンスを持つ Windows クライアント アプリケーションを作成するための次世代プレゼンテーション システムです

Combobox内の選択アイテムと、リスト内のelementの一致、という条件設定の仕方

解決済

回答 1

投稿 編集

  • 評価
  • クリップ 0
  • VIEW 242

mgmk

score 9

コードの細かな書き方について質問です。
上記List<Transaction>transactionsがあります。

transactions内の、あるひとつのelement(例えばtransactionという変数でくくる)の中に、account.ToString()を見つけ出したいです。

質問1.var transaction などの変数をつくり、account.ToString()を抽出したいです。(使用メソッドは、Containsでいいのでしょうか。それとも、FirstOrDefaultでしょうか。)

その後、

  1. コンボボックス(ドロップダウンメニューのようなもの)の選択されたitem
  2. 前述のtransactions内にある任意の内のelement内のaccount.ToString()
    1と2が同じである時(if(1==2){})、という条件設定が必要なのですが、どういう風に書けるでしょうか?

そこで質問2ですが、下記コードの日本語部分に該当するコードの書き方が知りたいです。

前回質問がコードの方向性的なざっくりした内容だったのですが、こちらはのちの具体的部分になるかと思います。まだ、場数をこなしていないため、コードの細かな書き方がわかりにくいです。よろしくお願いします。

if (Combobox1.SelectedItem.ToString().Equals(transactionsに含まれるaccount.ToString()を表すためのコード))
{
  //リスト表示
  Listbox1.Items.Add(transactionsのパラメーターであるdate, transactionType,depAmountを表示させたいです。);
}


簡単に言ってしまうと、上記if文のカッコ内は、コンボボックスの選択中のアイテム(口座タイプ、アカウントナンバー)と、ローカル変数であるaccount.ToString()(口座タイプ、アカウントナンバー)の照合です。

しかし、もっと言うと、アカウントナンバー自体は、BankAccountという親クラスにプロパティがあります。
var account = Combobox1.SelectedItem as BankAccount;このようにして、asオペレーターでBankAccountにあるプロパティのようにふるまわせています。
Transactionクラスはあくまで以下のプロパティしかもっておらず、BankAccountクラスとはアソシエーションの関係にあります…。

よって、BankAccount内のAccountNumberプロパティを直接照合させて使うこともできるはずなのですが(そのほうがややこしくなさそうです)、object.AccountNumberとするためには、オブジェクトが必要になりそうです。

namespace practice
{
    [Serializable]
    class Transaction
    {
        internal bool AccountNumber;
 public double Amount { get; set; }
        public DateTime Date { get; set; }
        public string TransactionType { get; set; }
        public List<BankAccount> BankAccounts { get; private set; } = new List<BankAccount>();


        // Constructors
        public Transaction(DateTime date, string transactionType, double amount, string s)
        {
            Amount = amount;
            Date = DateTime.Now;
            TransactionType =transactionType ;

        }

        // Methods
        public override string ToString()
        {
            return $"\n{Date} {TransactionType}: {Amount}SEK ";
        }


クラス図のクラス名は、コード上では、Accout→BankAccount,History→Transactionとなっています。

イメージ説明

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

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

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

    クリップを取り消します

  • 良い質問の評価を上げる

    以下のような質問は評価を上げましょう

    • 質問内容が明確
    • 自分も答えを知りたい
    • 質問者以外のユーザにも役立つ

    評価が高い質問は、TOPページの「注目」タブのフィードに表示されやすくなります。

    質問の評価を上げたことを取り消します

  • 評価を下げられる数の上限に達しました

    評価を下げることができません

    • 1日5回まで評価を下げられます
    • 1日に1ユーザに対して2回まで評価を下げられます

    質問の評価を下げる

    teratailでは下記のような質問を「具体的に困っていることがない質問」、「サイトポリシーに違反する質問」と定義し、推奨していません。

    • プログラミングに関係のない質問
    • やってほしいことだけを記載した丸投げの質問
    • 問題・課題が含まれていない質問
    • 意図的に内容が抹消された質問
    • 広告と受け取られるような投稿

    評価が下がると、TOPページの「アクティブ」「注目」タブのフィードに表示されにくくなります。

    質問の評価を下げたことを取り消します

    この機能は開放されていません

    評価を下げる条件を満たしてません

    評価を下げる理由を選択してください

    詳細な説明はこちら

    上記に当てはまらず、質問内容が明確になっていない質問には「情報の追加・修正依頼」機能からコメントをしてください。

    質問の評価を下げる機能の利用条件

    この機能を利用するためには、以下の事項を行う必要があります。

質問への追記・修正、ベストアンサー選択の依頼

  • YAmaGNZ

    2019/01/14 18:24

    前回の質問のクラス図と言われても、TransactionクラスもBankAccountクラスも書いてありませんが・・・

    キャンセル

  • mgmk

    2019/01/14 18:29

    UMLのクラスダイアグラムの画像なのですが。見られないでしょうか。

    キャンセル

  • mgmk

    2019/01/14 18:33

    ちなみにUMLのクラス名ですが、
    Accountクラス=BankAccoutクラス
    Historyクラス = Transactionクラス
    です。

    キャンセル

回答 1

checkベストアンサー

0

現状のTransactionクラスでは質問にある「account.ToString()」が保持されるような作りになっていません。
コンストラクタでstring sと指定していますが、このsは使用されていません。
ですので、照合しようと思っても不可能です。
まずは、TransactionクラスにてAccountプロパティなり、コンストラクタで指定されているsを保持しクラス外で参照できるようにしてください。

質問内容からずれていますね。

// リストにある条件をみたす最初のアイテム
var item = transactions.First(a => a.該当プロパティ == "条件");
// リストにある条件をみたす複数のアイテム
var items = transactions.Where(a => a.該当プロパティ== "条件");


とやればリストから抽出できるかと思います。
現状ですと、該当プロパティが不明のため、その部分を実装しないとダメですが・・・

投稿

編集

  • 回答の評価を上げる

    以下のような回答は評価を上げましょう

    • 正しい回答
    • わかりやすい回答
    • ためになる回答

    評価が高い回答ほどページの上位に表示されます。

  • 回答の評価を下げる

    下記のような回答は推奨されていません。

    • 間違っている回答
    • 質問の回答になっていない投稿
    • スパムや攻撃的な表現を用いた投稿

    評価を下げる際はその理由を明確に伝え、適切な回答に修正してもらいましょう。

  • 2019/01/15 01:09 編集

    ありがとうございます。string sの件ですが、まさにaccount.toString();をメインプログラムで使うために後付けしたパラメーターでして…。単純にtransactionの該当リストに直接アクセスする方法があればよいのですが、Transactionクラス外にある口座番号情報ないことには該当情報にアクセスができず…。設問ミスのようなものなのでしょうかね。

    キャンセル

  • 2019/01/15 07:18

    Transactionクラスは履歴のようですから、このクラスで保持する情報は
    ・取引日時
    ・取引種別
    ・取引金額
    ・取引者
    ・取引相手
    だと思われます。ソースを見ると、「取引者」と「取引相手」の実装が不明です。(ここに書いた項目が必要なのかも私には分かりませんが)
    このあたりを整理して考えてください。

    キャンセル

  • 2019/01/15 20:15

    ありがとうございます。そうですね。Transactionクラスのプロパティをリスト化して、預金と引き出しのイベントごとに保存するようにしているのですが、そこに他クラスにある口座番号が必要になると思われます。しかし他クラスにある口座番号AccountNumberの使い方がわからず…。ローカル変数をつくり、リストに口座番号要素を一時的に追加して、どのtransactionのことなのかを特定するということなのでしょうか。混乱してきます。

    キャンセル

  • 2019/01/15 20:38

    Transactionクラスのプロパティをリスト化して複数の取引の事象を持つのではなく、Transactionクラスは1回の取引情報とすべきかと思います。
    それで、イベント毎にTransactionクラスを生成し、List<Transaction>に追加していくのではないでしょうか。

    キャンセル

同じタグがついた質問を見る

  • C#

    9059questions

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

  • WPF

    827questions

    Windows Presentation Foundation (WPF) は、魅力的な外観のユーザー エクスペリエンスを持つ Windows クライアント アプリケーションを作成するための次世代プレゼンテーション システムです