1. Ruby / Говнокод #11643

    −160

    1. 01
    2. 02
    3. 03
    4. 04
    5. 05
    6. 06
    7. 07
    8. 08
    9. 09
    10. 10
    11. 11
    12. 12
    13. 13
    14. 14
    15. 15
    16. 16
    17. 17
    18. 18
    19. 19
    20. 20
    21. 21
    22. 22
    23. 23
    24. 24
    25. 25
    26. 26
    27. 27
    28. 28
    29. 29
    30. 30
    31. 31
    32. 32
    33. 33
    34. 34
    35. 35
    36. 36
    37. 37
    38. 38
    39. 39
    40. 40
    41. 41
    42. 42
    43. 43
    44. 44
    45. 45
    46. 46
    47. 47
    48. 48
    49. 49
    50. 50
    <div class="comment-content">
  		<% if comment.body.include?('[/quote]') %>
  			<% @text = Array.new %>
  			<% summ = '' %>
  			<% @text = simple_format(comment.body).split(' ') %>
  			<% namecheck = false %>
  			<% contentcheck = false %>
  			<% quotestart = false %>
  			<% donothing = true %>
  			<% quotecount = 0 %>
  			<% count_quotes = 0 %>
  			<% @text.each do |t| %>
  				<% if t.include?('[/quote]') %>
  					<% count_quotes ||= 0 %>
  					<% count_quotes += 1 %>
  				<% end %>
  			<% end %>
  			<% @text.each do |t| %>
  				<% if t.include?('[/quote]') %>
  					<% contentcheck = false %>
  					<% namecheck = false %>
  					<% t = t.gsub('[/quote]', '</div></div>') %>
  				<% elsif t.include?('[quote=') %>
  					<% namecheck = true %>
  					<% donothing = false %>
  					<% t = t.gsub('[quote=', '') %>
  				<% elsif t.include?(']') %>
  					<% t = t.gsub(']', '') %>
  				<% end %>
  				<% if donothing == false %>
  					<% if contentcheck == true and namecheck == true and t != '' and t != '<p>' and t != '</p>' %>
  						<% t = '<div class=quotecontent>' + '<div class=quotestart>' + '<div class=quotename>' + 'Цитата - ' + t + '</div>' %>
  						<% contentcheck = true %>
  						<% namecheck = false %>
  					<% elsif contentcheck == true and t != '' and t != '<p>' and t != '</p>' %>
  						<% t = '<div class=quotecontent>' + t %>
  						<% contentcheck = false %>
  					<% elsif namecheck == true and t != '' and t != '<p>' and t != '</p>' %>
  						<% t = '<div class=quotestart>' + '<div class=quotename>' + 'Цитата - ' + t + '</div>' %>
  						<% namecheck = false %>
  						<% contentcheck = true %>
  					<% end %>
  				<% end %>
  				<% summ = summ + t + ' ' %>
  			<% end %>
  		<% else %>
  			<% summ = comment.body %>
  		<% end %>
    	<%=raw simple_format(summ) %>
    </div>

    Вьюха списка комментариев. Когда это попало к верстальщику - он был мягко говоря шокирован.

    Запостил: ruby-code, 24 Августа 2012

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

    • ava alex322 24.08.2012 10:06 # −6

      показать все, что скрытоНе понимаю, зачем такой шаблонизатор вообще было придумывать, лучше бы на голом PHP вывод сделали и не заморачивались с этим "чудо языком".
      Ответить

      • ava Lure Of Chaos 24.08.2012 10:21 # +5

        где здесь PHP, @alex322?
        диагноз: PHP головного мозга
        Ответить

        • ava alex322 24.08.2012 10:30 # −6

          показать все, что скрытоя написал "лучше бы на PHP"
          Ответить

          • ava Lure Of Chaos 24.08.2012 10:34 # +5

            с какого перепугу?
            руби - это искусство.
            Ответить

            • ava alex322 24.08.2012 16:11 # −4

              Извиняюсь, не обратил внимания что раздел ruby.

              Подумал очередной самопальный шаблонизатор для РНР. Просто с руби никогда не работал и синтаксиса его не знаю.
              Ответить

              • ava vistefan 28.08.2012 11:09 # +2

                > не обратил внимания что раздел ruby
                Даже после вопроса "Где здесь %language%, %username%?"
                Ответить

      • ava bormand 24.08.2012 10:28 # +3

        Нормальный в рубирельсах шаблонизатор. Вполне вменяемый.

        Просто один чудак на букву М втащил половину бизнес-логики во вьюху (хотя стоило бы запилить хелперы), поналепил кучу лишних открывающих и закрывающих тегов (видимо редактор бажный и не может подсветить многострочные), и умудрился куски HTML описать как строковые литералы (привет похапешникам).

        Если все это устранить - получится приличная вьюха, удобная для доработки верстальщиком.
        Ответить

        • ava vistefan 28.08.2012 11:26 # +4

          > один чудак на букву М
          На букву "М" у меня подобралась отличная коллекция. Морган, отравитель; Меридью, оставивший о себе жуткую память в виде коллекции скальпов; Мортимер, кокаинист и убийца...
          Ответить

        • ava 6oHo6o 11.03.2021 02:41 # 0

          Да нет, ERB и правда говно, но проблема конечно в высирании логики в шаблон. В хороших шаблонизаторах сделать это сложнее (попробуй сделать это в Zope или Angular), но и там эта проблема может случиться.

          Долбоёбы срут бизнес логикой в убогих тагах посреди HTMLя на всем пространстве веб-параши от Jinja до Blazor
          Ответить

    • ava roman-kashitsyn 24.08.2012 11:03 # 0

      Самые годные темплейты с точки зрения дизайнера а Lift. Вот, к примеру, реальный темплейт 
      http://tinyurl.com/9bu8oda
      Ответить

      • ava bormand 24.08.2012 11:40 # 0

        Эм, это же чистый HTML. А как в него набивать значения?
        Ответить

        • ava roman-kashitsyn 24.08.2012 11:58 # +2

          Я ждал этого вопроса. А вот так (scala-код осуществляющий привязку значений по css-селекторам): 
          private def defaultBindings(comment: Comment): CssSel = {
    ".comment-text *" #> bindCommentContent(comment) &
      ".author" #> comment.author.map(_.nickName.get).openOr("Guest") &
      ".date" #> TimeSpanFormatter(comment.createdAt) &
      ".comment-rating *" #> RateBinder(comment)(ratingTemplate) &
      ".avatar [src]" #> Avatar(comment.author, comment.ipAddress) &
      ".author-link [href]" #> comment.author.map(_.link).openOr("#") &
      ".comment-link [href]" #> comment.link &
      ".parent-link [href]" #> comment.responseTo.map(_.link).openOr("#") &
      bindDelete(comment) &
      ".unseen [class]" #> unseen(comment) &
      ".replies [id]" #> comment.repliesAnchor &
      ".comment [id]" #> comment.anchor
  }

          > Эм, это же чистый HTML
          В этом как раз весь профит для дизайнера ;)
          Ответить

          • ava Vasiliy 24.08.2012 12:06 # 0

            блоки без контента могут вести себя иначе чем с контентом.
            Ответить

            • ava roman-kashitsyn 24.08.2012 12:10 # 0 

              (if (!models.isEmpty) ".models *" #> bindAllTheItems(models)
else ".models *" #> <div class="sorry">Nothing here today :[</div>)
              Ответить

              • ava akostrikov 22.07.2013 18:09 # 0

                Прикольно, но реализация самого биндинга неприятна (субъективно).
                Ответить

          • ava bormand 24.08.2012 12:09 # 0

            CSS-подобное развешивание значений. Интересная идея.
            Ответить

      • ava Lure Of Chaos 24.08.2012 17:39 # 0

        > Самые годные темплейты с точки зрения дизайнера а Lift
        поглядел. Все же отсутствие плейсхолдеров запутывает, и, например, в визивиге не видно что где.
        Ответить

        • ava rat4 24.08.2012 18:33 # 0

          >Все же отсутствие плейсхолдеров запутывает, и, например, в визивиге не видно что где.

          Никто не запрещает их понатыкать.
          Ответить

    • ava HaskellGovno 24.08.2012 11:11 # 0

      %>
      Косит под С++, диграфы?
      Ответить

    • ava guest 17.08.2013 20:50 # 0

      Хелпер, не, не слышал
      Ответить

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