Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

This Challenge is about Reflection #44

Open
wants to merge 4 commits into
base: master
Choose a base branch
from

Conversation

Massi-Amb
Copy link

I created a challenge about Reflection. Let me know if it looks good sir!

@kevmoens
Copy link
Owner

I'm looking for a test that show knowledge of using the System.Relection namespace.
Examples of this would include examining a class for properties, methods, constructors and fields. They maybe private or public. They can also be instance or static. Also handling overloaded methods and constructors.

@kevmoens
Copy link
Owner

This is what I am looking for but the problem is your sample doesn't have a way for the user to code one thing and have it pass multiple tests. Maybe you make two separate challenges. The first challenge you have a second class that you have 3 different methods with the same parameter signature. Then the method the user codes you pass in a string for the method name.
At that point make a second challenge where you have a second class that has multiple methods with the same name but with different signatures.
Also revert your changes of anything not related to the reflection tests.

Massi-Amb and others added 2 commits February 19, 2023 17:25
Added 2 Challenges, 1 to  call property and 1 to call method by using Reflection.
Also, I reverted  the other challenge that has nothing to do with Reflection.
@kevmoens
Copy link
Owner

kevmoens commented Mar 6, 2023

This is awesome and what I wanted. But the code I looked at didn't look like it would work. Test it out and see if you can get all three to pass.

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