Просто помните, вы спросили.

Очень плохая реализация.

• Не создавайте такие потоки. Очень мало иметь больше потоков, чем процессорные ядра. Дополнительные потоки будут в значительной степени конкурировать друг с другом, тем более, что все они работают с одним и тем же кодом.

• Вам нужно реализовать блоки с использованием. Если вы выбросите исключение (и, скорее всего, вы это сделаете), тогда вы будете терять ресурсы.

• Какова цель возврата bool? Вы проверяете его где-то? В любом случае, ваша обработка ошибок и исключений - беспорядок.

  • Когда вы получаете ответ не-200, вы не видите код ошибки.
  • Вы сравниваете свойство Message, чтобы решить, является ли он тайм-аутом. Microsoft должна помещать пробел между "временем" и "выходить", просто зная вас.
  • Если это не тайм-аут, вы отображаете только свойство Message, а не все исключение, а свойство Message уже является строкой и вам не нужно вызывать ToString() на нем.

Следующая партия изменений

Это не закончено, я не думаю, но попробую:

public static void Main()
{
    // Don't mind the interpretation. I needed an excuse to define "rdr"
    using (var conn = new SqlConnection())
    {
        conn.Open();
        using (var cmd = new SqlCommand("SELECT Url FROM UrlsToCheck", conn))
        {
            using (var rdr = cmd.ExecuteReader())
            {
                while (rdr.Read())
                {
                    // Use the thread pool. Please.
                    ThreadPool.QueueUserWorkItem(
                        delegate(object weburl)
                            {
                                // I invented a reason for you to return bool
                                if (!ConnectionAvailable(weburl.ToString()))
                                {
                                    // Console would be getting pretty busy with all
                                    // those threads
                                    Debug.WriteLine(
                                        String.Format(
                                            "{0} was not available",
                                            weburl));
                                }
                            },
                            rdr[0]);
                }
            }
        }
    }
}

public static bool ConnectionAvailable(string strServer)
{
    try
    {
        strServer = "http://" + strServer;
        var reqFp = (HttpWebRequest)WebRequest.Create(strServer);
        reqFp.Timeout = 10000;
        reqFp.Method = "HEAD";

        // BTW, what an "FP"?
        using (var rspFp = (HttpWebResponse) reqFp.GetResponse()) // IDisposable 
        {
            if (HttpStatusCode.OK == rspFp.StatusCode)
            {
                Debug.WriteLine(string.Format("{0} - OK", strServer));
                return true; // Dispose called when using is exited
            }

            // Include the error because it nice to know these things
            Debug.WriteLine(String.Format(
                 "{0} Server returned error: {1}", 
                 strServer, rspFp.StatusCode));
            return false;
        }
    }
    catch (WebException x)
    {
        // Don't tempt fate and don't let programs read human-readable messages
        if (x.Status == WebExceptionStatus.Timeout)
        {
            Debug.WriteLine(string.Format("{0} - Timed out", strServer));
        }
        else
        {
            // The FULL exception, please
            Debug.WriteLine(x.ToString());
        }

        return false;
    }
}

Почти выполнено - не протестирован Код поздней ночи

public static void Main()
{
    using (var conn = new SqlConnection())
    {
        conn.Open();
        using (var cmd = new SqlCommand("", conn))
        {
            using (var rdr = cmd.ExecuteReader())
            {
                if (rdr == null)
                {
                    return;
                }

                while (rdr.Read())
                {
                    ThreadPool.QueueUserWorkItem(
                        CheckConnectionAvailable, rdr[0]);
                }
            }
        }
    }
}

private static void CheckConnectionAvailable(object weburl)
{
    try
    {
        // If this works, it a lot simpler
        var strServer = new Uri("http://" + weburl);
        using (var client = new WebClient())
        {
            client.UploadDataCompleted += ClientOnUploadDataCompleted;
            client.UploadDataAsync(
                strServer, "HEAD", new byte[] {}, strServer);
        }
    }
    catch (WebException x)
    {
        Debug.WriteLine(x);
    }
}

private static void ClientOnUploadDataCompleted(
    object sender, UploadDataCompletedEventArgs args)
{
    if (args.Error == null)
    {
        Debug.WriteLine(string.Format("{0} - OK", args.UserState));
    }
    else
    {
        Debug.WriteLine(string.Format("{0} - Error", args.Error));
    }
}

Не использовать потоки.

Асинхронные обратные вызовы и очереди. Зачем создавать поток, когда ресурс, который все они хотят, - это доступ к внешнему миру. Ограничьте свои потоки примерно до 5, а затем реализуйте класс, который использует очередь. разделите код на две части, выборку и процесс. Один управляет потоком данных, а другой контролирует доступ к внешнему миру.

Используйте любой язык, который вам нравится, но вы не ошибетесь, если считаете, что потоки предназначены для обработки, а хруст числа и асинхронные обратные вызовы предназначены для управления ресурсами.

Используйте класс ThreadPool. Не создавайте сотни таких нитей. У потоков есть такие огромные накладные расходы, и в вашем случае происходит то, что ваш процессор будет тратить 99% времени на переключение контекста и 1% на выполнение реальной работы.