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

Add refactoring: Initialize field from constructor #13

Open
justcla opened this issue Jul 8, 2016 · 7 comments
Open

Add refactoring: Initialize field from constructor #13

justcla opened this issue Jul 8, 2016 · 7 comments
Assignees

Comments

@justcla
Copy link
Owner

justcla commented Jul 8, 2016

No description provided.

@justcla
Copy link
Owner Author

justcla commented Jul 8, 2016

Loren will look into this.

@lorenpaulsen
Copy link
Collaborator

See the Roslyn Quoter here, to learn the Roslyn syntax: http://roslynquoter.azurewebsites.net/

@lorenpaulsen
Copy link
Collaborator

Now modifying the existing constructor if one exists:

2016-07-20_00-05-21

@lorenpaulsen
Copy link
Collaborator

I just cleaned up the code, added some more comments, and pushed my branch

@lorenpaulsen
Copy link
Collaborator

I'll keep testing and hammering out edge cases though

@justcla
Copy link
Owner Author

justcla commented Jul 20, 2016

Great work Loren. Thank!

Suggestion: What about putting the new field assignment as the first line of the constructor instead of the last line? (Not sure)
If it was smart enough, it would jump over the first X number of parameters if they are defined at the top. But otherwise, it would be more useful at the top than the bottom. Just a thought.

@justcla
Copy link
Owner Author

justcla commented Jul 20, 2016

Which branch did you push? I can't see it.

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

No branches or pull requests

2 participants