while loopの問題について

Question

cnt = 0
g = 0
grade = []
while(1):
    cnt += 1
    g = int(input('Enter grade '+str(cnt) + ': '))
    if g==-1:
        break
    else:
        grade.append
        



avg = sum(grade)/len(grade)
print('Average :' + str(avg))


if 90 <= avg:
    grade = 'A++'
elif 80 <= avg:
    grade = 'A+'
elif 70 <= avg:
    grade = 'A'
elif 60 <= avg:
    grade = 'B'
elif 50 <= avg:
    grade = 'C'
elif 40 <= avg:
    grade = 'D'
elif 33 <= avg:
    grade = 'F+'
elif 20 <= avg:
    grade = 'F'
else:
    grade = 'FF'
print ('Your grade is:{} '.format(avg))

この問題をうまく作りたいのですがどこを変えたらいいでしょうか？？
エラーでますよとか実行すればわかるこというのやめてください。
単純に解決策を探しているのでわからない方は回答しないでください。

Comments

[miyabi_pudding]

エラーがでているのでしょうか？
エラー文も記載お願いします。

[退会済みユーザー]

Enter grade 1: 80
Enter grade 2: 80
Enter grade 3: 9
Enter grade 4: -1
Traceback (most recent call last):
File "/Users/aya/Documents/new practice file.py", line 766, in <module>
avg = sum(grade)/len(grade)
ZeroDivisionError: division by zero
>>>

とこのような形になります。本来はマイナスのあたいを入力したのちそれまでの平均値を出し、その平均値がどのグレードに当てはまるのかを示さなければならないのですが。

よろしくお願いいたします。

[miyabi_pudding]

エラーを翻訳にかけると、

> トレースバック（最後の最後の呼び出し）：
<module>内のファイル "/ Users / aya / Documents / new Practice file.py"、行766
avg = sum（grade）/ len（grade）
ZeroDivisionError：ゼロによる除算

となります。
つまり、0に対して除算しちゃってるからエラーだよ
と言われています。

while文内の、
grade.appendは、中身を何も追加していないようですが、
どんな値を入れるのでしょうか？
gradeリストに何も入っていないため、
len(grade)が0となり、その行の計算が失敗しているのかと。

[退会済みユーザー]

エラーって翻訳にかけられるんですか！？初めて知りました。
グレードのところは下の部分にあるif 文をいれるつもりで書いてみたのですが、違うのですね。ご丁寧にありがとうございます。再度書いてみます。

[miyabi_pudding]

Google翻訳です。
日本語に直すと、だいたいのエラーはよりわかりやすくなります。

if文を直接入れることはできません。
appendメソッドは、
リスト変数に対して、値を追加するメソッドです。
list.append(なんらかの値)
というふうに使います。
つまり、()で実行していない時点で、間違った使い方をしているということです。

[退会済みユーザー]

なるほど。それではどのようにアプローチするのがいいのでしょうか？
今ある自分の知識ではまったくおもいつきません。。。。
.appendをなくした方がいいのでしょうか？それともbreak らへんが違うのでしょうか？

[miyabi_pudding]

どのようなアプローチがいいかなんて、第三者の私にはわかりません。
なぜなら、どのような実装をしたいか、が質問本文に記載されていないからです。
要件次第では、全然違う構文の方がいい可能性すらあります。

要件を定義し、質問本文に記載して下さい。

Answer 1

エラーでますよとか実行すればわかるこというのやめてください。

ならまず実行してみましょう。なにがおきますか？

Comments

[退会済みユーザー]

エラーが起きますｗ爆笑

[y_waiwai]

エラーが起きるなら、そのエラーメッセージを読みましょう
そして、どういうエラーなのかを理解することです

で、それについて他人に聞きたいのであれば、あなたがやったこと、そしてそこで何が起きたかをすべて提示しましょう。

[退会済みユーザー]

エラーメッセージをよんでわかるほどパイソンをまだやっていないのでわかりません。できればでいいのですが私の質問にあなたに回答して欲しくないです。初心者ですが自分なりに頑張ろうとしてるので、あなたにとって当たり前でも私にとってはそうではありません。では

[y_waiwai]

残念ながら、あなたがどう思おうと、回答したいところに回答していきます
それが嫌ならあなたのほうが退場しましょう

[退会済みユーザー]

あ、はい。

Answer 2

※構文を見てみての、おそらくこういうことがしたいのだろう、というエスパー回答なので、ご了承ください。

今回の実装では、回数制限を設けない、while構文は不適切でしょう。

なぜなら、数値ではない値、もしくは-1が入力されない限り、
ループが止まらないからです。
また、gradeリスト変数に、値が入っているかも確認する必要があります。
（値が入っていないため、エラーが起きる）
回数を決めた上で、grade変数の値があるかも確認すると、下記のようになるでしょう。

# cnt = 0 # 以降で使わない、ループ内の定義で十分なため、ここでの定義は必要ない
# g = 0 # 上記と同様

grade = []

# 今回の実装は回数を決めた方がいい
for i in range(20):
  cnt = i + 1
  g = int(input('Enter grade ' + str(cnt) + ': '))
  if g == -1:
    break
  else:
    # gradeに、gを入れてください
    grade.append(g)

if len(grade) > 0:
  avg = sum(grade) / len(grade)
  print('Average :' + str(avg))

  result = ""
  if 90 <= avg:
    # 変数を壊す代入は避けた方がいい
    # grade = 'A++'
    result = 'A++'
  elif 80 <= avg:
    result = 'A+'
  elif 70 <= avg:
    result = 'A'
  elif 60 <= avg:
    result = 'B'
  elif 50 <= avg:
    result = 'C'
  elif 40 <= avg:
    result = 'D'
  elif 33 <= avg:
    result = 'F+'
  elif 20 <= avg:
    result = 'F'
  else:
    result = 'FF'
  
  print ('Your grade is:{} '.format(avg))
  
else:
  # 値が入っていないので、そう表示
  print("None Values")

Comments

[退会済みユーザー]

まるでエスパーですね！！ありがとうございます。
とりあえずfor分でもう一度復習してみてそのあとwhileにチャレンジしてみようと思います！ありがとうございました。

[miyabi_pudding]

whileの使い所、を勉強されるとよいかと。
whileは基本的に、”何回処理するかわからない”って時に用います。
今回のものでは、回数が決まってないと破綻する要件だとお見受けします。
なので、forで処理するのが適切です。

[miyabi_pudding]

あとは、「要件定義」をできるようになってください。
どのような実装をしたいのか、という明確なものを自身で定義していくことです。
過去質問を拝見いたしましたが、
どうにも、要件定義がされていない、自分で作りたいものの質問をしているはずなのに、
人にその仕様を考えることも要求している、といった質問が目立ちましたので。
今回の場合は、コメントの最後で「どのようなアプローチがいいか」と聞かれたのが該当します。
そこは、第三者でどうにかできる部分ではありません。

[退会済みユーザー]

なるほど、この問題がwhileを学ぶところのセクションにあったのでwhileで解こうと思っていたのですがそうとも限らないようですね。どちらを使うべきか判断できるようになれるようがんばります。
ありがとうございます。

[miyabi_pudding]

> この問題がwhileを学ぶところのセクションにあったので

それこそ、第三者が知り得ない事情なので・・・。
問題文なども総合的にみないと判断がつきません。
とかく、情報は、なるたけ全て質問本文に記載するようにされるとよいかと。
そうしていけば、自ずと、質問への低評価も減るかと思います。
ただし、問題や課題の場合は、そうだと書いてしまうと、
よっぽど突き詰めた質問でないと、（やってみたがどうしてもわからない、って突き詰めないと）
「課題とかなら自分でやれよ」と突っぱねられます。teratailだと。

[退会済みユーザー]

なるほど。今日始めたばかりであまりよくわからず。。。。どこまでが質問の丸投げなのか。。。とか
ある程度自分でもコードを書いているのにまるなげになるんですねｗ
学びますｗｗｗ
無料なので民度が低いのは仕方ないですよね。
今日は自分のいつものメンターさんが忙しくこちらを初めてみたのです（＾◇＾）
miyabiさんの丁寧な回答に救われました。
ありがとうございます。

[miyabi_pudding]

どこまでも、原因は自分にあるのでは？と真摯に受け止める方がよいかと。
民度が低いのではなく、
単純に「あくまで赤の他人にできる範囲」と線引きをしているだけだと思いますよ。みなさん。
最新の質問二つは、要件定義や、自分はここまでやって、わかって、どこがわからないかを、明示しており、
回答つきやすくなっているとお見受けします。
質問の仕方いかんで、人の反応は変わっていくと思います。
どうか頑張って。