3

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

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);
    }
}

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

4

2 回答 2

4

这里要考虑的一件事是，如果您开始超过限制，您正在使用的 api 可能会“惩罚”您。我认为最好主动并限制您的 api 请求，这样您就不会收到 429 错误。

我们从我们正在使用的 api 开始在我们的应用程序上遇到同样的事情（429 错误）。这个特定 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();
    }

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

于 2017-12-26T20:13:28.703 回答
2

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

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

除非我弄错了 Network 类，否则我认为你应该只捕获 WebExceptions。我认为这就是你应该如何从这个函数中获取 HTTP 错误代码的方式，但我不是真的 sur。

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

于 2017-12-26T20:11:01.293 回答