ASP.Net System.ArgumentOutOfRangeException

Question

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

string strIP = Request.ServerVariables["REMOTE_ADDR"]; 
if (
    (strIP.Substring(0, 9) != "XXX.XX.X.") 
&& (strIP.Substring(0, 10) != "XXX.XX.XX.") 
&& (strIP.Substring(0, 6) != "XX.XX.") 
&& (strIP.Substring(0, 6) != "XX.XX.") 
) 
{ 
..// Check user for being an ADMIN // .... 
} 

Этот код работал нормально в течение нескольких недель, но неожиданно начал последовательно выходить из строя. Сообщение об ошибке:

Исключение

Исключение Тип: System. ArgumentOUtOfRangeException

Сообщение об исключении: указатель и длина должны относиться к местоположению внутри строки. Имя параметра: длина.

Когда я удаляю линию с помощью «Подстроки (0,10)», все работает. Кроме того, когда я меняю строку «Подстрока (0,10)» на «Подстрока (0,9)» и удаляю последнюю «.», Все работает.

Может ли кто-нибудь сказать мне, почему или, возможно, проинструктировать о том, что делается неправильно? Для жизни я не могу понять, что происходит.

Answer 1

проблема заключается в тха t strIP не имеет 10 символов, потому что по какой-то причине ваша конфигурация изменилась. Вы могли бы сделать что-то вроде:

(strIP.Length >= 9 && strIP.Substring(0, 9) != "XXX.XX.X.") 
|| (strIP.Length >= 10 && strIP.Substring(0, 10) != "XXX.XX.XX.") 
|| (strIP.Length >= 6 && strIP.Substring(0, 6) != "XX.XX.") 

Обратите внимание, что четвертая строка была дубликатом третьего.

Answer 2

Не позволяют из оценки точности произойдет, поставив галочку на длину strIP, прежде чем пытаться делать какие-либо из суб-сравнения строк, например:

if (strIP.Length == 10) 
{ 
    if ((strIP.Substring(0, 9) != "XXX.XX.X.") 
     && (strIP.Substring(0, 10) != "XXX.XX.XX.") 
     && (strIP.Substring(0, 6) != "XX.XX.") 
     && (strIP.Substring(0, 6) != "XX.XX.")) 
    { 
     ..// Check user for being an ADMIN // .... 
    } 
} 
else 
{ 
    // Do something here, error, message to user, deny access, etc. 
} 

UPDATE:

Если вы хотите применить только проверки, на основании длины строки, а затем использовать switch заявление, например:

switch (strIP.Length) 
{ 
    case 6: 
     if(strIP.Substring(0, 6) != "XX.XX.") 
     { 
      // Check user for being an ADMIN 
     } 
     break; 
    case 9: 
     if(strIP.Substring(0, 9) != "XXX.XX.X.") 
     { 
      // Check user for being an ADMIN 
     } 
     break; 
    case 10: 
     if(strIP.Substring(0, 10) != "XXX.XX.XX.") 
     { 
      // Check user for being an ADMIN 
     } 
     break; 
    default: 
     // IP string is not in format expected, do something here 
     // Most likely would want to err on the side of caution and deny access 
     break; 
}