Как вы тестируете переменные Request.QueryString []? - PullRequest
Новая
       28

Как вы тестируете переменные Request.QueryString []?

50 голосов
/ 08 декабря 2008

Я часто использую Request.QueryString[] переменные.

В моем Page_load я часто делаю такие вещи, как: 

       int id = -1;

        if (Request.QueryString["id"] != null) {
            try
            {
                id = int.Parse(Request.QueryString["id"]);
            }
            catch
            {
                // deal with it
            }
        }

        DoSomethingSpectacularNow(id);

Все это кажется немного неуклюжим и мусором. Как вы справляетесь со своими Request.QueryString[] с?

Ответы [ 11 ]

52 голосов
/ 08 декабря 2008

Ниже приведен метод расширения, который позволит вам написать код, подобный этому: 

int id = request.QueryString.GetValue<int>("id");
DateTime date = request.QueryString.GetValue<DateTime>("date");

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

public static T GetValue<T>(this NameValueCollection collection, string key)
{
    if(collection == null)
    {
        throw new ArgumentNullException("collection");
    }

    var value = collection[key];

    if(value == null)
    {
        throw new ArgumentOutOfRangeException("key");
    }

    var converter = TypeDescriptor.GetConverter(typeof(T));

    if(!converter.CanConvertFrom(typeof(string)))
    {
        throw new ArgumentException(String.Format("Cannot convert '{0}' to {1}", value, typeof(T)));
    }

    return (T) converter.ConvertFrom(value);
}
34 голосов
/ 08 декабря 2008

Вместо этого используйте int.TryParse, чтобы избавиться от блока try-catch: 

if (!int.TryParse(Request.QueryString["id"], out id))
{
  // error case
}
19 голосов
/ 29 сентября 2010

Попробуйте, чувак ... 

List<string> keys = new List<string>(Request.QueryString.AllKeys);

Тогда вы сможете легко найти парня для поиска строки ... 

keys.Contains("someKey")
17 голосов
/ 08 декабря 2008

Я использую небольшой вспомогательный метод: 

public static int QueryString(string paramName, int defaultValue)
{
    int value;
    if (!int.TryParse(Request.QueryString[paramName], out value))
        return defaultValue;
    return value;
}

Этот метод позволяет мне читать значения из строки запроса следующим образом: 

int id = QueryString("id", 0);
10 голосов
/ 08 декабря 2008

Ну, во-первых, вместо этого используйте int.TryParse ... 

int id;
if (!int.TryParse(Request.QueryString["id"], out id))
{
    id = -1;
}

Это предполагает, что "не присутствует" должно иметь тот же результат, что и "не целое число", конечно.

РЕДАКТИРОВАТЬ: В других случаях, когда вы все равно будете использовать параметры запроса в качестве строк, я думаю, что это определенно хорошая идея, чтобы проверить, что они присутствуют.

9 голосов
/ 08 декабря 2008

Вы также можете использовать методы расширения, указанные ниже, и делать это так 

int? id = Request["id"].ToInt();
if(id.HasValue)
{

}

// Методы расширения 

public static int? ToInt(this string input) 
{
    int val;
    if (int.TryParse(input, out val))
        return val;
    return null;
}

public static DateTime? ToDate(this string input)
{
    DateTime val;
    if (DateTime.TryParse(input, out val))
        return val;
    return null;
}

public static decimal? ToDecimal(this string input)
{
    decimal val;
    if (decimal.TryParse(input, out val))
        return val;
    return null;
}
4 голосов
/ 08 декабря 2008 
if(!string.IsNullOrEmpty(Request.QueryString["id"]))
{
//querystring contains id
}
1 голос
/ 19 октября 2009

Я изменил ответ Брайана Уоттса, чтобы, если параметр, который вы запрашиваете, не существовал, и вы указали тип, допускающий значение NULL, он вернул бы ноль: 

public static T GetValue<T>(this NameValueCollection collection, string key)
    {
        if (collection == null)
        {
            return default(T);
        }

        var value = collection[key];

        if (value == null)
        {
           return default(T);
        }

        var type = typeof(T);

        if (type.IsGenericType && type.GetGenericTypeDefinition() == typeof(Nullable<>))
        {
            type = Nullable.GetUnderlyingType(type);
        }

        var converter = TypeDescriptor.GetConverter(type);

        if (!converter.CanConvertTo(value.GetType()))
        {
            return default(T);
        }

        return (T)converter.ConvertTo(value, type);
    }

Теперь вы можете сделать это: 

Request.QueryString.GetValue<int?>(paramName) ?? 10;
1 голос
/ 08 декабря 2008

У меня на самом деле есть служебный класс, который использует Generics для «переноса» сеанса, который выполняет всю «грубую работу» за меня, у меня также есть кое-что почти идентичное для работы со значениями QueryString.

Это помогает удалить дубликаты кода для (часто многочисленных) проверок.

Например: 

public class QueryString
{
    static NameValueCollection QS
    {
        get
        {
            if (HttpContext.Current == null)
                throw new ApplicationException("No HttpContext!");

            return HttpContext.Current.Request.QueryString;
        }
    }

    public static int Int(string key)
    {
        int i; 
        if (!int.TryParse(QS[key], out i))
            i = -1; // Obviously Change as you see fit.
        return i;
    }

    // ... Other types omitted.
}

// And to Use..
void Test()
{
    int i = QueryString.Int("test");
}

Примечание:

Это очевидно использует статику, которая не нравится некоторым людям из-за того, как она может влиять на тестовый код. Вы можете легко реорганизовать что-то, что работает на основе экземпляров и любых интерфейсов, которые вам требуются. Я просто думаю, что пример статики самый легкий.

Надеюсь, это поможет / даст пищу для размышлений.

1 голос
/ 08 декабря 2008

э-э-э, это риск кармы ...

У меня есть СУХАЯ абстракция, проверяемая модулем, потому что, ну, в общем, потому что было слишком много переменных строки запроса, чтобы оставить их в унаследованном преобразовании.

Приведенный ниже код взят из служебного класса, конструктор которого требует ввода NameValueCollection (this.source) и строкового массива "keys", потому что старое приложение было довольно органичным и разработало возможность для нескольких различных строк быть потенциальными клавиша ввода. Однако мне нравится растяжимость. Этот метод проверяет коллекцию для ключа и возвращает ее в требуемом типе данных. 

private T GetValue<T>(string[] keys)
{
    return GetValue<T>(keys, default(T));
}

private T GetValue<T>(string[] keys, T vDefault)
{
    T x = vDefault;

    string v = null;

    for (int i = 0; i < keys.Length && String.IsNullOrEmpty(v); i++)
    {
        v = this.source[keys[i]];
    }

    if (!String.IsNullOrEmpty(v))
    {
        try
        {
            x = (typeof(T).IsSubclassOf(typeof(Enum))) ? (T)Enum.Parse(typeof(T), v) : (T)Convert.ChangeType(v, typeof(T));
        }
        catch(Exception e)
        {
            //do whatever you want here
        }
    }

    return x;
}
Страница:
...