Question

我正在点击一个api，它将返回429，请求太多，如果你在五分钟内击中它超过250次就会响应。计数每五分钟重置一次，所以我一直这样处理：

try
{
    return request.GetResponse();  
}
catch (Exception e)
{
    if (e.Message.Contains("429"))
    {
        System.Threading.Thread.Sleep(5 * 60 * 1000);
        return request.GetResponse();
    }
    else
    {
        throw new Exception(e.Message);
    }
}

这是处理这种情况的正确方法吗？

Answer 1

您不应该在异常类上进行字符串解析。反过来，异常类不应该将重要的信息放入消息字段中。你对异常的整个捕获和重新抛出也是错误的。所以在前面你应该阅读适当的异常处理。这里经常链接两篇文章：

http://blogs.msdn.com/b/ericlippert/archive/2008/09/10/vexing-exceptions.aspx http://www.codeproject.com/Articles/9538/Exception-Handling-Best-Practices-in-NET

除非我得到错误的网络课程，否则我认为你应该catch WebExceptions only。我认为这就是你应该如何从这个函数中获取HTTP错误代码，但我不是真的很喜欢。

最好完全避免这种例外。所以我要问：你为什么甚至连续调用那个函数？批量撤销没有适当的功能吗？该服务是否打算像这样自动化？

Answer 2

这里要考虑的一件事是，你正在消费的api可能＆＃34;惩罚＆＃34;如果你开始超越限制你。我认为最好主动并限制你的api请求，所以你永远不会收到429错误。

我们开始在我们的app上遇到同样的事情（429个错误）来自我们正在消费的api。这个特定api的限制是每60秒10次。我们实现了一个throttle()函数，该函数利用嵌入了日期/时间的内存缓存。我们使用的API也跟踪帐户的使用情况。你可能不需要那样。但这是我们用来限制我们的请求的片段，以确保我们始终处于限制之下：

    private void throttle()
    {
        var maxPerPeriod = 250;
        //If you utilize multiple accounts, you can throttle per account. If not, don't use this:
        var keyPrefix = "a_unique_id_for_the_basis_of_throttling";
        var intervalPeriod = 300000;//5 minutes
        var sleepInterval = 5000;//period to "sleep" before trying again (if the limits have been reached)
        var recentTransactions = MemoryCache.Default.Count(x => x.Key.StartsWith(keyPrefix));
        while (recentTransactions >= maxPerPeriod)
        {
            System.Threading.Thread.Sleep(sleepInterval);
            recentTransactions = MemoryCache.Default.Count(x => x.Key.StartsWith(keyPrefix));
        }
        var key = keyPrefix + "_" + DateTime.Now.ToUniversalTime().ToString("yyyyMMddHHmm");
        var existing = MemoryCache.Default.Where(x => x.Key.StartsWith(key));
        if (existing != null && existing.Any())
        {
            var counter = 2;
            var last = existing.OrderBy(x => x.Key).Last();
            var pieces = last.Key.Split('_');
            if (pieces.Count() > 2)
            {
                var lastCount = 0;
                if (int.TryParse(pieces[2], out lastCount))
                {
                    counter = lastCount + 1;
                }
            }
            key = key + "_" + counter;
        }
        var policy = new CacheItemPolicy
        {
            AbsoluteExpiration = DateTimeOffset.UtcNow.AddMilliseconds(intervalPeriod)
        };
        MemoryCache.Default.Set(key, 1, policy);
    }

我们在代码中使用了这个throttle()函数，如下所示：

    public override void DoAction()
    {
        throttle();
        var url = ContentUri;
        var request = (HttpWebRequest)WebRequest.Create(url);
        request.Method = "GET";
        request.Headers.Add("Authorization", "Bearer " + AccessToken);
        request.Accept = "application/json";
        WebResponse response = request.GetResponse();
        var dataStream = new MemoryStream();
        using (Stream responseStream = request.GetResponse().GetResponseStream())
        {
            //DO STUFF WITH THE DOWNLOADED DATA HERE...
        }
        dataStream.Close();
        response.Close();
    }

它基本上跟踪缓存中的请求。如果已达到限制，则会暂停，直到经过足够的时间，以使您仍处于限制之下。

处理HTTP 429错误的最佳方法

2 个答案: