Packet.cs is extremely long and has a lot of potential to refactor out smaller pieces of code. Logic around handling specific fields can and should be abstracted out of the Packet class. This would make development and testing much easier and perhaps make the file less intimidating to new developers.
Examples of opportunity here include basically all of the Handle... or Decode... methods in to various types: InformationField, MaidenheadGrid, etc.
Acceptance Criteria
[ ] Find potential refactors
[ ] Evaluate and make decisions on if refactors should be applied (be prepared to justify decision in PR or in this issue discussion)
Description
Packet.csis extremely long and has a lot of potential to refactor out smaller pieces of code. Logic around handling specific fields can and should be abstracted out of thePacketclass. This would make development and testing much easier and perhaps make the file less intimidating to new developers.Examples of opportunity here include basically all of the
Handle...orDecode...methods in to various types:InformationField,MaidenheadGrid, etc.Acceptance Criteria