pyhon attribute error の原因を探っています

Question

前提・実現したいこと

初心者です。
pythonでtrurtle をimportしピンポンゲームを作っています。

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

Traceback (most recent call last):
  File "C:\Users\1200728\Desktop\test\test.py", line 29, in <module>
    ball.speed(0)
  File "C:\Program Files\WindowsApps\PythonSoftwareFoundation.Python.3.9_3.9.2032.0_x64__qbz5n2kfra8p0\lib\turtle.py", line 2168, in speed
    return self._speed
AttributeError: 'int' object has no attribute '_speed'

### 該当のソースコード
import turtle as t
playerAscore=0
playerBscore=0
window=t.Screen()
window.title=("pong game")
window.bgcolor("green")
window.setup(width=800,height=600)
window.tracer(0)


leftpaddle=t.Turtle()
leftpaddle.speed(0)
leftpaddle.shape("square")
leftpaddle.color("white")
leftpaddle.shapesize(stretch_wid=5,stretch_len=1)
leftpaddle.penup()
leftpaddle.goto(-350,0)

rightpaddle=t.Turtle()
rightpaddle.speed(0)
rightpaddle.shape("square")
rightpaddle.color("white")
rightpaddle.shapesize(stretch_wid=5,stretch_len=1)
rightpaddle.penup()
rightpaddle.goto(-350,0)


ball=t.Turtle
ball.speed(0)
ball.shape("circle")
ball.color("red")
ball.penup()
ball.goto(5.5)
ballxdirection=0.2
ballydirection=0.2


pen=t.Turtle
pen.speed(0)
pen.color("Blue")
pen.penup()
pen.hideturtle()
pen.goto(0,260)
pen.write("score",align="center",font=('Arial',24,'Normal' ))


def leftpaddleup():
    y=leftpaddle.ycor()
    y=y+90
    leftpaddle.sety(y)

def leftpaddledown():
    y=leftpaddle.ycor()
    y=y-90
    leftpaddle.sety(y)

def rightpaddleup():
    y=rightpaddle.ycor()
    y=y+90
    rightpaddle.sety(y)

def rightpaddledown():
    y=rightpaddle.ycor()
    y=y-90
    rightpaddle.sety(y)


window.listen()
window.onkeypress(leftpaddleup,'w')
window.onkeypress(leftpaddledown,'s')
window.onkeypress(rightpaddleup,'Up')
window.onkeypress(leftpaddledown,'Down')

while True:
    window.update()


    ball.setx(ball.xcor(+ballxdirection))
    ball.sety(ball.ycor(+ballydirection))

    if ball.ycor()>290:
        ball.sety(290)
        ballydirection=ballydirection*-1
    if ball.ycor()<-290:
        ball.sety(-290)
        ballydirection=ballydirection*-1

    if (ball.xcor())<-390:
        ball.goto(0,0)
        ballxdirection=ballxdirection*-1
        playerAscore=playerBscore+1
        pen.clear()
        pen.write("player A:{}  player B:{}".format(playerAscore,playerBscore),align='center',font=('Arial',24,'normal'))


    if (ball.xcor()>340)and(ball.xcor()<350)and(ball.ycor()<rightpaddle.ycor()+40 and ball.ycor()>rightpaddle.ycor()-40):
        ball.setx(340)
        ballxdirection=ballxdirection*-1


    if (ball.xcor()<-340)and(ball.xcor()<-350)and(ball.ycor()<leftpaddle.ycor()+40 and ball.ycor()>leftpaddle.ycor()-40):
        ball.setx(340)
        ballxdirection=ballxdirection*-1

試したこと

その前に出ていたsyntax errorはすべて修正しました。
AttributeErrorについてたくさん調べたのですがよくわかりません。
親切な方、ご回答お願い致しますm(__)m

補足情報（FW/ツールのバージョンなど）

python version 3.9

Comments

[meg_]

・コードは「コードの挿入｣で記入してください。
・タイトルに初心者と書くよりも初心者マークを着けた方が良いかと思います。

[rawr]

ありがとうございます。修正しました。

[1T2R3M4]

ball,penと
leftpaddle,rightpaddleのソースコード上の違いは把握されてますか。

[Daregada]

> ありがとうございます。修正しました。
コード部分の修正に失敗していますね。
1. コードを載せたい位置に空行(何も書いていない行)を作り、そこをクリックしてから「<code>」ボタンを押す。
2. 「ここに言語を入力」の文字を全て消して、先頭の「```」の直後に「Python」と書く→挿入部分の先頭行は「```Python」となる。
3. 「コード」の文字を全て消して、あなたがコードを書いているアプリ(VSCodeなど)から、コードを全てコピーし、貼り付ける。
4. 貼り付けたコードの末尾に、「```」だけの行があることを確認する。もし、コードの末尾の行と「```」が1行に繋がっていたら、間をクリックしてEnterキーで分離する→挿入部分の末尾行は「```」になる。
5. 下に表示されるプレビューで表示を確認する。四角い灰色の領域の中にコードだけが表示されます。

[y_waiwai]

このままではコードが読めないので、質問を編集し、<code>ボタンを押し、出てくる’’’の枠の中にコードを貼り付けてください

[rawr]

<code>ボタンの存在を知りませんでした(>_<)教えてくださってありがとうございます！

[rawr]

>>ball,penと
leftpaddle,rightpaddleのソースコード上の違いは把握されてますか。

えっとそれぞれの機能を表していることはわかります。
詳しいことは把握しているかどうか。。。

Answer 1

質問文に現在(2021-09-06 22:00:00)提示されているソースコードを元に、一応ゲームっぽく動く状態したソースコードとの差分を示します。

なお、先頭の「-」は修正前(削除する行)、「+」は修正後(追加する行)を意味します。「+」や「-」は実際には付きません。

右パドルの初期位置

-rightpaddle.goto(-350, 0)
+rightpaddle.goto(350, 0)

rightpaddleの位置が画面左側(leftpaddleと同じ)「-350」になっています。中央を挟んで画面右側の「350」に修正します。

なお、leftpaddle.get_shapepoly()とleftpaddle.get_shapepoly()で得られるリストを見ると分かりますが、パドルはどちらも幅20ピクセル、高さ100ピクセルです。

ボールとペンのオブジェクト

-ball = t.Turtle
+ball = t.Turtle()
 ball.speed(0)
 ball.shape("circle")
 ball.color("red")
 ball.penup()
-ball.goto(5.5)
+ball.goto(0, 0)
 ballxdirection = 0.2
 ballydirection = 0.2


-pen = t.Turtle
+pen = t.Turtle()
 pen.speed(0)
 pen.color("Blue")
 pen.penup()
 pen.hideturtle()
 pen.goto(0, 260)
-pen.write("score", align="center", font=('Arial', 24, 'Normal'))
+pen.write(f"player A:{playerAscore}  player B:{playerBscore}", align='center', font=('Arial', 24, 'normal'))

Turtleオブジェクトの生成は、他の回答者の回答にもあるように、t.Turtleではなくt.Turtle()です。

ボールの初期位置はball.goto(5, 5)のように、x座標とy座標をカンマ区切りで指定します。元のコードでは(5, 5)に設定しようとして(カンマではなくドットを書いて)失敗していますが、パドルで受け止め損ねて再開するときは(0, 0)に設定しているので、ここでは(0, 0)に揃えてみました。

なお、ball.get_shapepoly()で得られるリストを見ると分かりますが、ボールの半径は10ピクセル、実際には円ではなく20角形です。

また、ペンで文字を描画する処理では、fontの設定中の'Normal'は'normal'と小文字で書く必要があります。0対0から点数を表示するために、後に書いているスコア表示部分と揃えてみました。

文字列中に変数の値を埋め込むときは、"{}".format(変数名)を使うより、f"{変数名}"を使ったほうが読みやすくなります。

↓キーに対する関数割り当て

 window.onkeypress(leftpaddleup, 'w')
 window.onkeypress(leftpaddledown, 's')
 window.onkeypress(rightpaddleup, 'Up')
-window.onkeypress(leftpaddledown, 'Down')
+window.onkeypress(rightpaddledown, 'Down')

↓キー(Down)を押したときに右側のパドルを下に移動させるはずが、leftpaddledownを割り当てているために左側のパドルが下に移動してしまいます。rightpaddledownを割り当ててください。

ボールの移動処理

-    ball.setx(ball.xcor(+ballxdirection))
-    ball.sety(ball.ycor(+ballydirection))
+    ball.setx(ball.xcor() + ballxdirection)
+    ball.sety(ball.ycor() + ballydirection)

ボールのx座標を得るball.xcor()や、y座標を得るball.ycor()には引数を指定しません。それらの返り値に対して、ballxdirectionやballydirectionを加算してください。

画面の左右端でボールを消して中央に戻し、得点を加算する処理

-    if (ball.xcor()) < -390:
+    if ball.xcor() < -390:
         ball.goto(0, 0)
         ballxdirection = ballxdirection * -1
-        playerAscore = playerBscore + 1
+        playerAscore += 1
         pen.clear()
-        pen.write("player A:{}  player B:{}".format(playerAscore, playerBscore), align='center', font=('Arial', 24, 'normal'))
+        pen.write(f"player A:{playerAscore}  player B:{playerBscore}", align='center', font=('Arial', 24, 'normal'))

+    if ball.xcor() > 390:
+        ball.goto(0, 0)
+        ballxdirection = ballxdirection * -1
+        playerBscore += 1
+        pen.clear()
+        pen.write(f"player A:{playerAscore}  player B:{playerBscore}", align='center', font=('Arial', 24, 'normal'))
+

ボールの半径が10ピクセル(直径は20ピクセル)、画面の幅が800ピクセルなので、概算で(-400, 400)の範囲と見なし、ボールの一部が画面からはみ出さないように、±390を比較対象としています。

元のソースでは、左端での処理しか書いていなかったので、右端での処理を追加します。playerAscore = playerBscore + 1なんて書いてありました。+=を使えば、右辺に同じ変数名を書かずに、値の更新を明示できます。

今回は修正個所が膨大になるためやりませんでしたが、ballxdirection = ballxdirection * -1もballxdirection *= -1と短く書けます。

左右のパドルとボールの接触判定とボールが跳ねる処理

-    if (ball.xcor() > 340) and (ball.xcor() < 350) and (ball.ycor() < rightpaddle.ycor() + 40 and ball.ycor() > rightpaddle.ycor() - 40):
-        ball.setx(340)
+    if 330 < ball.xcor() < 350 and rightpaddle.ycor() - 50 < ball.ycor() < rightpaddle.ycor() + 50:
+        ball.setx(330)
         ballxdirection = ballxdirection * -1

-    if (ball.xcor() < -340) and (ball.xcor() < -350) and (ball.ycor() < leftpaddle.ycor() + 40 and ball.ycor() > leftpaddle.ycor() - 40):
-        ball.setx(340)
+    if -350 < ball.xcor() < -330 and leftpaddle.ycor() - 50 < ball.ycor() < leftpaddle.ycor() + 50:
+        ball.setx(-330)
         ballxdirection = ballxdirection * -1

「x > A and x < B」を「A < x < B」と書けるのがPythonの特長のひとつです。

説明が面倒なので、右のパドルとの接触判定だけ。

パドルの幅は20ピクセルあり、右のパドル中心のx座標350の左右に10ピクセルずつ広がっています。よって、中心より10ピクセル中央よりの340が右のパドルの前面のx座標です。

ボールの半径が10ピクセルなので、その右端がパドルの前面と接触するときのボール中心のx座標は330、左端がパドルの前面と接触するときのボール中心のx座標が350です。

ということで、右のパドルの前面とボールが重なるときのボール中心のx座標の条件は330 < ball.xcor() < 350ということになります。

同様に、パドルの高さは100ピクセルあり、右のパドル中心のy座標rightpaddle.ycor()の上下に50ピクセルずつ広がっています。

ということで、ボール中心のy座標ball.ycor()が右のパドルと重なる条件は、rightpaddle.ycor() - 50 < ball.ycor() < rightpaddle.ycor() + 50ということになります。

本当はこちらもボールの半径10ピクセルを考慮したいところですが、面倒なので省略します。

なお、元のコードでは、左のパドルとの接触判定の条件式ball.xcor() < -350の不等号が反対ですし、条件成立時になぜかボールのx座標を(-340ではなく)340に設定しています。

---

以上、現状のコードを(クラスを作ったりせずに)できるだけそのままの形で動くようにしてみました。

Comments

[rawr]

ありがとうございます！
加算の方法、fontの設定など知らないことをたくさん教えていただきました。右のパドルが表示できない理由も解決できて良かったです。数値の設定も解説してくださって本当に助かりました。
ゲームとしては機能するようになったのであとはボールの速さを調節してみたいと思います。

[Daregada]

あとは、

・左右のパドルの中心y座標が画面の範囲、概算で(-300, 300)を超えるとパドルが表示されなくなるので、leftpaddleupなどの関数で「パドルの中心が画面からはみ出さないときだけ、パドルのy座標を変更する」
・パドルやボールの大きさを決め打ちして数値を書いている部分を、get_shapepoly()で得られるリスト内の数値を参考にしてパドルの高さやボールの半径を求め、それを使った式に変更する→パドルやボールの大きさを変えてもそのまま使えるようにする。
・プレイヤーや左右のパドルの処理が似ているので、クラスを作って共通化できないか検討する

あたりを、少しずつ試してみてください。

[rawr]

返信ありがとうございます。やってみます。

Answer 2

t.Turtleではなくt.Turtle()ではないでしょうか？

ball=t.Turtle()

pen=t.Turtle()

Comments

[rawr]

ありがとうございます！（）付け足しました！
するとspeedの方のエラーがなくなって
turtle.Vec2D() argument after * must be an iterable, not float
File "C:\Users\1200728\Desktop\test\test.py", line 33, in <module>
ball.goto(5.5)
となったのですがどういう意味なんでしょうか？
調べたけど分からないので教えてほしいです????お願い致します。

[__horito]

ball.goto(5.5)ではなくball.goto(5,5)では？（ピリオドではなくカンマでは？）

[Daregada]

一応動くようにしてみましたが、

・キー割り当て時の左右のパドルの取り違え
・ボールを動かす際の式の間違い
・スコア加算時の変数の取り違え
・ボールが右端に到達したときの処理がない
・左側のパドルで跳ね返すときの条件式と設定するx座標の符号間違い

など大量のバグがありますね。
今から食事に出かけるので対応できませんが。

[rawr]

>>ball.goto(5.5)ではなくball.goto(5,5)では？（ピリオドではなくカンマでは？）
本当ですね！修正します(>_<)

[rawr]

>>一応動くようにしてみましたが、

・キー割り当て時の左右のパドルの取り違え
・ボールを動かす際の式の間違い
・スコア加算時の変数の取り違え
・ボールが右端に到達したときの処理がない
・左側のパドルで跳ね返すときの条件式と設定するx座標の符号間違い

ありがとうございます。自分でもチェックしてみますがお時間のある時に教えてもらうことってできますでしょうか。