Merge pull request #10 from manicar2093/feat/identify-empty-slices

feat: add identify empty slices

goption.go

@@ -32,7 +32,7 @@ func (c Optional[T]) Get() (T, error) {
 	return c.value, nil
 }
 
-// IsPresent returns true if there is a value present, otherwise false.
+// IsPresent returns true if there is a value present, otherwise false. It recognizes an empty slice as not present so it returns false
 func (c Optional[T]) IsPresent() bool {
 	return c.isValidValue
 }
@@ -69,8 +69,13 @@ func isValidData[T any](value T) bool {
 	}
 	kind := typeOfValue.Kind()
 	val := reflect.ValueOf(value)
-	if kind == reflect.Pointer {
+
+	switch kind {
+	case reflect.Pointer:
 		return !val.IsNil()
+	case reflect.Slice:
+		return val.Len() != 0
+	default:
+		return !val.IsZero()
 	}
-	return !val.IsZero()
 }

goption_test.go

@@ -74,6 +74,8 @@ var _ = Describe("Goption", func() {
 			Entry("empty int", int(0), false),
 			Entry("filled int", int(1), true),
 			Entry("empty optional", goption.Optional[string]{}, false),
+			Entry("empty slice", []string{}, false),
+			Entry("empty slice of pointers", []*string{}, false),
 		)
 	})