Skip to main content

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index] [List Home]
Re: [ide-dev] Why we dropped Eclipse in favour of IntelliJ | Java Code Geeks

Ok guys, I didn't say it was a *satisfying* solution, just *a* solution… ;)

On 2013-09-24, at 1:20 PM, Robin Stocker <robin@xxxxxxxxx> wrote:

> Miles Parker wrote:
>> To be fair, you can simply exclude java.awt.* from imports
>> altogether..
> Type filters are not a satisfying solution for various reasons:
> * The user needs to know about them and where to configure it. You
>  already lost most of the users with this.
> * It doesn't work when your project also needs to use AWT, SWT
>  (org.eclipse.swt.widgets.List) or other libraries including a List
>  type - you would still have to manually select the correct one.
> * There are more cases than java.awt.List.
> Here's the bug report for JDT:
> Regards,
>  Robin
>> On 2013-09-22, at 7:36 AM, Robin Stocker <robin@xxxxxxxxx> wrote:
>>> The problem with List is still there. Having "Organize Imports" as
>>> save action, type the following:
>>>   List<String> l = new ArrayList<String>();
>>> Now save. What happens: ArrayList is automatically imported because
>>> there is only one possibility. Good.
>>> But List is not. Now invoke quick fix on it. The proposals are:
>>> 1. Import 'List' (java.awt)
>>> 2. Import 'List' (java.util)
>>> Not helpful. Just for fun, select the first entry. The result
>>> is an error, because java.awt.List is not generic.
>>> Even if it was generic, it would still be an error, because
>>> ArrayList is not a subtype of java.awt.List.
>>> So the right thing to do in this case would be to just import
>>> the only valid possibility, java.util.List.
>>> IntelliJ probably gets this right, and I can imagine it being
>>> one of the reasons people prefer it.
>>> Can we really not do any better?
>>> I know about Code Recommenders, but last time I checked, it
>>> only reordered JDT's proposals. IMO the proposals leading to
>>> errors should not be proposed in the first place so that the
>>> extra key presses for invoking quick fix are not necessary.
>>> I would be surprised if the above was not already in Bugzilla
>>> somewhere, but I couldn't find it right away. I'll report it
>>> in case it's really not yet known.
>>> Regards,
>>> Robin Stocker
> _______________________________________________
> ide-dev mailing list
> ide-dev@xxxxxxxxxxx

Back to the top