Skip to main content

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index] [List Home]
Re: [recommenders-dev] [SnipMatch] Search result display's parameters solution

One other thing: I don't see a search box in that image - please
confirm that it's just missing from that screenshot :) We definitely
need one.

On Thu, Apr 5, 2012 at 11:18 AM, Doug Wightman <douglasw@xxxxxxxxx> wrote:
> Cheng,
>
> I also like the idea of removing that dialog box, so long as doing so
> doesn't cripple the functionality. _If_ parameters are included in the
> search box (rather than entered in the parameter dialog box), will
> they still be used to modify the inserted code?
>
> This might be a reasonable compromise. When parameters aren't
> specified in the search box (e.g. the user types "print" instead of
> "print hello"), it inserts defaults, and those defaults can be
> selected using the Eclipse Templates-style form filling code. However,
> when parameters are included (e.g. "print hello"), the inserted code
> _does_ include them (e.g. System.out.println("hello");). What do you
> think?
>
> Parameters are a critical component of SnipMatch. We wrote a ton of
> code to support parameters (not just strings, also local variables).
> This is one of the major advantages of this system (i.e. significantly
> fewer characters need to be typed).
>
> Doug
>
>
> On Thu, Apr 5, 2012 at 9:23 AM, Chen Cheng <chengchendoc@xxxxxxxxx> wrote:
>> Hi all,
>>
>> We have to decide search result display's parameters solution now. Here is
>> my personally idea:
>>
>> As i am improving the search result display module, want it to be similar
>> with JDT's content assist, but Eclipse content assist does not have any
>> "parameters input" dialog, so i want to remove the parameters dialog in
>> SnipMatch, or it will looks strange, and does not match Eclipse users'
>> habit. Meanwhile:
>>
>> 1. Use default parameter values for inserted code snip, this is how JDT's
>> content assist work. For example, there is such a method in class:
>>
>>     public static void test1(int intValue, String str, double doubleValue){
>> //Do anything here
>>    }
>>
>> We pop up content assist in main method, we can find method test1 in content
>> assist's list(see attach file a.png), then, if we select test1, it will
>> insert following code into current place:
>>
>> test1(intValue, str, doubleValue)
>>
>> See, JDT's content assist just use default value for the parameters(usually
>> parameters used to define the method), there isn't any parameters input
>> dialog. This is how Eclipse usually does. I think SnipMatch should also
>> follow this rule.
>>
>> 2. The dummy parameters inserted into editor are usually "string" user
>> defined when they create the json format code snip file, but user can change
>> them in code snip edit panel( I will improve SnipMatch's code snip edit
>> panel, this part is in GSoC's proposal).
>>
>> Your ideas ?
>>
>> --
>> Best Regards From Cheng Chen [chengchendoc@xxxxxxxxx]
>>
>> _______________________________________________
>> recommenders-dev mailing list
>> recommenders-dev@xxxxxxxxxxx
>> http://dev.eclipse.org/mailman/listinfo/recommenders-dev
>>


Back to the top