Pastable version of Helper Method could be more efficient

<pre class='prettyprint' language='ruby'>def build_criteria(data_cols, i)
     filterHTML = ''
     typeObjects = ''
     data_cols.each do |d|
       typeObjects += &quot;&lt;option value='#{d}'&gt;#{d}&lt;/option&gt;&quot;
     end

filterHTML += &quot;&lt;div id='additionalcriteria#{i}'&gt;&quot;
     filterHTML +=  &quot;&lt;select name='dataType#{i}'&gt;#{typeObjects}&lt;/select&gt;&quot;
     filterHTML += select_tag (&quot;operator#{i}&quot;, &quot;
                                 &lt;option value='&lt;'&gt;is less than&lt;/option&gt;
                                 &lt;option value='&lt;='&gt;is less than or equal to&lt;/option&gt;
                                 &lt;option value='=='&gt;is equal to&lt;/option&gt;
                                 &lt;option value='&gt;='&gt;is greater than or equal to&lt;/option&gt;&quot;)
     filterHTML += text_field_tag(&quot;value#{i}&quot;)
     filterHTML += &quot;&lt;/div&gt;&quot;
end</pre>
<a href="http://www.refactormycode.com/codes/841-helper-method-could-be-more-efficient" style="color:#fff" title="As seen on RefactorMyCode.com"><img alt="Small_logo" src="http://www.refactormycode.com/images/small_logo.gif" style="border:0" /></a>

Refactor :my => 'code'

Recent

Popular

Pastable version of Helper Method could be more efficient