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

Refactoring: Implement constructor body #6787

Closed
dpoeschl opened this issue Nov 15, 2015 · 3 comments
Closed

Refactoring: Implement constructor body #6787

dpoeschl opened this issue Nov 15, 2015 · 3 comments
Labels
Area-IDE Feature Request Resolution-Fixed The bug has been fixed and/or the requested behavior has been implemented
Milestone

Comments

@dpoeschl
Copy link
Contributor

Today we have "generate constructor" where you highlight properties/fields and we offer to generate a constructor that accepts values for and populates those members, and we also have Generate Method which has custom constructor handling of assigning the parameters to fields. We should offer another variation -- the user writes an empty constructor definition and we offer to generate the backing fields and constructor body.

The only way to have it generate this boilerplate such that the constructor's parameters are in the desired order is to create a call to the to-be-generated constructor, which is not always convenient.

(From a conversation with @matthejo)

@Pilchie Pilchie added this to the 1.2 milestone Nov 17, 2015
@Pilchie Pilchie self-assigned this Nov 17, 2015
@Pilchie Pilchie removed this from the 1.2 milestone Jan 31, 2016
@Pilchie Pilchie removed their assignment Jan 31, 2016
@Pilchie Pilchie added this to the Unknown milestone Jan 31, 2016
@Pilchie
Copy link
Member

Pilchie commented Jun 12, 2017

@dpoeschl Do you think @CyrusNajmabadi new fixes around adding fields for parameters and null checks covers this?

@dpoeschl
Copy link
Contributor Author

Probably, yes. I'll run through it with @matthejo to confirm.

@CyrusNajmabadi
Copy link
Member

This is available now:

image

as well as:

image

@sharwell sharwell added Resolution-Fixed The bug has been fixed and/or the requested behavior has been implemented and removed 0 - Backlog labels Mar 2, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Area-IDE Feature Request Resolution-Fixed The bug has been fixed and/or the requested behavior has been implemented
Projects
None yet
Development

No branches or pull requests

4 participants