Говнокод #11577 — C# — Говнокод.ру

Говнокод: по колено в коде.

Нашли или выдавили из себя код, который нельзя назвать нормальным, на который без улыбки не взглянешь? Не торопитесь его удалять или рефакторить, — запостите его на говнокод.ру, посмеёмся вместе!

C# / Говнокод #11577

+107

public Quote[] ParseQuotesFromFile(string p){
            HtmlDocument d = new HtmlDocument();
            d.Load(p, new UTF8Encoding());
#if !DEBUG
            return
#endif
#if DEBUG
            var v1 =
#endif
 d.GetElementbyId("body")//main div
#if DEBUG
; var v2 = v1
#endif
.ChildNodes//subnodes
#if DEBUG
; var v3 = v2
#endif
.Where(a => a.Attributes.Contains("class"))//nodes with classes
#if DEBUG
; var v4 = v3
#endif
.Where(a => a.Attributes["class"].Value == "quote")//quotes
#if DEBUG
; var v5 = v4
#endif
.Where(a => a.ChildNodes.Any(b => b.Attributes.Contains("class")))//only real quotes
#if DEBUG
; var v6 = v5
#endif
.Select(a => a.ChildNodes.Where(b => b.Attributes.Contains("class")).ToArray())//only usable
#if DEBUG
; var v7 = v6
#endif
.Where(a => a.Length == 2)//drop ads
#if DEBUG
; var v8 = v7
#endif
.Select(a => new Quote()
{
    Text = conv.HtmlSpecialCharsDecode(a.First(b => b.Attributes["class"].Value == "text").InnerHtml.Replace("<br>", "\r\n"),StringTools.ReplaceType.Both),
    ID=BadParse(GetIdString(a)),
    Score=BadParse(GetScoreString(a))})
#if DEBUG
; var v9 = v8
#endif
.ToArray();
#if DEBUG
            return v9;
#endif
        } private static string GetScoreString(HtmlNode[] a){
#if !DEBUG
            return
#endif
#if DEBUG
            var v1 =
#endif
 a.First(b => b.Attributes["class"].Value == "actions")
#if DEBUG
; var v2 = v1
#endif
.ChildNodes.Where(b => b.Name != "#text")
#if DEBUG
; var v3 = v2
#endif
.First(b => b.Attributes["class"].Value == "rating-o")
#if DEBUG
; var v4 = v3
#endif
.FirstChild
#if DEBUG
; var v5 = v4
#endif
.InnerText;
#if DEBUG
            return v5;
#endif
        } private static string GetIdString(HtmlNode[] a) {
#if !DEBUG
            return
#endif
#if DEBUG
            var v1=
#endif
            a.First(b => b.Attributes["class"].Value == "actions")
#if DEBUG
; var v2 = v1
#endif
.ChildNodes.Where(b => b.Name != "#text")
#if DEBUG
; var v3 = v2
#endif
.First(b => b.Attributes["class"].Value == "id")
#if DEBUG
; var v4 = v3
#endif
                .InnerText.Replace("#", "");
#if DEBUG
            return v4;
#endif
}

Это должен был быть парсер bash.org.ru
Самоое странное, что код даже работает

Запостил:

kasthack, 12 Августа 2012

Комментарии (18) RSS

bormand 12.08.2012 19:06 # +1

А вот так уже не кошерно?

public Quote[] ParseQuotesFromFile(string p){
            HtmlDocument d = new HtmlDocument();
            d.Load(p, new UTF8Encoding());
            var v1 = d.GetElementbyId("body"); //main div
            var v2 = v1.ChildNodes; //subnodes
            var v3 = v2.Where(a => a.Attributes.Contains("class")); //nodes with classes
            var v4 = v3.Where(a => a.Attributes["class"].Value == "quote"); //quotes
            var v5 = v4.Where(a => a.ChildNodes.Any(b => b.Attributes.Contains("class"))); //only real quotes
            var v6 = v5.Select(a => a.ChildNodes.Where(b => b.Attributes.Contains("class")).ToArray()); //only usable
            var v7 = v6.Where(a => a.Length == 2); //drop ads
            var v8 = v7.Select(a => new Quote() {
                Text = conv.HtmlSpecialCharsDecode(a.First(b =>b.Attributes["class"].Value == "text").InnerHtml.Replace("<br>", "\r\n"),StringTools.ReplaceType.Both),
                ID=BadParse(GetIdString(a)),
                Score=BadParse(GetScoreString(a))
            });
            var v9 = v8.ToArray();
            return v9;
}

Ответить

HaskellGovno 12.08.2012 20:13 # +2

Нет конечно. AssParallol забыл.
Ответить
- bormand 12.08.2012 20:15 # +4
  
  AssTrollollel?
  Ответить
  - HaskellGovno 12.08.2012 20:21 # −2
    
    >AssTrollollel
    Кстати, очень даже созвучно:
    АссПараллел
    АссТроллоллел
    Только л вторая лишняя как мне кажется.
    Ответить

kasthack 12.08.2012 21:27 # 0

А теперь версия не для отладки

public Quote[] ParseQuotesFromFile(string p){
	HtmlDocument d = new HtmlDocument();
	d.Load(p, new UTF8Encoding());
	return  d.GetElementbyId("body")//main div
		.ChildNodes//subnodes
		.Where(a => a.Attributes.Contains("class"))//nodes with classes
		.Where(a => a.Attributes["class"].Value == "quote")
		.Where(a => a.ChildNodes.Any(b => b.Attributes.Contains("class")))
		.Select(a => a.ChildNodes.Where(b => b.Attributes.Contains("class")).ToArray())//only usable
		.Where(a => a.Length == 2)//drop ads
		.Select(a => new Quote()
		{
			Text = conv.HtmlSpecialCharsDecode(a.First(b => b.Attributes["class"].Value == "text").
				InnerHtml.Replace("<br>", "\r\n"),StringTools.ReplaceType.Both),
			ID=BadParse(GetIdString(a)),
			Score=BadParse(GetScoreString(a))
		})
		.ToArray();
}
private static string GetScoreString(HtmlNode[] a){
	return a.First(b => b.Attributes["class"].Value == "actions")
		.ChildNodes.Where(b => b.Name != "#text")
		.First(b => b.Attributes["class"].Value == "rating-o")
		.FirstChild
		.InnerText;
}
private static string GetIdString(HtmlNode[] a){
	return a.First(b => b.Attributes["class"].Value == "actions")
		.ChildNodes.Where(b => b.Name != "#text")
		.First(b => b.Attributes["class"].Value == "id")
		.InnerText.Replace("#", "");
}

Ответить

guest 14.08.2012 21:02 # 0

Быдло хоть чуть-чуть не освоило xpath
Ответить

eth0 12.08.2012 20:40 # +1

Боюсь себе представить, что там в неотладочной версии.
Ответить
- bormand 12.08.2012 20:47 # +6
  
  В неотладочной версии там... *барабанная дробь*... LINQ!
  Ответить
- kasthack 12.08.2012 21:03 # +1
  
  LINQ, но без промежуточных переменных.
  Писалось где-то в пять утра.
  Со сна показалось, что так проще дебажить
  Ответить
  - bormand 12.08.2012 21:05 # 0
    
    > Со сна показалось, что так проще дебажить
    А самое главное - проще поддерживать ;)
    Ответить
    - kasthack 12.08.2012 21:15 # 0
      
      Одноразовый код код же.
      С парсингом он справился, а дальше в /dev/null его.
      Проще потом нормально написать, если пригодится
      Ответить
      - bormand 12.08.2012 21:27 # +5
        
        > Одноразовый код код же.
        Ну и нафига тогда весь этот геморрой с дебагами? Чем мешали временные переменные в релизном режиме - даже не могу придумать.
        Ответить
roman-kashitsyn 12.08.2012 23:36 # +3

не проще ли было набросать xsl-преобразование, чем ковырять c#
Ответить
sayidandrtfm 13.08.2012 09:48 # +1

Посмотрел на такой парсер как на г-но.
Ответить
3.14159265 13.08.2012 13:40 # +2

ОЛОЛО LINQ НА БАШОРГ!!!
Простите не удержался.
Ответить
- Lure Of Chaos 13.08.2012 22:56 # 0
  башим!
  HTTP Error 301 - Moved permanently Please update your bookmarks
  Ответить
  - scriptin 13.08.2012 23:40 # +1
    
    >Error
    >301
    Ответить
  - rat4 14.08.2012 07:42 # 0
    
    $ wget bash.org
    --2012-08-14 08:41:23-- http://bash.org/
    Распознаётся bash.org... 69.61.106.93
    Подключение к bash.org|69.61.106.93|:80... соединение установлено.
    HTTP-запрос отправлен. Ожидание ответа... 200 OK
    Ответить

Говнокод: по колено в коде.

C# / Говнокод #11577

Комментарии (18) RSS

Добавить комментарий