Q&A
質問にある仕様はAccontクラスの仕様としていいと思いますが、この内容と、「実行結果がこうなってほしい」という出力結果の内容が離れすぎています。
これを実現するのであれば、ユーザインタフェイスを実現する部分とAccountクラスは別にするべきだと思います。
この質問の元になった問題のでどころを確認してください。
python
1class Account: 2 3 __date={} 4 5 def __init__(self): 6 7 def register(self,id,pw): 8 id=input("アカウント名を入力->") 9 pw=input("パスワードを入力->") 10 return "登録完了" 11 12 def update(self,id,old_pw,new_pw): 13 id=input("アカウント名を入力->") 14 old_pw=input("旧パスワードを入力->") 15 new_pw=input("新パスワードを入力->") 16 if self.if==id and self.pw==pw: 17 return "更新完了" 18 else: 19 return "パスワードが間違っています" 20 21 def delete(self,id,pw): 22 id=input("アカウント名を入力->") 23 pw=input("パスワードを入力->") 24 if self.id==id and self.pw==pw: 25 return "削除完了" 26 else: 27 return "パスワードが間違っています"
１．辞書形のプライベート変数 __data を作成
・以降，keyがID，valueがPasswordとなるようにする
２．コンストラクタで初期アカウントの作成
・引数なし
・初期アカウント：ID が root Password が root とする
３．アカウントの登録を行うregisterメソッド
・引数は、id, pw
・返り値は、”登録完了”
４．パスワードが合っていればアカウントのパスワードの更新を行うupdateメソッド
・引数は、更新するアカウントのid, 元のパスワードold_pw, 新しいパスワードnew_pw
・返り値は、"更新完了" or "パスワードが間違っています"
５．パスワードが合っていればアカウントを削除するdeleteメソッド
・引数は、id, pw
・返り値は、”削除完了” or “パスワードが間違っています”
実行結果が以下のようになるようなプログラムを作成したいです。
ここからどこをどう書けばよいのか教えてほしいです。
回答1件
0
ベストアンサー
仕様はクラスの定義なので、それに合ったクラスを作るとします。
とすると、質問のコードについては以下のように思います
・ input文は仕様にないのですべて不要
・ 明示していないが、1つのインスタンスがアカウントDBとなるようなので、「__data」をクラス変数にする必要はない。
以下、説明書こうと思ったんですが、 書けそうにないので、コンストラクタとupdateの実装例を示してみます。
python
1class Account: 2 def __init__(self): 3 self.__data = {'root': 'root'} 4 5 def update(self,id,old_pw,new_pw): 6 if id in self.__data.keys(): 7 if self.__data[id] == old_pw: 8 self.__data[id] = new_pw 9 return "更新完了" 10 else: 11 return "パスワードが間違っています" 12 else: 13 pass 14 # 本来idがなかったときの返り値が必要
コンストラクタ(init)で__dataを定義します。書いてあるとおり、ここでデフォルトのアカウントを作ります。
updateは厳密に仕様の通りにするのであれば、一番外側のifは不要です。でも、存在しないIDを指定されたときの動作が不定だったり「パスワードが間違えています」では気持ち悪いので入れてあります。ただし何も返しませんのでどうするかは考えていただければ。
内側のifは、更新するアカウントの 元のパスワードold_pwが合っていれば、新しいパスワードnew_pwに変更する処理です。
残りのメソッドもこれでわかるでしょう。
また、この先、このクラスを利用して ユーザとやりとりする部分のプログラムを作る必要があるかもしれません。
投稿2022/06/23 03:34
総合スコア12763
あなたの回答
tips
プレビュー
