Python Pong using turtle

Question

I am a new Python programmer, and I am learning how to use the turtle import. I am making some projects, and now I am trying to do a Pong game using turtle.

It is working but not really well; the hitbox and the collisions can be improved. Can you please look at my code and tell me how I can improve it?

Code:

import turtle
import os
import random
import math


#screen
wn = turtle.Screen()
wn.title("PingPangPong")
wn.bgcolor("black")
wn.setup(width = 1.0, height = 1.0)

#Shapes
turtle.register_shape("pong.gif")

#bordure
bordure = turtle.Turtle()
bordure.penup()
bordure.color("white")
bordure.pensize(3)
bordure.ht()
bordure.setposition(-500, 300)
bordure.speed(5)
def bordure1():
        bordure.pendown()
        bordure.fd(1000)
        bordure.pu()
        bordure.setposition(-500,-300)
        bordure.pd()
        bordure.fd(1000)

def filet():
    bordure.pensize(1)
    bordure.setposition(0, -300)
    bordure.setheading(90)
    for i in range (12):
        bordure.fd(26)
        bordure.penup()
        bordure.fd(26)
        bordure.pendown()

bordure1()
filet()


#Ball
ball = turtle.Turtle()
ball.shape("circle")
ball.color("white")
ballspeed = 8
ballheading = random.randint(1,360)
ball.penup()
ball.setheading(ballheading)
ball.speed(0)

#Player 1
player1 = turtle.Turtle()
player1.shape("pong.gif")
player1.turtlesize(2,2)
player1.color("white")
player1.penup()
player1.setposition(-520, 0)
player1.speed(0)
playerspeed = 50

#Player 2
player2 = turtle.Turtle()
player2.shape("pong.gif")
player2.shapesize(5,5)
player2.color("white")
player2.penup()
player2.setposition(520, 0)
player2.speed(0)
playerspeed = 50

#Player 1 Movement
def up():
        y = player1.ycor()
        y += playerspeed
        if y < 270:
                player1.sety(y)

def down():
        y = player1.ycor()
        y -= playerspeed
        if y > -270:
                player1.sety(y)               
#Player 2 movement
def up1():
        y = player2.ycor()
        y += playerspeed
        if y < 280:
                player2.sety(y)

def down1():
        y = player2.ycor()
        y -= playerspeed
        if y > -280:
                player2.sety(y)
#Speed hack
def hack():
        global ballspeed
        if ballspeed == 8:
                ballspeed = 20
        elif ballspeed == 20:
                ballspeed = 8

def start():
        ballheading = random.randint(1,360)
        ball.setposition(0, 0)
        ball.setheading(ballheading)


def distance(t1,t2):
        distance = math.sqrt(math.pow(t1.xcor()-t2.xcor(),2)+math.pow(t1.ycor()-t2.ycor(),2))
        if distance < 45:
                t2.right(91)

def angle():
        print(ball.heading())



#We assign z/s to move the player 1
turtle.listen()
turtle.onkey(up, "z")
turtle.onkey(down,"s")
#We assign up and down arrow to move the player 2
turtle.onkey(up1, "Up")
turtle.onkey(down1,"Down")
#SpeedHack
turtle.onkey(hack, "m")
#Restart
turtle.onkey(start,"p")
#Print the Ball heading
turtle.onkey(angle, "l")
#Player 2 score's
Score1 = 0
s1 = turtle.Turtle()
s1.speed(0)
s1.ht()
s1.color("white")
s1.pu()
s1.setposition(-250, 250)
s1.write(Score1, font=("Arial", 44, "normal"))
#Player 2 score's
Score2 = 0
s2 = turtle.Turtle()
s2.speed(0)
s2.ht()
s2.color("white")
s2.pu()
s2.setposition(250, 250)
s2.write(Score1, font=("Arial", 44, "normal"))

#Mainloop
while True:
        ball.fd(ballspeed)
        degree = ball.heading()
        y = ball.ycor()
        x = ball.xcor()
#We define the border colision 
        if y > 279 or y < -279:
                 ball.right(91)
#We define Loose              
        if x > 520:
                ballheading = random.randint(1,360)
                Score1 += 1
                s1.clear()
                s1.write(Score1, font=("Arial", 44, "normal"))
                start()
        if x < -520:
                ballheading = random.randint(1,360)
                Score2 += 1
                s2.clear()
                s2.write(Score2, font=("Arial", 44, "normal"))
                start()

#Colision beetween players and ball                
        distance(player1,ball)
        distance(player2,ball)

turtle.mainloop()

Answer 1

The major issues I see with your code are:

• Use of while True: in an event driven environment. We need to put the ball on a timer event instead.

• The angle of bounce off the sides of the court is unrealistic. You can't simply ball.right(91) as which way you turns depends on your original direction. We can reflect the heading instead.

• Ditto for the bouncing off the players. Also, the initial ball angle is too broad and leads to unplayable scenarios.

My rework of your code attempts to address these issues as well as clean up the code style-wise:

from turtle import Turtle, Screen
from random import randint, choice

ballspeed = 8
playerspeed = 50

cursor_size = 20
player_height = 60
player_width = 20

court_width = 1000
court_height = 600

FONT = ("Arial", 44, "normal")

def draw_border():
    border.pensize(3)
    border.penup()
    border.setposition(-court_width/2, court_height/2)
    border.pendown()
    border.forward(court_width)
    border.penup()
    border.sety(-court_height/2)
    border.pendown()
    border.backward(court_width)

def filet():
    border.penup()
    border.pensize(1)
    border.setposition(0, -court_height/2)
    border.setheading(90)
    border.pendown()

    for _ in range(court_height // 50):
        border.forward(50 / 2 + 1)
        border.penup()
        border.forward(50 / 2 + 1)
        border.pendown()

# Player 1 Movement
def up1():
    y = player1.ycor()
    y += playerspeed
    if y < court_height/2 - player_height/2:
        player1.sety(y)

def down1():
    y = player1.ycor()
    y -= playerspeed
    if y > player_height/2 - court_height/2:
        player1.sety(y)

# Player 2 movement
def up2():
    y = player2.ycor()
    y += playerspeed
    if y < court_height/2 - player_height/2:
        player2.sety(y)

def down2():
    y = player2.ycor()
    y -= playerspeed
    if y > player_height/2 - court_height/2:
        player2.sety(y)

def reset_ball():
    ball.setposition(0, 0)
    ball.setheading(choice([0, 180]) + randint(-60, 60))

def distance(t1, t2):
    my_distance = t1.distance(t2)

    if my_distance < player_height/2:
        t2.setheading(180 - t2.heading())
        t2.forward(ballspeed)

# Mainloop
def move():
    global score1, score2

    ball.forward(ballspeed)

    x, y = ball.position()

    if x > court_width/2 + cursor_size: # We define scoring
        score1 += 1
        s1.undo()
        s1.write(score1, font=FONT)
        reset_ball()
    elif x < cursor_size - court_width/2:
        score2 += 1
        s2.undo()
        s2.write(score2, font=FONT)
        reset_ball()
    elif y > court_height/2 - cursor_size or y < cursor_size - court_height/2:
        # We define the border collision
        ball.setheading(-ball.heading())
    else:
        # Check collision between players and ball
        distance(player1, ball)
        distance(player2, ball)

    screen.ontimer(move, 20)

# screen
screen = Screen()
screen.title("Pong")
screen.bgcolor("black")
screen.setup(width=1.0, height=1.0)

# border
border = Turtle(visible=False)
border.speed('fastest')
border.color("white")

draw_border()
filet()

# Ball
ball = Turtle("circle")
ball.color("white")
ball.penup()
ball.speed("fastest")

reset_ball()

# Player 1
player1 = Turtle("square")
player1.turtlesize(player_height / cursor_size, player_width / cursor_size)
player1.color("white")
player1.penup()
player1.setx(cursor_size - court_width/2)
player1.speed("fastest")

# Player 2
player2 = Turtle("square")
player2.shapesize(player_height / cursor_size, player_width / cursor_size)
player2.color("white")
player2.penup()
player2.setx(court_width/2 + cursor_size)
player2.speed("fastest")

# Player 1 score
score1 = 0
s1 = Turtle(visible=False)
s1.speed("fastest")
s1.color("white")
s1.penup()
s1.setposition(-court_width/4, court_height/3)
s1.write(score1, font=FONT)

# Player 2 score"s
score2 = 0
s2 = Turtle(visible=False)
s2.speed("fastest")
s2.color("white")
s2.penup()
s2.setposition(court_width/4, court_height/3)
s2.write(score2, font=FONT)

# We assign s/z to move the player 1
screen.onkey(up1, "s")
screen.onkey(down1, "z")

# We assign up and down arrow to move the player 2
screen.onkey(up2, "Up")
screen.onkey(down2, "Down")

# Restart
screen.onkey(reset_ball, "p")

screen.listen()

move()
screen.mainloop()

Not perfect, but playable. I've tossed some of your original details to simplify my example. You should be able to add them back as desired.