How do I use the codes below and refactor them into 4 different functions and ask atleast 1 function to return value to Main()?

Question

1.00/5 (1 vote)

See more:

C#

class Program
{
    static void Main(string[] args)
    {
        double tribase = 9;
        double triwidth = 6;
        double trihypotenuse = Math.Sqrt((tribase * tribase) + (triwidth * triwidth));
        double triarea = 0.5 * (tribase * triwidth);
        double triperimeter = tribase + triwidth + trihypotenuse;
        Console.WriteLine("Area of right triangle = {0}\nPerimeter of rigth triangle = {1}\n", triarea, triperimeter);

        double squareside = 8;
        double squarearea = squareside * squareside;
        double squareperimeter = 4 * squareside;
        Console.WriteLine("Area of square = {0}\nPerimeter of square = {1}\n",  squarearea, squareperimeter);
    }
}

Posted 23-Feb-14 7:01am

RalvarezHose

Add a Solution

Comments

CHill60 23-Feb-14 13:05pm

This sounds like homework. Which bits do *you* think could go into a separate function?

RalvarezHose 23-Feb-14 13:10pm

Yes, it was one of my homework question, it is now 3 weeks past due, got only half a point. Professor told me to use any resource and get it right and she will give me couple of extra points. So far I have got following:

static double GetSquareArea(double squareside)
{
double squarearea = squareside * squareside;
return squarearea;
}
static double GetSquarePerimeter(double squareside)
{
double squareperimeter = 4 * squareside;
return squareperimeter;

}
No idea how to do perimeter and area of right triangle in different functions.

george4986 23-Feb-14 13:22pm

are u using visual studio for coding?

RalvarezHose 23-Feb-14 13:25pm

Yes, I am using visual studio 2013 express for desktop.

george4986 23-Feb-14 13:34pm

for perimeter select these lines and follow steps

double tribase = 9;
double triwidth = 6;
double trihypotenuse = Math.Sqrt((tribase * tribase) + (triwidth * triwidth));
double triperimeter = tribase + triwidth + trihypotenuse;

george4986 23-Feb-14 13:29pm

in the code section select lines

double tribase = 9;
double triwidth = 6;
double triarea = 0.5 * (tribase * triwidth);

right click mouse, on the right click menu select 'refactor', on submenu select 'extract method' , type a name for function select ok

RalvarezHose 23-Feb-14 13:34pm

Thank you so much.

george4986 23-Feb-14 13:35pm

there are so many inbuilt features there .try them , u can learn the syntax too as a starter

CHill60 23-Feb-14 13:41pm

It's good to use the tools that are available to you, but is your teacher trying to teach you how to use the Visual Studio IDE or how to refactor - one is a tool, the other is a mind set. Make sure you watch what happens in case you need to explain it to your tutor. You'll learn more that way too

BillWoodruff 23-Feb-14 15:13pm

I think (I guess) your teacher is trying to get you to understand how to break a problem into modules that are independent, and that you can call from any program reliably. Well, that's my interpretation of the terms "refactor," and "return a value" in this context.

I suggest you think about how to break your program into separate tasks here. You can think of three major tasks: get input, perform calculations, present results. And, you can break the "calculations" task into sub-tasks: "calculate right triangle area;" "calculate right triangle perimeter, etc.

Each "calculation task" you can write as a method that takes two values as input, and returns a result of Type 'double.

The important thing is for you to develop understanding !

1 solution

Add a Solution

Add your solution here

Treat my content as plain text, not as HTML

Preview 0

…

Existing Members

Sign in to your account

...or Join us

Download, Vote, Comment, Publish.

Your Email
Password
Forgot your password?

Your Email
This email is in use. Do you need your password?
Optional Password

I have read and agree to the Terms of Service and Privacy Policy
Please subscribe me to the CodeProject newsletters

When answering a question please:

Read the question carefully.
Understand that English isn't everyone's first language so be lenient of bad spelling and grammar.
If a question is poorly phrased then either ask for clarification, ignore it, or edit the question and fix the problem. Insults are not welcome.
Don't tell someone to read the manual. Chances are they have and don't get it. Provide an answer or move on to the next question.

Let's work to help developers, not make them feel stupid.

This content, along with any associated source code and files, is licensed under The Code Project Open License (CPOL)

CHill60 · Accepted Answer · 2014-02-23T07:38:00

You are so close already...
This is the code you have already put into functions yourself...

C#

static double GetSquareArea(double squareside)
    {
        double squarearea = squareside * squareside;
        return squarearea;
    }
        static double GetSquarePerimeter(double squareside)
    {
        double squareperimeter = 4 * squareside;
        return squareperimeter;

        }

So let's revisit the code you have to refactor and see what we have left...the bits in bold are the bits that change because we now have your two functions ...

C#

class Program
{
    static void Main(string[] args)
    {
        double tribase = 9;
        double triwidth = 6;
        double trihypotenuse = Math.Sqrt((tribase * tribase) + (triwidth * triwidth));
        double triarea = 0.5 * (tribase * triwidth);
        double triperimeter = tribase + triwidth + trihypotenuse;
        Console.WriteLine("Area of right triangle = {0}\nPerimeter of rigth triangle = {1}\n", triarea, triperimeter);

        double squareside = 8;
        double squarearea = GetSquareArea(squareside);
        double squareperimeter =  GetSquarePerimeter(squareside);
        Console.WriteLine("Area of square = {0}\nPerimeter of square = {1}\n",  squarearea, squareperimeter);
    }
}

So do the same with the perimeter of a triangle ... the bits in bold need to go out into a separate function into which you will pass the values of tribase and triwidth

C#

class Program
{
    static void Main(string[] args)
    {
        double tribase = 9;
        double triwidth = 6;
        double trihypotenuse = Math.Sqrt((tribase * tribase) + (triwidth * triwidth));
        double triarea = 0.5 * (tribase * triwidth);
        double triperimeter = tribase + triwidth + trihypotenuse;
        Console.WriteLine("Area of right triangle = {0}\nPerimeter of rigth triangle = {1}\n", triarea, triperimeter);

        double squareside = 8;
        double squarearea = GetSquareArea(squareside);
        double squareperimeter =  GetSquarePerimeter(squareside);
        Console.WriteLine("Area of square = {0}\nPerimeter of square = {1}\n",  squarearea, squareperimeter);
    }
}

Notice that trihypotenuse is an interim calculation to get to the perimeter ... you don't need it anywhere else in Main so that needs to go into the function as well.
Have a go at it and post your attempt as a comment to this solution and I'll tell you if you've got it right