From e3aa6b9600a6972a684680e22e4f5c87042367c8 Mon Sep 17 00:00:00 2001
From: windyboy <windyboy@gmail.com>
Date: Mon, 29 Jul 2024 15:39:59 +0800
Subject: [PATCH] refactor: Update CNL pattern in config.go

---
 internal/config/config.go           | 12 +++++------
 internal/parsers/aviation_parser.go | 31 ++++++++++++++---------------
 2 files changed, 21 insertions(+), 22 deletions(-)

diff --git a/internal/config/config.go b/internal/config/config.go
index 6ec5fd0..7193f65 100644
--- a/internal/config/config.go
+++ b/internal/config/config.go
@@ -65,8 +65,8 @@ const (
 	// - arrival: the four-letter arrival airport code
 	arrPatternString = `^\((?P<category>[A-Z]{3})` +
 		`-(?P<number>[A-Z0-9]+)(\/?(?P<ssr>[A-Z0-9]+))?` +
-		`-(?P<departure>[A-Z]{4})` +
-		`-(?P<arrival>[A-Z]{4})(?P<time>\d{4})\)$`
+		`-(?P<dep>[A-Z]{4})` +
+		`-(?P<arr>[A-Z]{4})(?P<arr_time>\d{4})\)$`
 
 	// depPatternString represents the regular expression pattern used to match departure patterns.
 	// The pattern matches strings in the format: "(TYPE-NUMBER-SSR-DEPARTURE-DEPARTURE_TIME-ARRIVAL)".
@@ -77,7 +77,7 @@ const (
 	// - departure: the four-letter departure airport code
 	// - departure_time: the four-digit departure time
 	// - arrival: the four-letter arrival airport code
-	depPatternString = `^\((?P<category>[A-Z]{3})-(?P<number>[A-Z0-9]+)(\/(?P<ssr>[A-Z0-9]+))?-(?P<departure>[A-Z]{4})(?P<departure_time>\d{4})-(?P<arrival>[A-Z]{4})\)$`
+	depPatternString = `^\((?P<category>[A-Z]{3})-(?P<number>[A-Z0-9]+)(\/(?P<ssr>[A-Z0-9]+))?-(?P<dep>[A-Z]{4})(?P<dep_time>\d{4})-(?P<arr>[A-Z]{4})\)$`
 
 	// fplPatternString represents the regular expression pattern used to match flight plan patterns.
 	// The pattern matches strings in the format: "(CATEGORY-NUMBER-INDICATOR-AIRCRAFT-SURVE-DEPARTURE-DEPARTURE_TIME-SPEED-LEVEL-ROUTE-DESTINATION-ESTT-ALTER-OTHER)".
@@ -97,11 +97,11 @@ const (
 	// - alter: one or more sequences of a whitespace character followed by exactly four uppercase letters
 	// - other: any other relevant information, starting with three uppercase letters followed by a forward slash and zero or more characters (including newline)
 
-	fplPatternString = `\((?P<category>[A-Z]{3})-(?P<number>[A-Z]+\d+)-(?P<indicator>[A-Z]{2})\n-(?P<aircraft>[A-Z]+\d+\/?[A-Z]?)\n?-(?P<surve>.*)\n?-(?P<departure>[A-Z]{4})(?P<departure_time>\d{4})\n?-(?P<speed>[A-Z]+\d+)(?P<level>[A-Z0-9]+)\s+(?P<route>(.|\n)+)\n-(?P<destination>[A-Z]{4})(?P<estt>\d{4})\s?(?P<alter>(\s[A-Z]{4})+)\n?-([A-Z]{3}\/(?:[A-Z]{4}\d{4}\s?)+)?(?P<other>(?m)[A-Z]{3}\/(.|\n)*)\)$`
+	fplPatternString = `\((?P<category>[A-Z]{3})-(?P<number>[A-Z]+\d+)-(?P<indicator>[A-Z]{2})\n-(?P<aircraft>[A-Z]+\d+\/?[A-Z]?)\n?-(?P<surve>.*)\n?-(?P<dep>[A-Z]{4})(?P<dep_time>\d{4})\n?-(?P<speed>[A-Z]+\d+)(?P<level>[A-Z0-9]+)\s+(?P<route>(.|\n)+)\n-(?P<dest>[A-Z]{4})(?P<estt>\d{4})\s?(?P<alter>(\s[A-Z]{4})+)\n?-([A-Z]{3}\/(?:[A-Z]{4}\d{4}\s?)+)?(?P<other>(?m)[A-Z]{3}\/(.|\n)*)\)$`
 
-	cnlPatternString = `^\((?P<category>[A-Z]{3})-(?P<number>\w+\d+)-?(?P<departure>[A-Z]{4})?-?(?<arrival>[A-Z]{4})\)$`
+	cnlPatternString = `^\((?P<category>[A-Z]{3})-(?P<number>\w+\d+)-?(?P<dep>[A-Z]{4})?-?(?<arr>[A-Z]{4})\)$`
 
-	dlaPatternString = `^\((?P<category>[A-Z]{3})-(?P<number>\w+\d+)-?(?P<departure>[A-Z]{4})(?P<departure_time>\d{4})?-?(?<arrival>[A-Z]{4})(?<arrival_time>\d{4})?\)$`
+	dlaPatternString = `^\((?P<category>[A-Z]{3})-(?P<number>\w+\d+)-?(?P<dep>[A-Z]{4})(?P<dep_time>\d{4})?-?(?<arr>[A-Z]{4})(?<arr_time>\d{4})?\)$`
 )
 
 // Initialize the bodyPatterns map
diff --git a/internal/parsers/aviation_parser.go b/internal/parsers/aviation_parser.go
index 3201568..f5d4772 100644
--- a/internal/parsers/aviation_parser.go
+++ b/internal/parsers/aviation_parser.go
@@ -20,17 +20,16 @@ const (
 	CategoryArrival      = "ARR"
 	FlightNumber         = "number"
 	SSR                  = "ssr"
-	Departure            = "departure"
-	ArrivalLocation      = "arrival"
-	Time                 = "time"
+	DepartureCode        = "dep"
+	DepartureTime        = "dep_time"
+	ArrivalCode          = "arr"
+	ArrivalTime          = "arr_time"
 	CategoryDeparture    = "DEP"
-	DepartureTime        = "departure_time"
-	Destination          = "destination"
+	DestinationCode      = "dest"
 	OtherInfo            = "other"
 	CategoryCancellation = "CNL"
 	CategoryDelay        = "DLA"
 	NewDepartureTime     = "new_departure_time"
-	ArrivalTime          = "arrival_time"
 	CategoryFlightPlan   = "FPL"
 	ReferenceData        = "reference_data"
 	Aircraft             = "aircraft"
@@ -143,8 +142,8 @@ func (bp *BodyParser) createBodyData(data map[string]string) (string, interface{
 			Category:         data[Category],
 			AircraftID:       data[FlightNumber],
 			SSRModeAndCode:   data[SSR],
-			DepartureAirport: data[Departure],
-			ArrivalAirport:   data[ArrivalLocation],
+			DepartureAirport: data[DepartureCode],
+			ArrivalAirport:   data[ArrivalCode],
 			ArrivalTime:      data[ArrivalTime],
 		}, nil
 	case CategoryDeparture:
@@ -152,24 +151,24 @@ func (bp *BodyParser) createBodyData(data map[string]string) (string, interface{
 			Category:         data[Category],
 			AircraftID:       data[FlightNumber],
 			SSRModeAndCode:   data[SSR],
-			DepartureAirport: data[Departure],
+			DepartureAirport: data[DepartureCode],
 			DepartureTime:    data[DepartureTime],
-			Destination:      data[ArrivalLocation],
+			Destination:      data[ArrivalCode],
 		}, nil
 	case CategoryCancellation:
 		return category, &domain.CNL{
 			Category:           data[category],
 			AircraftID:         data[FlightNumber],
-			DepartureAirport:   data[Departure],
-			DestinationAirport: data[ArrivalLocation],
+			DepartureAirport:   data[DepartureCode],
+			DestinationAirport: data[ArrivalCode],
 		}, nil
 	case CategoryDelay:
 		return category, &domain.DLA{
 			Category:         data[Category],
 			AircraftID:       data[FlightNumber],
-			DepartureAirport: data[Departure],
+			DepartureAirport: data[DepartureCode],
 			NewDepartureTime: data[DepartureTime],
-			ArrivalAirport:   data[ArrivalLocation],
+			ArrivalAirport:   data[ArrivalCode],
 			ArrivalTime:      data[ArrivalTime],
 		}, nil
 	case CategoryFlightPlan:
@@ -182,10 +181,10 @@ func (bp *BodyParser) createBodyData(data map[string]string) (string, interface{
 			SSRModeAndCode:          data[Surveillance],
 			FlightRulesAndType:      data[Indicator],
 			CruisingSpeedAndLevel:   data[Speed] + data[Level],
-			DepartureAirport:        data[Departure],
+			DepartureAirport:        data[DepartureCode],
 			DepartureTime:           data[DepartureTime],
 			Route:                   data[Route],
-			DestinationAndTotalTime: data[Destination] + data[EstimatedTime],
+			DestinationAndTotalTime: data[DestinationCode] + data[EstimatedTime],
 			AlternateAirport:        data[AlternateAirport],
 			OtherInfo:               data[OtherInfo],
 			Register:                otherData[Register],