Generated read() should just switch directly on field.id

THRIFT-623 introduced a performance regression in the Java bindings.

Instead of mapping field.id to an enum, testing if the enum is null, and then switching on the enum, the switch should just work directly on field.id, with a default case to handle unknown values.

In some benchmarks I ran, this fix saved 13% on deserialization time.