ratelimit/keeper: refactor with inversions and early returns for more idiomatic Go

[odeke-em]

Just while reading through this code I noticed this

ibc-rate-limiting/ratelimit/keeper/abci.go

Lines 11 to 17 in 29a35e9

	func (k Keeper) BeginBlocker(ctx sdk.Context) {
	if epochStarting, epochNumber := k.CheckHourEpochStarting(ctx); epochStarting {
	for _, rateLimit := range k.GetAllRateLimits(ctx) {
	if rateLimit.Quota.DurationHours != 0 && epochNumber%rateLimit.Quota.DurationHours == 0 {
	err := k.ResetRateLimit(ctx, rateLimit.Path.Denom, rateLimit.Path.ChannelId)
	if err != nil {
	k.Logger(ctx).Error(fmt.Sprintf("Unable to reset quota for Denom: %s, ChannelId: %s", rateLimit.Path.Denom, rateLimit.Path.ChannelId))

which can be refactored for more idiomatic Go, using early returns and inversions to reduce nesting but also make conditions much simpler to read and reason about so

func (k Keeper) BeginBlocker(ctx sdk.Context) {
   // As long as epochStarting is true,
   // epochNumber will always be >0.
	epochStarting, epochNumber := k.CheckHourEpochStarting(ctx)
   if !epochStarting {
      return
   }
	
   for _, rateLimit := range k.GetAllRateLimits(ctx) {
      ok := rateLimit.Quota.DurationHours != 0 && epochNumber%rateLimit.Quota.DurationHours == 0
      if !ok {
         continue
      }
			
      err := k.ResetRateLimit(ctx, rateLimit.Path.Denom, rateLimit.Path.ChannelId)
      if err != nil {
         k.Logger(ctx).Error(fmt.Sprintf("Unable to reset quota for Denom: %s, ChannelId: %s", rateLimit.Path.Denom, rateLimit.Path.ChannelId))
		 }
	}
}