code refactor

Author: lvan100

gs/internal/gs/gs.go

@@ -14,7 +14,7 @@
  * limitations under the License.
  */
 
-// Package gs provides all the concepts required for go-spring implementation.
+// Package gs provides all the concepts required for Go-Spring implementation.
 package gs
 
 import (
@@ -25,12 +25,12 @@ import (
 	"github.com/go-spring/spring-core/conf"
 )
 
-// BeanInitFunc defines the prototype for initialization functions,
-// e.g., `func(bean)` or `func(bean) error`.
+// BeanInitFunc defines the prototype for initialization functions.
+// For example: `func(bean)` or `func(bean) error`.
 type BeanInitFunc = interface{}
 
-// BeanDestroyFunc defines the prototype for destroy functions,
-// e.g., `func(bean)` or `func(bean) error`.
+// BeanDestroyFunc defines the prototype for destroy functions.
+// For example: `func(bean)` or `func(bean) error`.
 type BeanDestroyFunc = interface{}
 
 // BeanInitInterface defines an interface for bean initialization.
@@ -89,7 +89,7 @@ type Condition interface {
 	Matches(ctx CondContext) (bool, error)
 }
 
-// CondBean represents a bean with an ID, Name, TypeName, and Type.
+// CondBean represents a bean with Name and Type.
 type CondBean interface {
 	Name() string
 	Type() reflect.Type
@@ -132,6 +132,7 @@ type Callable interface {
 
 /*********************************** conf ************************************/
 
+// Properties represents read-only configuration properties.
 type Properties = conf.ReadOnlyProperties
 
 // Refreshable represents an object that can be dynamically refreshed.
@@ -144,8 +145,8 @@ type Refreshable interface {
 
 // ConfigurationParam holds configuration parameters for bean setup.
 type ConfigurationParam struct {
-	Includes []string // Methods to include
-	Excludes []string // Methods to exclude
+	Includes []string // List of methods to include
+	Excludes []string // List of methods to exclude
 }
 
 // BeanRegistration provides methods to configure and register bean metadata.

gs/internal/gs_cond/cond.go

@@ -57,6 +57,7 @@ func (c *onFunc) String() string {
 /******************************* OnProperty **********************************/
 
 // OnPropertyInterface defines the methods for evaluating a condition based on a property.
+// This interface provides flexibility for matching missing properties and checking their values.
 type OnPropertyInterface interface {
 	gs.Condition
 	MatchIfMissing() OnPropertyInterface
@@ -268,26 +269,26 @@ func (c *onExpression) String() string {
 
 /********************************** Not ***************************************/
 
-// not is a condition that negates another condition. It returns true if the wrapped
+// onNot is a condition that negates another condition. It returns true if the wrapped
 // condition evaluates to false, and false if the wrapped condition evaluates to true.
-type not struct {
+type onNot struct {
 	c gs.Condition // The condition to negate.
 }
 
 // Not creates a condition that inverts the result of the provided condition.
 func Not(c gs.Condition) gs.Condition {
-	return &not{c: c}
+	return &onNot{c: c}
 }
 
-func (c *not) Matches(ctx gs.CondContext) (bool, error) {
+func (c *onNot) Matches(ctx gs.CondContext) (bool, error) {
 	ok, err := c.c.Matches(ctx)
 	if err != nil {
 		return false, errutil.WrapError(err, "condition matches error: %s", c)
 	}
 	return !ok, nil
 }
 
-func (c *not) String() string {
+func (c *onNot) String() string {
 	return fmt.Sprintf("Not(%s)", c.c)
 }