Skip to content

Initial Branch Commit - Modifications Needed!!!#1

Open
cfrantzidis wants to merge 1 commit intomasterfrom
2.-Add-Custom's-Behavior
Open

Initial Branch Commit - Modifications Needed!!!#1
cfrantzidis wants to merge 1 commit intomasterfrom
2.-Add-Custom's-Behavior

Conversation

@cfrantzidis
Copy link
Copy Markdown
Owner

Your code has no comments.
There is no added functionality.

@cfrantzidis cfrantzidis added bug Something isn't working documentation Improvements or additions to documentation labels Feb 9, 2025
Copy link
Copy Markdown
Owner Author

@cfrantzidis cfrantzidis left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Needs major modifications

Comment thread Program.cs
Console.WriteLine("Press any key to exit...");
Console.ReadKey();
}
void MyFirstMethod()
Copy link
Copy Markdown
Owner Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

There is no functionality inside that method.
The method's name is not informative.
Please create comments.

Comment thread Program.cs
{

}
void MySecondMethod()
Copy link
Copy Markdown
Owner Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
void MySecondMethod()
static double CalculateBMI(double weight, double height){
return weight / height * height;
}

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working documentation Improvements or additions to documentation

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant