C# Geometry Calculator

[Dobbsjr]

Just a program that calculates the surface area and volume of several(well, one in this version)3D objects. Just want some feedback on this program.

 

Code + EXE: https://github.com/GenesisOfTheVoid/C-.NET-Projects/tree/master/GeoCalc

[LePawel]

Suggestion:

Switch from buttons to radio buttons so you can select a specific object type and display all fill forms below, and answer at the bottom. This reduced the number of popups, improves usability. Quick sketch:

[Dobbsjr]

Suggestion:

Switch from buttons to radio buttons so you can select a specific object type and display all fill forms below, and answer at the bottom. This reduced the number of popups, improves usability. Quick sketch:

 

Can you elaborate a little more on how to do this?

[madknight3]

Lets start simple. Separate the UI code from the code that runs the calculations and organize your shapes in their own classes. Each shape (Cylinder, Sphere, Cube, etc) would get their own class.

public class Cylinder{    private int _radius;    private int _height;    public Cylinder(int radius, int height)    {        _radius = radius;        _height = height;    }    public double SurfaceArea()    {        // For you to complete    }    public double Volume()    {        // For you to complete    }}

Then your buttons would create and use the Cylinder object instead of doing the calculations directly. A task for you to complete.

 

*Note that your form class is also named Cylinder, so if you create a new class named Cylinder in that project it will conflict. I would rename the form to CylinderForm first.*

[LePawel]

Can you elaborate a little more on how to do this?

Unfortunately, my C# knowledge is rather basic, so all I can do is

https://msdn.microsoft.com/en-us/library/System.Windows.Forms.RadioButton(v=vs.110).aspx?cs-save-lang=1&cs-lang=csharp#code-snippet-1 

http://stackoverflow.com/questions/1797907/which-radio-button-in-the-group-is-checked

http://csharp.net-informations.com/gui/cs-radiobutton.htm 

on click event: https://msdn.microsoft.com/en-us/library/system.windows.controls.radiobutton_events(v=vs.110).aspx

changing what's on windows form: http://stackoverflow.com/questions/13584902/change-content-in-a-windows-form 

http://stackoverflow.com/questions/14956774/how-to-change-the-content-of-windows-form-with-button-control-click

Sounds like a set of radio buttons with on-click events that change visibility of panels on the form (turn off for all others, turn on for new one)

[Dobbsjr]

Unfortunately, my C# knowledge is rather basic, so all I can do is

https://msdn.microsoft.com/en-us/library/System.Windows.Forms.RadioButton(v=vs.110).aspx?cs-save-lang=1&cs-lang=csharp#code-snippet-1 

http://stackoverflow.com/questions/1797907/which-radio-button-in-the-group-is-checked

http://csharp.net-informations.com/gui/cs-radiobutton.htm 

on click event: https://msdn.microsoft.com/en-us/library/system.windows.controls.radiobutton_events(v=vs.110).aspx

changing what's on windows form: http://stackoverflow.com/questions/13584902/change-content-in-a-windows-form 

http://stackoverflow.com/questions/14956774/how-to-change-the-content-of-windows-form-with-button-control-click

Sounds like a set of radio buttons with on-click events that change visibility of panels on the form (turn off for all others, turn on for new one)

Thanks!