Сумасшедший кусок кода: Проверка параметра запроса

Posted by Dmytro Shteflyuk on under ASP.NET · English (11,291 views)

Как часто вы падаете под стол, увидев чей-то код? Сегодня я наткнулся на сногсшибательный пример в текущем проекте, и просто не могу удержаться и не выложить его здесь. Итак,

1
2
3
4
5
6
7
8
9
10
11
if (Request.QueryString.HasKeys())
{
    string[] keys = Request.QueryString.AllKeys;
    foreach (string k in keys)
    {
        if (k == "memberpagemode" && (string)Request.QueryString.GetValues(k).GetValue(0) == "edit")
        {
            pSett.ChangeFormViewMode(FormViewMode.Edit);
        }
    }
}

А как вы ищете в хэше ключ с определенным значением?

13 Responses to this entry

Subscribe to comments with RSS

said on Декабрь 18, 2007 at 08:17 · Permalink

Ну … если почитать отжиги студентов то это покажется просто невинной детской забавой

said on Декабрь 25, 2007 at 07:18 · Permalink

я тоже ВТФшником…так удобнее)

Kigorw @
said on Декабрь 25, 2007 at 16:07 · Permalink

я уже подобный велосипед встречал)

said on Май 8, 2008 at 15:07 · Permalink

У меня подружка в универе с информатикой недружит, она иногда такое выдает, что препод под стол ложится. Может быть это она пыталась чтонибудь сделать, по книжке)))

said on Май 19, 2008 at 14:45 · Permalink

да… видно чел сильно учился. это как мазок кисти великого учителя :)))

Pavel @
said on Май 30, 2008 at 17:59 · Permalink

Давно не писал на ASP.NET, интересно а как автор видит для себя правильный пример, лично мне кажется, что ASP.NET (1.X, может быть 2.Х) страдает черсчур усложенным синтаксисом, когда кроме как вот так
(string)Request.QueryString.GetValues(k).GetValue(0)
никак по-другому нельзя обратиться

said on Май 31, 2008 at 08:39 · Permalink

Весь вышеприведенный код можно записать двумя строчками:

1
2
if (Request.QueryString["memberpagemode"] == "edit")
    pSett.ChangeFormViewMode(FormViewMode.Edit);
said on Июнь 10, 2008 at 13:50 · Permalink

В перлы для програмистов))) Сразу видно либо блондикнка кодила, либо тот кто недавно начал обучаться)

said on Октябрь 21, 2008 at 17:32 · Permalink

Не забывайте про индусов! Поговаривают, им платят за количество строк кода :)

said on Октябрь 24, 2008 at 20:12 · Permalink

if (Request.QueryString["memberpagemode"] == "edit") – это код полный бред, нифига не работает.

Riccon
said on Апрель 2, 2009 at 18:07 · Permalink

Если у него в QueryString нет искомого ключа, то, естественно, у него “нифига не работает”. Надобно на null проверить :)

Comments are closed

Comments for this entry are closed for a while. If you have anything to say – use a contact form. Thank you for your patience.