바쁜 ASP.NET MVC에서 전역 변수를 정의하는 방법

Question

내 사이트에서 제 3 자 API를 호출합니다. 속도 제한을 피하기 위해 요청을 대기열에 넣기 위해 전역 변수를 정의해야합니다. (I'm using RateLimiter 더 좋은 해결책?)

namespace MySite.App_Start 
{ 
    public static class Global 
    { 
     public static int MaxCount { get; set; } = 30; 
     public static TimeSpan Interval { get; set; } = TimeSpan.FromSeconds(1); 

     private static TimeLimiter rateLimiter; 
     public static TimeLimiter RateLimiter 
     { 
      get 
      { 
       if (rateLimiter == null) 
        rateLimiter = TimeLimiter.GetFromMaxCountByInterval(MaxCount, Interval); 

       return rateLimiter; 
      } 
     } 
    } 
} 

그럼 RateLimiter 속성을 사용합니다. 그러나 나는 전역 변수를 갖는 것이 좋은 생각이 아니라는 것을 많이 읽었습니다. 내 사이트가 초당 요청이 많다는 것을 고려하면 내 코드를 안전하게 사용할 수 있습니까? 감사.

Answer 1

코드는 100 % 안전합니다. 나는 큰 문제는 아니 겠지만, 코드를 제대로 작성하는 것이 더 낫다. 게으른

이

는 IoC 컨테이너는 잘 처리하는 일이지만, 당신이 하나를 사용하지 않으려면, 당신은 사용할 수 있습니다

private static TimeLimiter rateLimiter = new Lazy(() => 
    TimeLimiter.GetFromMaxCountByInterval(MaxCount, Interval)); 
public static TimeLimiter RateLimiter => rateLimiter.Value; 

Answer 2

코드가 스레드 안전하지 않습니다. 이 시도 :

public class Singleton 
{  
    protected Singleton() { } 
    private sealed class SingletonCreator 
    { 
    private static readonly Singleton instance = new Singleton(); 
    public static Singleton Instance { get { return instance; } } 
    } 
    public static Singleton Instance 
    { 
    get { return SingletonCreator.Instance; } 
    } 
} 

또는

Answer 3

어쩌면 SingleInstance 객체를 생성하여 좋아하는 IOC의 컨테이너를 사용하여, 당신은 잠금 문을 사용하여 스레드로부터 안전 할 수 있습니다. 그것은 처음에 TimeLimiter의 여러 인스턴스를 만들 수 있습니다 및 코드 주변에 따라서는 문제가 될 수 있기 때문에

public static class Global 
{ 
    public static int MaxCount { get; set; } = 30; 
    public static TimeSpan Interval { get; set; } = TimeSpan.FromSeconds(1); 

    private static object _lockObject = new object(); 
    private static TimeLimiter rateLimiter; 
    public static TimeLimiter RateLimiter 
    { 
     get 
     { 
      lock (_lockObject) 
      { 
       if (rateLimiter == null) 
        rateLimiter = TimeLimiter.GetFromMaxCountByInterval(MaxCount, Interval); 
       return rateLimiter; 
      } 

     } 
    } 
}