[
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
|
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:
https://bugs.eclipse.org/bugs/show_bug.cgi?id=417961
Regards,
Robin
> On 2013-09-22, at 7:36 AM, Robin Stocker <robin@xxxxxxxxx> wrote:
>
> >> http://www.javacodegeeks.com/2013/09/why-we-dropped-eclipse-in-favour-of-intellij.html
> >
> > 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