need help

Cause you create a new object inside the constructor taking two arguments instead of calling an overloaded constructor. Creating a new object and throwing it away right after just to be able to access an int literal is wasteful. Then you could just have written radius = 10; Task description however ask to call another constructor from within the constructor with two parameters. And this is done using this (the reference to the current object) followed by the three arguments you intend to use -> this(x, y, 10);