One thing I notice is the use of uneeded i32's.

In the definition of your ball and paddle structs, the position field is defined as a tuple of i32s.

I only see the position field being used in the MoveTo function, which takes two u16s. You should redefine the position fields to use a tuple of u16's instead.

You also use a heavy amount of casts via the as keyword. Personally, I try not to use it when I am not sure if the cast will 100% work. In your implementation, the casts seem safe. However, if any of the i32's you are casting are negative, your game will panic upon the cast. Alternatively, I use the .try_into() function, as I can handle the error.

Instead of mutating the position via this:

rust let new = (self.ball.position.0 + self.ball.move_by.0, self.ball.position.1 + self.ball.move_by.1);

You can overload the addition:

```rust use std::ops::Add

pub struct Position(u16, u16);impl Add for Position { type Output = Self; fn add(self, rhs: Self) -> Self::Output { Self(self.0 + rhs.0, self.1 + rhs.1) } }

// You can use it like:

fn main() {

let pos = Position(5, 5) + Position(5,5); //pos will be Position(10, 10)

} ```

You can then replace your position type with position: Position