Encapsulate parameters in (parameter-) class (11019) extension, maybe?

http://www.intellij.net/tracker/resharper/viewSCR?publicId=11019

This is a great feature, I see it is available in the forthcoming 211.

I'd like to discuss a an addition to the feature, before I submit them as a
request...

Extend parameter class - i.e., if I have
void Func(FuncParams funcParams) {...}

and I change it to

void Func(FuncParams funcParams, int anotherInt) {...}

I'd like a context action that will add anotherInt to FuncParams.

Or do you feel that it is not needed because I can just manipulate the
FuncParams class directly...




6 comments
Comment actions Permalink

Hello Amir,

Great suggestion. Maybe it should be an option for the Create Class from
Parameters Refactoring to move parameters to one of the parameter types instead
of creating a new class. I'm not sure whether we are be able to include either
a context action or refactoring change into 2.0 because of the feature freeze
that will probably follow build 211, but we definitely need to discuss this.

http://www.intellij.net/tracker/resharper/viewSCR?publicId=11019

This is a great feature, I see it is available in the forthcoming 211.

I'd like to discuss a an addition to the feature, before I submit them
as a request...

Extend parameter class - i.e., if I have
void Func(FuncParams funcParams) {...}
and I change it to

void Func(FuncParams funcParams, int anotherInt) {...}

I'd like a context action that will add anotherInt to FuncParams.

Or do you feel that it is not needed because I can just manipulate the
FuncParams class directly...

Thanks,
Andrey Simanovsky


0
Comment actions Permalink

Should I add this to the tracker?

"Andrey Simanovsky (JetBrains)" <ands@intellij.com> wrote in message
news:c8a8a15de1718c7c89df70e6879@news.intellij.net...

Hello Amir,

>

Great suggestion. Maybe it should be an option for the Create Class from
Parameters Refactoring to move parameters to one of the parameter types
instead of creating a new class. I'm not sure whether we are be able to
include either a context action or refactoring change into 2.0 because of
the feature freeze that will probably follow build 211, but we definitely
need to discuss this.

>
>> http://www.intellij.net/tracker/resharper/viewSCR?publicId=11019
>>
>> This is a great feature, I see it is available in the forthcoming 211.
>>
>> I'd like to discuss a an addition to the feature, before I submit them
>> as a request...
>>
>> Extend parameter class - i.e., if I have
>> void Func(FuncParams funcParams) {...}
>> and I change it to
>>
>> void Func(FuncParams funcParams, int anotherInt) {...}
>>
>> I'd like a context action that will add anotherInt to FuncParams.
>>
>> Or do you feel that it is not needed because I can just manipulate the
>> FuncParams class directly...
>>

Thanks,
Andrey Simanovsky

>



0
Comment actions Permalink

Hello Amir,

Should I add this to the tracker?

Perhaps, it is better to wait a while for more feedback.

Andrey Simanovsky.


0
Comment actions Permalink

Will do...
"Andrey Simanovsky (JetBrains)" <ands@intellij.com> wrote in message
news:c8a8a15de17a8c7c8a2f155bd57@news.intellij.net...

Hello Amir,

>
>> Should I add this to the tracker?

Perhaps, it is better to wait a while for more feedback.

>

Andrey Simanovsky.

>



0
Comment actions Permalink

+1, Very nice feature, I'm going to use this a lot...

Andrey Simanovsky (JetBrains) wrote:

Hello Amir,

>> Should I add this to the tracker?


Perhaps, it is better to wait a while for more feedback.

Andrey Simanovsky.

0
Comment actions Permalink

Good enough... I'll add it to the tracker...
"Wiebe Tijsma" <wiebeREMOVE@CAPITALStijsma.com> wrote in message
news:dn9ijc$tr1$1@is.intellij.net...

+1, Very nice feature, I'm going to use this a lot...

>

Andrey Simanovsky (JetBrains) wrote:

>> Hello Amir,
>>
>>> Should I add this to the tracker?
>>
>> Perhaps, it is better to wait a while for more feedback.
>>
>> Andrey Simanovsky.
>>

0

Please sign in to leave a comment.