C#•3mo ago

OOP help with a project please

please dm need help

66 Replies

Buddy•3mo ago

We do not help in DMs $details

MODiX•3mo ago

When you ask a question, make sure you include as much detail as possible. Such as code, the issue you are facing, what you expect the result to be, what .NET version you are using and what platform/environment (if any) are relevant to your question. Upload code here https://paste.mod.gg/ (see $code for more information on how to paste your code)

Tajaba•3mo ago

ok ill paste

Buddy•3mo ago

Read all of it We do not help with entire projects.

Tajaba•3mo ago

i understand i wrote most of it but facing a problem which i dont know how to countiue can i screen shot?

Buddy•3mo ago

a problem which is ..? Please avoid screenshotting unless it contains the error

Tajaba•3mo ago

i wrote a program which drawing shape in the console and it it drawing an endless loop of shapes when i starting it i want to know why

Angius•3mo ago

Just send the code, then

Tajaba•3mo ago

ok wait public static class ScreenSaver { // Start the screen saver public static void Start() { // Loop until any key is pressed while (!Console.KeyAvailable) { // Create a random shape Shape shape = GenerateRandomShape(); // Initialize shape with random values shape.InitWithRandomValues(); // Draw the shape shape.Draw(); // Wait for half a second Thread.Sleep(500); // Show details of the shape shape.ShowDetails(); // Wait for another half a second Thread.Sleep(500); // Clear the console Console.Clear(); } } // Generate a random shape private static Shape GenerateRandomShape() { Random rand = new Random(); int shapeType = rand.Next(1, 6); // Random number between 1 and 5 switch (shapeType) { case 1: return new Square(0, 0, Shape.ConsoleColor.white, 0,0); case 2: return new Circle(0, 0, Shape.ConsoleColor.white, 0); case 3: return new Rectangle(0, 0, Shape.ConsoleColor.white, 0, 0); case 4: return new VerticalLine(0, 0, 0, Shape.ConsoleColor.white, 0); case 5: return new HorizontalLine(0, 0, 0, Shape.ConsoleColor.white, 0); default: return new Square(0, 0, Shape.ConsoleColor.white, 0,0); } } } }

Angius•3mo ago

$code

MODiX•3mo ago

To post C# code type the following: ```cs // code here ``` Get an example by typing $codegif in chat For longer snippets, use: https://paste.mod.gg/

Tajaba•3mo ago

$code public static class ScreenSaver { // Start the screen saver public static void Start() { // Loop until any key is pressed while (!Console.KeyAvailable) { // Create a random shape Shape shape = GenerateRandomShape(); // Initialize shape with random values shape.InitWithRandomValues(); // Draw the shape shape.Draw(); // Wait for half a second Thread.Sleep(500); // Show details of the shape shape.ShowDetails(); // Wait for another half a second Thread.Sleep(500); // Clear the console Console.Clear(); } } // Generate a random shape private static Shape GenerateRandomShape() { Random rand = new Random(); int shapeType = rand.Next(1, 6); // Random number between 1 and 5 switch (shapeType) { case 1: return new Square(0, 0, Shape.ConsoleColor.white, 0,0); case 2: return new Circle(0, 0, Shape.ConsoleColor.white, 0); case 3: return new Rectangle(0, 0, Shape.ConsoleColor.white, 0, 0); case 4: return new VerticalLine(0, 0, 0, Shape.ConsoleColor.white, 0); case 5: return new HorizontalLine(0, 0, 0, Shape.ConsoleColor.white, 0); default: return new Square(0, 0, Shape.ConsoleColor.white, 0,0); } } } }

MODiX•3mo ago

To post C# code type the following: ```cs // code here ``` Get an example by typing $codegif in chat For longer snippets, use: https://paste.mod.gg/

Tajaba•3mo ago

'''cs public static class ScreenSaver { // Start the screen saver public static void Start() { // Loop until any key is pressed while (!Console.KeyAvailable) { // Create a random shape Shape shape = GenerateRandomShape(); // Initialize shape with random values shape.InitWithRandomValues(); // Draw the shape shape.Draw(); // Wait for half a second Thread.Sleep(500); // Show details of the shape shape.ShowDetails(); // Wait for another half a second Thread.Sleep(500); // Clear the console Console.Clear(); } } // Generate a random shape private static Shape GenerateRandomShape() { Random rand = new Random(); int shapeType = rand.Next(1, 6); // Random number between 1 and 5 switch (shapeType) { case 1: return new Square(0, 0, Shape.ConsoleColor.white, 0,0); case 2: return new Circle(0, 0, Shape.ConsoleColor.white, 0); case 3: return new Rectangle(0, 0, Shape.ConsoleColor.white, 0, 0); case 4: return new VerticalLine(0, 0, 0, Shape.ConsoleColor.white, 0); case 5: return new HorizontalLine(0, 0, 0, Shape.ConsoleColor.white, 0); default: return new Square(0, 0, Shape.ConsoleColor.white, 0,0); } } } }

Somgör from Human Resources•3mo ago

Its `

Angius•3mo ago

` not '

Somgör from Human Resources•3mo ago

Not '

Angius•3mo ago

The key left to 1, above Tab At least on my keyboard, ymmv

Tajaba•3mo ago

still it wont let me enter the code

Angius•3mo ago

    public static class ScreenSaver
    {
        // Start the screen saver
        public static void Start()
        {
            // Loop until any key is pressed
            while (!Console.KeyAvailable)
            {
                // Create a random shape
                Shape shape = GenerateRandomShape();

                // Initialize shape with random values
                shape.InitWithRandomValues();

                // Draw the shape
                shape.Draw();

                // Wait for half a second
                Thread.Sleep(500);

                // Show details of the shape
                shape.ShowDetails();

                // Wait for another half a second
                Thread.Sleep(500);

                // Clear the console
                Console.Clear();
            }
        }

        // Generate a random shape
        private static Shape GenerateRandomShape()
        {
            Random rand = new Random();
            int shapeType = rand.Next(1, 6); // Random number between 1 and 5
            switch (shapeType)
            {
                case 1:
                    return new Square(0, 0, Shape.ConsoleColor.white, 0,0);
                case 2:
                    return new Circle(0, 0, Shape.ConsoleColor.white, 0);
                case 3:
                    return new Rectangle(0, 0, Shape.ConsoleColor.white, 0, 0);
                case 4:
                    return new VerticalLine(0, 0, 0, Shape.ConsoleColor.white, 0);
                case 5:
                    return new HorizontalLine(0, 0, 0, Shape.ConsoleColor.white, 0);
                default:
                    return new Square(0, 0, Shape.ConsoleColor.white, 0,0);
            }
        }
    }



}

    public static class ScreenSaver
    {
        // Start the screen saver
        public static void Start()
        {
            // Loop until any key is pressed
            while (!Console.KeyAvailable)
            {
                // Create a random shape
                Shape shape = GenerateRandomShape();

                // Initialize shape with random values
                shape.InitWithRandomValues();

                // Draw the shape
                shape.Draw();

                // Wait for half a second
                Thread.Sleep(500);

                // Show details of the shape
                shape.ShowDetails();

                // Wait for another half a second
                Thread.Sleep(500);

                // Clear the console
                Console.Clear();
            }
        }

        // Generate a random shape
        private static Shape GenerateRandomShape()
        {
            Random rand = new Random();
            int shapeType = rand.Next(1, 6); // Random number between 1 and 5
            switch (shapeType)
            {
                case 1:
                    return new Square(0, 0, Shape.ConsoleColor.white, 0,0);
                case 2:
                    return new Circle(0, 0, Shape.ConsoleColor.white, 0);
                case 3:
                    return new Rectangle(0, 0, Shape.ConsoleColor.white, 0, 0);
                case 4:
                    return new VerticalLine(0, 0, 0, Shape.ConsoleColor.white, 0);
                case 5:
                    return new HorizontalLine(0, 0, 0, Shape.ConsoleColor.white, 0);
                default:
                    return new Square(0, 0, Shape.ConsoleColor.white, 0,0);
            }
        }
    }



}

Works just fine

Tajaba•3mo ago

i now wait

    public static class ScreenSaver
    {
        // Start the screen saver
        public static void Start()
        {
            // Loop until any key is pressed
            while (!Console.KeyAvailable)
            {
                // Create a random shape
                Shape shape = GenerateRandomShape();

                // Initialize shape with random values
                shape.InitWithRandomValues();

                // Draw the shape
                shape.Draw();

                // Wait for half a second
                Thread.Sleep(500);

                // Show details of the shape
                shape.ShowDetails();

                // Wait for another half a second
                Thread.Sleep(500);

                // Clear the console
                Console.Clear();
            }
        }

        // Generate a random shape
        private static Shape GenerateRandomShape()
        {
            Random rand = new Random();
            int shapeType = rand.Next(1, 6); // Random number between 1 and 5
            switch (shapeType)
            {
                case 1:
                    return new Square(0, 0, Shape.ConsoleColor.white, 0,0);
                case 2:
                    return new Circle(0, 0, Shape.ConsoleColor.white, 0);
                case 3:
                    return new Rectangle(0, 0, Shape.ConsoleColor.white, 0, 0);
                case 4:
                    return new VerticalLine(0, 0, 0, Shape.ConsoleColor.white, 0);
                case 5:
                    return new HorizontalLine(0, 0, 0, Shape.ConsoleColor.white, 0);
                default:
                    return new Square(0, 0, Shape.ConsoleColor.white, 0,0);
            }
        }
    }



}

    public static class ScreenSaver
    {
        // Start the screen saver
        public static void Start()
        {
            // Loop until any key is pressed
            while (!Console.KeyAvailable)
            {
                // Create a random shape
                Shape shape = GenerateRandomShape();

                // Initialize shape with random values
                shape.InitWithRandomValues();

                // Draw the shape
                shape.Draw();

                // Wait for half a second
                Thread.Sleep(500);

                // Show details of the shape
                shape.ShowDetails();

                // Wait for another half a second
                Thread.Sleep(500);

                // Clear the console
                Console.Clear();
            }
        }

        // Generate a random shape
        private static Shape GenerateRandomShape()
        {
            Random rand = new Random();
            int shapeType = rand.Next(1, 6); // Random number between 1 and 5
            switch (shapeType)
            {
                case 1:
                    return new Square(0, 0, Shape.ConsoleColor.white, 0,0);
                case 2:
                    return new Circle(0, 0, Shape.ConsoleColor.white, 0);
                case 3:
                    return new Rectangle(0, 0, Shape.ConsoleColor.white, 0, 0);
                case 4:
                    return new VerticalLine(0, 0, 0, Shape.ConsoleColor.white, 0);
                case 5:
                    return new HorizontalLine(0, 0, 0, Shape.ConsoleColor.white, 0);
                default:
                    return new Square(0, 0, Shape.ConsoleColor.white, 0,0);
            }
        }
    }



}

Angius•3mo ago

And the issue is that the loop never stops running, right?

Tajaba•3mo ago

i dont know its just spamming ** all over the console

maxmahem•3mo ago

does it stop if you hit a key?

Tajaba•3mo ago

Angius•3mo ago

I assume * is the character you use to draw the shapes

Tajaba•3mo ago

i think theres a problem with one of the shapes

Angius•3mo ago

Well, I'd say use the debugger to see how it goes

Tajaba•3mo ago

correct

Angius•3mo ago

Go ahead, then

Tajaba•3mo ago

i dont know how to use it):

Angius•3mo ago

$debug

MODiX•3mo ago

https://docs.microsoft.com/en-us/visualstudio/get-started/csharp/tutorial-debugger?view=vs-2022

Tutorial: Debug C# code and inspect data - Visual Studio (Windows)

Learn features of the Visual Studio debugger and how to start the debugger, step through code, and inspect data in a C# application.

Tajaba•3mo ago

Tajaba•3mo ago

thats whats it doing

Angius•3mo ago

Did you place a breakpoint anywhere?

Tajaba•3mo ago

i think the problem is in one of the shape classes i have wrote yes

Angius•3mo ago

And did it break the code execution? If so, then step through and see what prints this mass of asterisks

Tajaba•3mo ago

still coudlnt found it

Angius•3mo ago

Do a test Instead of printing a random shape, try printing them individually' Outside of the loop, even Comment it out See which shape causes issues

Tajaba•3mo ago

tried to do circle it and thats what it showed de

Angius•3mo ago

What's this entire property

Tajaba•3mo ago

     public class Circle :Shape
    {
        private int Radius
         {
            get { return Radius; }
            set { if (Radius<12.5) Radius = value; }
        }
        public Circle()
        {}

        public Circle(int x, int y, ConsoleColor color, int radius) :base(x,y,color)
        {
            Radius = radius;
        }
        public override void InitWithRandomValues()
        {
            Random rnd = new Random();
            X = rnd.Next(1, 80);
            Y = rnd.Next(1, 25);
            Color = (ConsoleColor)rnd.Next(Enum.GetNames(typeof(ConsoleColor)).Length);
            Radius = rnd.Next(1, (int)12.5);
        }
        public override void Draw()
        {
            double angleStep = 1.0 / Radius;
            Console.SetCursorPosition(X, Y);
            Console.ForegroundColor = (System.ConsoleColor)color;
            for (double angle = 0; angle <= 2 * Math.PI; angle += angleStep)
            {
                int x = (int)Math.Round(X + Radius * Math.Cos(angle));
                int y = (int)Math.Round(Y + Radius * Math.Sin(angle));
                Console.SetCursorPosition(x, y);
                Console.Write("*");

            }
        }

        public override double GetArea()
        {
            return (Math.PI * Radius * Radius);
        }

        public override double GetPerimeter()
        {
            return (2 * Math.PI * Radius);
        }

        public override void ShowDetails()
        {
            Console.Write("X" + X, "Y" + Y, "Color" + color, "Radius" + Radius + "Area" + GetArea() + "Perimeter" + GetPerimeter());

        }


    }
}

     public class Circle :Shape
    {
        private int Radius
         {
            get { return Radius; }
            set { if (Radius<12.5) Radius = value; }
        }
        public Circle()
        {}

        public Circle(int x, int y, ConsoleColor color, int radius) :base(x,y,color)
        {
            Radius = radius;
        }
        public override void InitWithRandomValues()
        {
            Random rnd = new Random();
            X = rnd.Next(1, 80);
            Y = rnd.Next(1, 25);
            Color = (ConsoleColor)rnd.Next(Enum.GetNames(typeof(ConsoleColor)).Length);
            Radius = rnd.Next(1, (int)12.5);
        }
        public override void Draw()
        {
            double angleStep = 1.0 / Radius;
            Console.SetCursorPosition(X, Y);
            Console.ForegroundColor = (System.ConsoleColor)color;
            for (double angle = 0; angle <= 2 * Math.PI; angle += angleStep)
            {
                int x = (int)Math.Round(X + Radius * Math.Cos(angle));
                int y = (int)Math.Round(Y + Radius * Math.Sin(angle));
                Console.SetCursorPosition(x, y);
                Console.Write("*");

            }
        }

        public override double GetArea()
        {
            return (Math.PI * Radius * Radius);
        }

        public override double GetPerimeter()
        {
            return (2 * Math.PI * Radius);
        }

        public override void ShowDetails()
        {
            Console.Write("X" + X, "Y" + Y, "Color" + color, "Radius" + Radius + "Area" + GetArea() + "Perimeter" + GetPerimeter());

        }


    }
}

maxmahem•3mo ago

oh your in VS? Just pause while it's drawing. You should go straight to where the loop is (probably)

Angius•3mo ago

        private int Radius
         {
            get { return Radius; }
            set { if (Radius<12.5) Radius = value; }
        }

        private int Radius
         {
            get { return Radius; }
            set { if (Radius<12.5) Radius = value; }
        }

I needed just the property So, what it does is: when you get a value from Radius, it gets a value from Radius

Tajaba•3mo ago

oh my english is not that good so forgive me

Angius•3mo ago

To do that, it needs to get a value from Radius, which gets a value from Radius, which gets a value from Radius, which gets a value from Radius, which gets a value from Radius, which gets a value from Radius, which gets a value from Radius, which gets a value from Radius, which gets a value from Radius, which gets a value from Radius, which gets a value from Radius, which gets a value from Radius, which gets a value from Radius, which gets a value from Radius, which gets a value from Radius, which gets a value from Radius, which gets a value from Radius Error: stack overflow If you want to have a property like that, you need a backing field You cannot reference the property in itself, you will get an infinite loop And this will overflow the stack

Tajaba•3mo ago

got it so how i fix it

Angius•3mo ago

By using a backing field $getsetdevolve

MODiX•3mo ago

class Foo
{
    private int _bar;

    public int GetBar()
    {
        return _bar;
    }

    public void SetBar(int bar)
    {
        _bar = bar;
    }
}

class Foo
{
    private int _bar;

    public int GetBar()
    {
        return _bar;
    }

    public void SetBar(int bar)
    {
        _bar = bar;
    }
}

can be shortened to

class Foo
{
    private int _bar;

    public int GetBar() => _bar;

    public void SetBar(int bar) => _bar = bar;
}

class Foo
{
    private int _bar;

    public int GetBar() => _bar;

    public void SetBar(int bar) => _bar = bar;
}

can be shortened to

class Foo
{
    private int _bar;
    public int Bar {
        get { return _bar; }
        set { _bar = value; }
    }
}

class Foo
{
    private int _bar;
    public int Bar {
        get { return _bar; }
        set { _bar = value; }
    }
}

can be shortened to

class Foo
{
    private int _bar;
    public int Bar {
        get => _bar;
        set => _bar = value;
    }
}

class Foo
{
    private int _bar;
    public int Bar {
        get => _bar;
        set => _bar = value;
    }
}

can be shortened to

class Foo
{
    public int Bar { get; set; }
}

class Foo
{
    public int Bar { get; set; }
}

Angius•3mo ago

See the examples that use _underscored fields Second to last example, in particular That is a property with a backing field

Tajaba•3mo ago

ok but in my program i get a random value for the radius

Angius•3mo ago

So? The value you give to the property can be whatever Using a backing field does not dictate what value can you give the property We're only fixing this The usage will remain the exact same

Tajaba•3mo ago

the backing field shot be in the same class?

Angius•3mo ago

How is it in the examples posted above?

Tajaba•3mo ago

bro i got confused fr

Angius•3mo ago

What's confusing about it?

Tajaba•3mo ago

i didnt understand what to change in the code and where

Angius•3mo ago

Instead of referencing the property, which references the property, which references the property, which references the property, which references the property, which references the property, which references the property Make a field And reference the field from the property

Tajaba•3mo ago

ok how to do it

Angius•3mo ago

BAD

class Foo
{
    public int Bar {
        get => Bar;
        set => Bar = value;
    }
}

class Foo
{
    public int Bar {
        get => Bar;
        set => Bar = value;
    }
}

GOOD

class Foo
{
    private int _bar;
    public int Bar {
        get => _bar;
        set => _bar = value;
    }
}

class Foo
{
    private int _bar;
    public int Bar {
        get => _bar;
        set => _bar = value;
    }
}

Tajaba•3mo ago

oh wait ill do it

 private int radius;
 public int Radius
 {
     get => radius;
     set => radius = value;
 }

 private int radius;
 public int Radius
 {
     get => radius;
     set => radius = value;
 }

like that?

Angius•3mo ago

Yes

Tajaba•3mo ago

Angius•3mo ago

Except your code has some logic in the serrer

set { if (Radius<12.5) Radius = value; }

set { if (Radius<12.5) Radius = value; }

So you might want to add the same logic here

Gaming

Programming

OOP help with a project please