fix: intrange recommendations - refactor loops

Signed-off-by: Mike Nguyen <[email protected]>

Author: mikeee

actor/manager/manager.go

@@ -37,11 +37,11 @@ var ignoredActorMethods = []string{"Type"}
 // init initializes the action method exclusion list with methods from ServerImplBaseCtx and ReminderCallee interfaces.
 func init() {
 	serverImplBaseCtxType := reflect.TypeOf(&actor.ServerImplBaseCtx{})
-	for i := 0; i < serverImplBaseCtxType.NumMethod(); i++ {
+	for i := range serverImplBaseCtxType.NumMethod() {
 		ignoredActorMethods = append(ignoredActorMethods, serverImplBaseCtxType.Method(i).Name)
 	}
 	ReminderCallType := reflect.TypeOf((*actor.ReminderCallee)(nil)).Elem()
-	for i := 0; i < ReminderCallType.NumMethod(); i++ {
+	for i := range ReminderCallType.NumMethod() {
 		ignoredActorMethods = append(ignoredActorMethods, ReminderCallType.Method(i).Name)
 	}
 }
@@ -265,7 +265,7 @@ type MethodType struct {
 // suitableMethods returns suitable Rpc methods of typ.
 func suitableMethods(typ reflect.Type) map[string]*MethodType {
 	methods := make(map[string]*MethodType)
-	for m := 0; m < typ.NumMethod(); m++ {
+	for m := range typ.NumMethod() {
 		method := typ.Method(m)
 		// skip methods from ServerImplBaseCtx struct and ServerContext and ReminderCallee interfaces.
 		if slices.Contains(ignoredActorMethods, method.Name) {

client/actor.go

@@ -289,7 +289,7 @@ func (c *GRPCClient) implActor(actor actor.Client, serializer codec.Codec) {
 	}
 
 	numField := valueOfActor.NumField()
-	for i := 0; i < numField; i++ {
+	for i := range numField {
 		t := typeOfActor.Field(i)
 		methodName := t.Name
 		if methodName == "Type" {
@@ -312,7 +312,7 @@ func (c *GRPCClient) implActor(actor actor.Client, serializer codec.Codec) {
 			}
 
 			funcOuts := make([]reflect.Type, outNum)
-			for i := 0; i < outNum; i++ {
+			for i := range outNum {
 				funcOuts[i] = t.Type.Out(i)
 			}
 

client/client_test.go

@@ -466,7 +466,7 @@ func (s *testDaprServer) SubscribeConfiguration(in *pb.SubscribeConfigurationReq
 		return err
 	}
 
-	for i := 0; i < 5; i++ {
+	for range 5 {
 		select {
 		case <-stopCh:
 			return nil

client/state.go

@@ -484,7 +484,7 @@ func (c *GRPCClient) DeleteBulkState(ctx context.Context, storeName string, keys
 	}
 
 	items := make([]*DeleteStateItem, 0, len(keys))
-	for i := 0; i < len(keys); i++ {
+	for i := range keys {
 		item := &DeleteStateItem{
 			Key:      keys[i],
 			Metadata: meta,
@@ -502,7 +502,7 @@ func (c *GRPCClient) DeleteBulkStateItems(ctx context.Context, storeName string,
 	}
 
 	states := make([]*v1.StateItem, 0, len(items))
-	for i := 0; i < len(items); i++ {
+	for i := range items {
 		item := items[i]
 		if err := hasRequiredStateArgs(storeName, item.Key); err != nil {
 			return fmt.Errorf("missing required arguments: %w", err)