Crazy piece of code: Checking query parameter

Posted by Dmytro Shteflyuk on under ASP.NET

How often you are laughing out loud when looking on the someone’s code? Today I found great code in my current project and I can’t hold posting this to my blog. So,

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

And how do you search through the hash for a key with specified value?

13 Responses to this entry

Subscribe to comments with RSS

said on December 18th, 2007 at 08:17 · Permalink

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

said on December 25th, 2007 at 07:18 · Permalink

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

said on December 25th, 2007 at 16:07 · Permalink

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

said on May 8th, 2008 at 15:07 · Permalink

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

said on May 19th, 2008 at 14:45 · Permalink

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

Pavel
said on May 30th, 2008 at 17:59 · Permalink

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

said on May 31st, 2008 at 08:39 · Permalink

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

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

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

said on October 21st, 2008 at 17:32 · Permalink

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

said on October 24th, 2008 at 20:12 · Permalink

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

Riccon
said on April 2nd, 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.