<?xml version="1.0" encoding="windows-1252"?>
<node id="624801" title="Re: Concatnate long statement" created="2007-07-03 18:10:26" updated="2007-07-03 14:10:26">
<type id="11">
note</type>
<author id="611550">
FunkyMonk</author>
<data>
<field name="doctext">
&lt;blockquote&gt;&lt;i&gt;&lt;/i&gt;&lt;/blockquote&gt;
&lt;code&gt;&lt;/code&gt;&lt;p&gt;
You really don't want a subroutine (it looks like that's what you're doing) with 18 arguments. It's going to cause you all sorts of problems when when you miss one of them out, or get some of them in the wrong order. That number of arguments really gives off a bad &lt;i&gt;code smell&lt;/i&gt;.&lt;p&gt;

Refactor your code &lt;b&gt;now&lt;/b&gt;, before it's too late.</field>
<field name="root_node">
624788</field>
<field name="parent_node">
624788</field>
</data>
</node>
