Skip to content

امین فیروزی-HW-L2-02-InitialFirstConsoleAppPart2 - #1

Open
AminFirouzi wants to merge 5 commits into
mainfrom
AminFirouzi
Open

امین فیروزی-HW-L2-02-InitialFirstConsoleAppPart2#1
AminFirouzi wants to merge 5 commits into
mainfrom
AminFirouzi

Conversation

@AminFirouzi

Copy link
Copy Markdown
Owner

امین فیروزی-HW-L2-02-InitialFirstConsoleAppPart2

@parsapanahpoor

Copy link
Copy Markdown

Code Review – InitialFirstConsoleAppPart2

Student: Amin Firouzi
Assignment: HW-L2-02 – InitialFirstConsoleAppPart2

Strengths

Tasks 3 and 4 are implemented well using top-level statements and inline class definitions. Task 3 correctly uses ref and out to modify the second number and produce the third. Task 4 includes empty-array handling, proper max/average calculation, and formatted average output. Separate Student and BankAccount classes exist with appropriate properties and methods.

Weaknesses / Mistakes

The Tasks1_2 project only prints “Hello, World!” and does not instantiate or demonstrate the Student or BankAccount classes. Tasks 1 and 2 are therefore not shown working at runtime. Task 2’s ShowBalance is never called in a demo sequence in that project.

Suggestions for Future Work

Wire up Tasks 1 and 2 in Program.cs: create a Student, call Introduce(), and run deposit/withdraw/show balance on a BankAccount. Submit one cohesive solution or clearly runnable projects for every task. Tasks 3 and 4 set a good standard—match that completeness for Tasks 1 and 2.

Rating: 6.5/10

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants