Optimize quickfixType Function for Performance & Readability

[SahibYar]

Key Optimizations:

✅ Replaces fallthrough with map lookups • Instead of a long switch with repeated fallthrough statements, we use map[string]bool. • This makes lookups O(1) instead of scanning all cases in O(n) time.

✅ Improves Readability • Grouping similar types into separate maps makes the function easier to read and maintain. • The switch statement is shorter and clearer.

✅ Eliminates Unnecessary Variables • Returns values directly instead of assigning to quickfixType. • Instead of assigning prototype inside a switch, we fetch it directly from the map. • If the type doesn’t exist in fieldTypeMapping, the function exits early.

[skrater]

Can you provide any benchmarks?