Java - Instantiation

Question

我正在尝试创建一个我不完全确定类类型的新类。这可以用我的代码更好地解释:

private static Class[] Packets = new Class[]
            {
                KeepAlivePacket.class, // 0x00
                LoginRequestPacket.class, // 0x01
                HandshakePacket.class, // 0x02
                    }
.......
class HandshakePacket extends TCPPacket
{
    public HandshakePacket()
    {
    }
    byte protocolVersion;
    String username;
    String host;
    int port;
    @Override
    public void writePacketData(DataOutputStream os) throws IOException {
        os.write(id);
        os.writeByte(protocolVersion);
        writeString(os, username);
        writeString(os, host);
        os.writeInt(port);
    }
    @Override
    public void readPacketData(DataInputStream is) throws IOException {
        protocolVersion = is.readByte();
        username = readString(is,16);
        host = readString(is,16);
        port = is.readInt();
    }
    @Override
    public void setId(byte id)
    {
        this.id = id;
    }
}
.......
    public static TCPPacket getNewPacket(int i)
    {
    try
    {
        Class var1 = (Class)Packets[i];
        return var1 == null ? null : (TCPPacket)var1.newInstance(); <-- error on this line
    }
    catch (Exception var2)
    {
        var2.printStackTrace();
        System.out.println("Skipping packet with id " + i);
        return null;
    }
}

对于那些想知道TCPPacket是什么的人:

package vc.voidwhisperer.proxy.packet;
import java.io.DataInputStream;
import java.io.DataOutputStream;
import java.io.IOException;
public class TCPPacket {
    public TCPPacket()
    {
    }
    public byte id = 0;
    public void writePacketData(DataOutputStream os) throws IOException
    {
    }
    public void readPacketData(DataInputStream is) throws IOException
    {
    }
    public void setId(byte id)
    {
    }
}

正如你所看到的，我正在尝试实例化一个新对象，我不能完全确定类类型是什么。然而，它抛出了这个异常:

java.lang.InstantiationException: vc.voidwhisperer.proxy.packet.Packet$HandshakePacket
at java.lang.Class.newInstance0(Unknown Source)
at java.lang.Class.newInstance(Unknown Source)
at vc.voidwhisperer.proxy.packet.Packet.getNewPacket(Packet.java:2509)
at vc.voidwhisperer.proxy.UserConnection.run(UserConnection.java:52)

Answer 1

反射对于这个来说是多余的。

只做

switch (i) {
  case 0: return new KeepAlivePacket();
  case 1: return new LoginRequestPacket();
  case 2: return new HandshakePacket();
  default: throw new IllegalArgumentException();
}

，理想情况下用enum代替i。

这将使您获得静态类型和签名检查的优势，使您的代码更具可维护性，并避免所有掩盖异常的反射废话。

Answer 2

您正在尝试从静态方法中实例化非静态内部类。创建一个新的HandshakePacket对象需要一个周围的Packet对象作为它的父对象，而您没有提供一个。

因此，要么使HandshakePacket成为一个静态内部类，使getNewPacket成为一个非静态方法，或者创建一个新的Packet对象作为您的新HandshakePacket对象的父对象。