fix: better error handling techniques

codescalersinternships · Sep 11, 2024 · 5f01597 · 5f01597
1 parent 73e5e03
commit 5f01597
Showing 1 changed file with 4 additions and 5 deletions.
diff --git a/pkg/iniparser.go b/pkg/iniparser.go
@@ -7,7 +7,6 @@ import (
 	"errors"
 	"fmt"
 	"os"
-	"reflect"
 	"sort"
 	"strings"
 )
@@ -130,10 +129,10 @@ func (i iniParser) GetSections() (map[string]section, error) {
 //	else --> nil
 func (i iniParser) Get(sectionName string, key string) (string, error) {
 
-	if reflect.DeepEqual(i.sections[sectionName], section{}) {
+	if _, ok := i.sections[sectionName]; !ok {
 		return "", errors.New("section not found")
 	}
-	if i.sections[sectionName].map_[key] == "" {
+	if _, ok := i.sections[sectionName].map_[key]; !ok {
 		return "", errors.New("key not found")
 	}
 	return i.sections[sectionName].map_[key], nil
@@ -148,10 +147,10 @@ func (i iniParser) Get(sectionName string, key string) (string, error) {
 //	If the key passed isn't found in the passed section --> "key not found"
 //	else --> nil
 func (i *iniParser) Set(sectionName string, key string, value string) error {
-	if reflect.DeepEqual(i.sections[sectionName], section{}) {
+	if _, ok := i.sections[sectionName]; !ok {
 		return errors.New("section not found")
 	}
-	if i.sections[sectionName].map_[key] == "" {
+	if _, ok := i.sections[sectionName].map_[key]; !ok {
 		return errors.New("key not found")
 	}
 	i.sections[sectionName].map_[key] = value