پیشنهادات / سوال / نظر

[BaseMax]

	sell, err := strconv.Atoi(currencies[name+"1"])
	buy, err := strconv.Atoi(currencies[name+"2"])

............

	buy, err := strconv.Atoi(currencies[name+"_2"])

همیشه بین عملگر یک فاصله قبل و بعد اش بزارید.

[BaseMax]

	if err != nil {
		return nil, err
	}
	sell, err := strconv.Atoi(currencies[name+"1"])

اگر این دو بلاک بهم دیگر ارتباطی ندارند بین انها فاصله قرار دهید.
https://github.com/BaseMax/CurrencyAPIGo/blob/main/currency.go#L72

[BaseMax]

	rdb = redis.NewClient(&redis.Options{
		Addr:     "localhost:6379",
		Password: "",
		DB:       0,
	})

اگر مشخصات محرمانه را از فایل بخوانید و در سورس کد نباشد بهتر است

[BaseMax]

	if err == redis.Nil {
		return nil, nil
	}

	if err != nil {
		return nil, err
	}

بهتر است که این دو بلاک بهم دیگر بچسبند و حتی تبدیل به if... else if بشوند.

[BaseMax]

	exp := rdb.Expire(ctx, key, ttl*time.Second)

قبل و بعد عملگر ها فاصله بزارید.

[BaseMax]

import (
	"context"
	"encoding/json"
	"time"

ترتیب کتابخانه ها می تواند بهتر باشد.

[BaseMax]

func main() {

	status := rdb.Ping(ctx)

یک خط اضافه اول تابع خالی افتاده حذف بشود

[BaseMax]

	return result

}

فضای الکی خالی اخر تابع بعد از ریترن هست که الان حذف میکنم.

[BaseMax]

import (
	"encoding/json"
	"fmt"
	"log"
	"regexp"

ترتیب کتابخانه ها رعایت بشود.
https://github.com/BaseMax/CurrencyAPIGo/blob/main/provider.go

[BaseMax]

	if slices.Contains(golds, name) {
		g, err := GetGold(name)
		if err != nil {
			fmt.Println(err)
			w.Write([]byte("not found"))
			return
		}
		json.NewEncoder(w).Encode(g)
		return
	}

	if slices.Contains(coins, name) {
		c, err := GetCoin(name)
		if err != nil {
			fmt.Println(err)
			w.Write([]byte("not found"))
			return
		}
		json.NewEncoder(w).Encode(c)
		return
	}

اگر در یک زمان فقط یکی از شرط ها برقرار می شود و تابع تمام می شود تبدیل به
if... else...
بشود
https://github.com/BaseMax/CurrencyAPIGo/edit/main/routes.go

[BaseMax]

	json.NewEncoder(w).Encode(uc)

}

اخر فایل و اخر تابع یک فضای الکی خالی است. حذف می کنم.
https://github.com/BaseMax/CurrencyAPIGo/edit/main/routes.go

[itsjoniur]

	if slices.Contains(golds, name) {
		g, err := GetGold(name)
		if err != nil {
			fmt.Println(err)
			w.Write([]byte("not found"))
			return
		}
		json.NewEncoder(w).Encode(g)
		return
	}

	if slices.Contains(coins, name) {
		c, err := GetCoin(name)
		if err != nil {
			fmt.Println(err)
			w.Write([]byte("not found"))
			return
		}
		json.NewEncoder(w).Encode(c)
		return
	}

اگر در یک زمان فقط یکی از شرط ها برقرار می شود و تابع تمام می شود تبدیل به
if... else...
بشود
https://github.com/BaseMax/CurrencyAPIGo/edit/main/routes.go

چشم در ریفکتور لحاظ میشود