Add go-style error handling

Author: mikehquan19

parser/cometCalendarParser.go

@@ -152,7 +152,11 @@ func ParseCometCalendar(inDir string, outDir string) {
 	}
 
 	multiBuildingMap := make(map[string]map[string]map[string][]schema.Event)
-	buildingAbbreviations, validAbbreviations := getLocationAbbreviations(inDir)
+	// Some events have only the building name, not the abbreviation
+	buildingAbbreviations, validAbbreviations, err := getLocationAbbreviations(inDir)
+	if err != nil {
+		panic(err)
+	}
 
 	for _, event := range allEvents {
 
@@ -248,7 +252,7 @@ func ParseCometCalendar(inDir string, outDir string) {
 }
 
 // getAbbreviations dynamically retrieves the all of the locations abbreviations
-func getLocationAbbreviations(inDir string) (map[string]string, []string) {
+func getLocationAbbreviations(inDir string) (map[string]string, []string, error) {
 	// Get the locations from the map scraper
 	var mapFile []byte
 
@@ -264,20 +268,21 @@ func getLocationAbbreviations(inDir string) (map[string]string, []string) {
 			// If fail to get the locations again, it's not because location is unscraped
 			mapFile, err = os.ReadFile(inDir + "/mapLocations.json")
 			if err != nil {
-				panic(err)
+				return nil, nil, err
 			}
 		} else {
-			panic(err)
+			return nil, nil, err
 		}
 	}
+
 	var locations []schema.MapBuilding
 	if err = json.Unmarshal(mapFile, &locations); err != nil {
-		panic(err)
+		return nil, nil, err
 	}
 
 	// Process the abbreviations
-	buildingsAbbreviations := make(map[string]string, 0)
-	validAbbreviations := make([]string, 0)
+	buildingsAbbreviations := make(map[string]string, 0) // Maps building names to their abbreviations
+	validAbbreviations := make([]string, 0)              // Valid building abreviations for checking
 
 	for _, location := range locations {
 		// Trim the following acronym in the name
@@ -292,5 +297,5 @@ func getLocationAbbreviations(inDir string) (map[string]string, []string) {
 		validAbbreviations = append(validAbbreviations, abbreviation)
 	}
 
-	return buildingsAbbreviations, validAbbreviations
+	return buildingsAbbreviations, validAbbreviations, nil
 }

scrapers/cometCalendar.go

@@ -131,7 +131,8 @@ func callAndUnmarshal(client *http.Client, page int, data *APICalendarResponse)
 // getTime parses the start and end time of the event
 func getTime(event RawEvent) (time.Time, time.Time) {
 	instance := convert[map[string]any](
-		convert[map[string]any](convert[[]any](event.Event["event_instances"])[0])["event_instance"])
+		convert[map[string]any](
+			convert[[]any](event.Event["event_instances"])[0])["event_instance"])
 
 	// Converts RFC3339 timestamp string to time.Time
 	startTime, err := time.Parse(time.RFC3339, convert[string](instance["start"]))
@@ -163,28 +164,28 @@ func getEventLocation(event RawEvent) string {
 
 // getFilters parses the types, topics, and target audiences
 func getFilters(event RawEvent) ([]string, []string, []string) {
-	eventTypes := []string{}
-	targetAudiences := []string{}
-	eventTopics := []string{}
+	types := []string{}
+	audiences := []string{}
+	topics := []string{}
 
 	filters := convert[map[string]any](event.Event["filters"])
 
 	rawTypes := convert[[]any](filters["event_types"])
 	for _, rawType := range rawTypes {
-		eventTypes = append(eventTypes, convert[string](convert[map[string]any](rawType)["name"]))
+		types = append(types, convert[string](convert[map[string]any](rawType)["name"]))
 	}
 
 	rawAudiences := convert[[]any](filters["event_target_audience"])
 	for _, audience := range rawAudiences {
-		targetAudiences = append(targetAudiences, convert[string](convert[map[string]any](audience)["name"]))
+		audiences = append(audiences, convert[string](convert[map[string]any](audience)["name"]))
 	}
 
 	rawTopics := convert[[]any](filters["event_topic"])
 	for _, topic := range rawTopics {
-		eventTopics = append(eventTopics, convert[string](convert[map[string]any](topic)["name"]))
+		topics = append(topics, convert[string](convert[map[string]any](topic)["name"]))
 	}
 
-	return eventTypes, targetAudiences, eventTopics
+	return types, audiences, topics
 }
 
 // getDepartmentsAndTags parses the departments, and tags