refactor: use if-else instead of switch for bool

loeffel-io · Dec 11, 2024 · ffd62fd · ffd62fd
1 parent 61ffb56
commit ffd62fd
Show file tree

Hide file tree

Showing 2 changed files with 8 additions and 10 deletions.
diff --git a/internal/config/config.go b/internal/config/config.go
@@ -106,18 +106,17 @@ func (config *Config) walkIndex(index RuleIndex, key string, list Ls) error {
 		}
 
 		if reflect.TypeOf(v).Kind() == reflect.Map {
-			switch key == "" {
-			case true:
+			if key == "" {
 				if err := config.walkIndex(index, k, v.(Ls)); err != nil {
 					return err
 				}
-			case false:
-				keyCombination := fmt.Sprintf("%s%s%s", key, sep, k)
-				if err := config.walkIndex(index, keyCombination, v.(Ls)); err != nil {
-					return err
-				}
+				continue
 			}
 
+			keyCombination := fmt.Sprintf("%s%s%s", key, sep, k)
+			if err := config.walkIndex(index, keyCombination, v.(Ls)); err != nil {
+				return err
+			}
 			continue
 		}
 

diff --git a/internal/linter/linter.go b/internal/linter/linter.go
@@ -244,10 +244,9 @@ func (linter *Linter) Run(filesystem fs.FS, paths map[string]struct{}, debug boo
 	if debug {
 		fmt.Printf("=============================\nls index\n-----------------------------\n")
 		for path, pathIndex := range index {
-			switch path == "" {
-			case true:
+			if path == "" {
 				fmt.Printf(".:")
-			case false:
+			} else {
 				fmt.Printf("%s:", path)
 			}