On Wed, 19 Mar 2008 03:11:33 -0700 (PDT), "Luca D."
<ldantona@gmail.com> wrote, quoted or indirectly quoted someone who
said :

why not just append your ConcreteVehicle1 methods to the VEHICLE1
enum?  Then you don't even need a getType method.

The above is riddled with bugs.

1) Maybe you could use vehicle.getClass().getName() instead?

2) The need to do so suggests the design is poor. Whatever is happening
in the 'if' statements, should perhaps be replaced by an instance method
of AbstractVehicle that the subtypes implement (or override) differently.

Instead of
       if(vehicle.GetType() == VehicleType.VEHICLE1)
           ...
       else if(vehicle.GetType() == VehicleType.VEHICLE2)
           ...
       ...
Maybe you can do
       vehicle.dotDotDot();

Here's a corrected SSCCE based on your buggy code above:
---------------------------------------8<---------------------------------
public class InheritTest1 {
    public static void main(String[] args) {
        InheritTest1 x = new InheritTest1();
        x.new VehicleManager().doSomething(x.new ConcreteVehicle1());
    }

    public enum VehicleType {
        VEHICLE1, VEHICLE2
    }

    public abstract class AbstractVehicle {
        public abstract VehicleType getType();
    }

    public class ConcreteVehicle1 extends AbstractVehicle {
        public VehicleType getType() {
            return VehicleType.VEHICLE1;
        }
    }

    public class ConcreteVehicle2 extends AbstractVehicle {
        public VehicleType getType() {
            return VehicleType.VEHICLE2;
        }
    }

    public class VehicleManager {
        public void doSomething(AbstractVehicle vehicle) {
            if (vehicle.getType() == VehicleType.VEHICLE1)
                System.out.println("Aha!");
            else if (vehicle.getType() == VehicleType.VEHICLE2)
                System.out.println("Oho!");
        }
    }

}
---------------------------------------8<---------------------------------
The main is a bit convoluted just so I can wrap everything into a single
class for ease of cut&paste&compile.

Here's another (better?) way to do achieve the same outcome:
---------------------------------------8<---------------------------------

public class InheritTest2 {
    public static void main(String[] args) {
        InheritTest2 x = new InheritTest2();
        x.new VehicleManager().doSomething(x.new ConcreteVehicle1());
    }

    public abstract class AbstractVehicle {
        public abstract void doSomething();
    }

    public class ConcreteVehicle1 extends AbstractVehicle {
        public void doSomething() {
            System.out.println("Aha!");
        }
    }

    public class ConcreteVehicle2 extends AbstractVehicle {
        public void doSomething() {
            System.out.println("Oho!");
        }
    }

    public class VehicleManager {
        public void doSomething(AbstractVehicle vehicle) {
            vehicle.doSomething();
        }
    }

}
---------------------------------------8<---------------------------------
Whether the above meets your needs is something I can't determine from
the example you posted. I think it illustrates a better way to think
about such problems but there will be some needs that can't be handled
this way.

Much better :-)

I'd have used Car and Truck instead of ConcreteVehicle1 and
ConcreteVehicle2 but maybe I have too much difficulty with abstract
expressions of concepts.