<?xml version="1.0" encoding="utf-8"?><rss version="2.0"
	xmlns:content="http://purl.org/rss/1.0/modules/content/"
	xmlns:dc="http://purl.org/dc/elements/1.1/"
	xmlns:atom="http://www.w3.org/2005/Atom"
	xmlns:sy="http://purl.org/rss/1.0/modules/syndication/"
		>
<channel>
	<title>Comments on: Bad Code Smells in WACT, a refactoring review</title>
	<atom:link href="http://www.procata.com/blog/archives/2004/10/18/bad-code-smells-in-wact-a-refactoring-review/feed/" rel="self" type="application/rss+xml" />
	<link>http://www.procata.com/blog/archives/2004/10/18/bad-code-smells-in-wact-a-refactoring-review/</link>
	<description>PHP Programming, Web Development, PHP Advocacy and PHP Best Practices.</description>
	<lastBuildDate>Sat, 11 Feb 2012 14:53:56 -0500</lastBuildDate>
	<generator>http://wordpress.org/?v=2.8.6</generator>
	<sy:updatePeriod>hourly</sy:updatePeriod>
	<sy:updateFrequency>1</sy:updateFrequency>
		<item>
		<title>By: Breana Brannigan</title>
		<link>http://www.procata.com/blog/archives/2004/10/18/bad-code-smells-in-wact-a-refactoring-review/#comment-86485</link>
		<dc:creator>Breana Brannigan</dc:creator>
		<pubDate>Tue, 10 Jan 2012 12:14:59 +0000</pubDate>
		<guid isPermaLink="false">http://www.procata.com/blog/archives/2004/10/18/bad-code-smells-in-wact-a-refactoring-review/#comment-86485</guid>
		<description>Jonny Dorey. VCU student. Found in the James river, Hopewell Marina Virginia.</description>
		<content:encoded><![CDATA[<p>Jonny Dorey. VCU student. Found in the James river, Hopewell Marina Virginia.</p>
]]></content:encoded>
	</item>
	<item>
		<title>By: Fax Free Internet</title>
		<link>http://www.procata.com/blog/archives/2004/10/18/bad-code-smells-in-wact-a-refactoring-review/#comment-86332</link>
		<dc:creator>Fax Free Internet</dc:creator>
		<pubDate>Thu, 29 Dec 2011 07:39:31 +0000</pubDate>
		<guid isPermaLink="false">http://www.procata.com/blog/archives/2004/10/18/bad-code-smells-in-wact-a-refactoring-review/#comment-86332</guid>
		<description>somebody essentially help to make severely posts i&#039;d state. this is the first time i frequented your website page and up to now? i amazed with the research you made to create this particular submit amazing. great activity!</description>
		<content:encoded><![CDATA[<p>somebody essentially help to make severely posts i&#8217;d state. this is the first time i frequented your website page and up to now? i amazed with the research you made to create this particular submit amazing. great activity!</p>
]]></content:encoded>
	</item>
	<item>
		<title>By: lashes</title>
		<link>http://www.procata.com/blog/archives/2004/10/18/bad-code-smells-in-wact-a-refactoring-review/#comment-86185</link>
		<dc:creator>lashes</dc:creator>
		<pubDate>Tue, 22 Nov 2011 06:42:07 +0000</pubDate>
		<guid isPermaLink="false">http://www.procata.com/blog/archives/2004/10/18/bad-code-smells-in-wact-a-refactoring-review/#comment-86185</guid>
		<description>Its like you study my mind! You appear to understand so substantially about this, like you wrote the book in it or anything. I assume that you simply could do with some pics to drive the message house a bit, but rather of that, this can be excellent blog. A excellent read. I&#039;ll undoubtedly be back.</description>
		<content:encoded><![CDATA[<p>Its like you study my mind! You appear to understand so substantially about this, like you wrote the book in it or anything. I assume that you simply could do with some pics to drive the message house a bit, but rather of that, this can be excellent blog. A excellent read. I&#8217;ll undoubtedly be back.</p>
]]></content:encoded>
	</item>
	<item>
		<title>By: clamshell packaging</title>
		<link>http://www.procata.com/blog/archives/2004/10/18/bad-code-smells-in-wact-a-refactoring-review/#comment-85984</link>
		<dc:creator>clamshell packaging</dc:creator>
		<pubDate>Fri, 11 Nov 2011 00:11:48 +0000</pubDate>
		<guid isPermaLink="false">http://www.procata.com/blog/archives/2004/10/18/bad-code-smells-in-wact-a-refactoring-review/#comment-85984</guid>
		<description>It’s hard to find knowledgeable people on this topic, but you sound like you know what you’re talking about! Thanks</description>
		<content:encoded><![CDATA[<p>It’s hard to find knowledgeable people on this topic, but you sound like you know what you’re talking about! Thanks</p>
]]></content:encoded>
	</item>
	<item>
		<title>By: military electronics</title>
		<link>http://www.procata.com/blog/archives/2004/10/18/bad-code-smells-in-wact-a-refactoring-review/#comment-85983</link>
		<dc:creator>military electronics</dc:creator>
		<pubDate>Fri, 11 Nov 2011 00:11:36 +0000</pubDate>
		<guid isPermaLink="false">http://www.procata.com/blog/archives/2004/10/18/bad-code-smells-in-wact-a-refactoring-review/#comment-85983</guid>
		<description>Witty! I&#039;m bookmarking you site for future use.</description>
		<content:encoded><![CDATA[<p>Witty! I&#8217;m bookmarking you site for future use.</p>
]]></content:encoded>
	</item>
	<item>
		<title>By: DIPTI.MALI</title>
		<link>http://www.procata.com/blog/archives/2004/10/18/bad-code-smells-in-wact-a-refactoring-review/#comment-10309</link>
		<dc:creator>DIPTI.MALI</dc:creator>
		<pubDate>Wed, 27 Jul 2005 14:36:01 +0000</pubDate>
		<guid isPermaLink="false">http://www.procata.com/blog/archives/2004/10/18/bad-code-smells-in-wact-a-refactoring-review/#comment-10309</guid>
		<description>I am doing a project where i am anaylsing how bad code documenatation becomes a nightmare for new/future developer.
i would be obliged if u can help me in any form.
thanks..........</description>
		<content:encoded><![CDATA[<p>I am doing a project where i am anaylsing how bad code documenatation becomes a nightmare for new/future developer.<br />
i would be obliged if u can help me in any form.<br />
thanks&#8230;&#8230;&#8230;.</p>
]]></content:encoded>
	</item>
	<item>
		<title>By: Harry Fuecks</title>
		<link>http://www.procata.com/blog/archives/2004/10/18/bad-code-smells-in-wact-a-refactoring-review/#comment-298</link>
		<dc:creator>Harry Fuecks</dc:creator>
		<pubDate>Tue, 26 Oct 2004 12:01:31 +0000</pubDate>
		<guid isPermaLink="false">http://www.procata.com/blog/archives/2004/10/18/bad-code-smells-in-wact-a-refactoring-review/#comment-298</guid>
		<description>Great write up. Should confess I&#039;m guilty in most cases - probably Shotgun Surgery.

Another one to watch is the relationship between ExpressionFilterFindingParser and ExpressionFilterParser (expression.inc.php) and in the latter, the ways Args is used is dubious.

Off the specific topic of bad smells, some particular issues;

The way compileall.inc.php works also needs review I think. For a start seems to bypass various code in a way that makes it&#039;s behaviour unpredictable plus that constant WACT_ERROR_CONTINUE is a hack.

Find &lt;a href=&quot;http://sourceforge.net/tracker/index.php?func=detail&amp;aid=1000806&amp;group_id=85372&amp;atid=575984&quot;&gt;Jason&#039;s list / script bug&lt;/a&gt; unsettling. The new HTMLParser is generally a big improvement but it&#039;s now got embedded knowledge of HTML (the script tag). Somehow think we&#039;re going to need to have a second stage of parsing. Also what about other document formats (where there happens to be some kind of script tag)?

Wonder if we need to consider some kind of document detection at last. There&#039;s still the issue with &lt;a href=&quot;http://sourceforge.net/tracker/index.php?func=detail&amp;aid=920756&amp;group_id=85372&amp;atid=575984&quot;&gt;form tags and XHTML&lt;/a&gt;. And if a document does claim to be well formed XML, perhaps we can switch to a real XML parser?</description>
		<content:encoded><![CDATA[<p>Great write up. Should confess I&#8217;m guilty in most cases &#8211; probably Shotgun Surgery.</p>
<p>Another one to watch is the relationship between ExpressionFilterFindingParser and ExpressionFilterParser (expression.inc.php) and in the latter, the ways Args is used is dubious.</p>
<p>Off the specific topic of bad smells, some particular issues;</p>
<p>The way compileall.inc.php works also needs review I think. For a start seems to bypass various code in a way that makes it&#8217;s behaviour unpredictable plus that constant WACT_ERROR_CONTINUE is a hack.</p>
<p>Find <a href="http://sourceforge.net/tracker/index.php?func=detail&#038;aid=1000806&#038;group_id=85372&#038;atid=575984">Jason&#8217;s list / script bug</a> unsettling. The new HTMLParser is generally a big improvement but it&#8217;s now got embedded knowledge of HTML (the script tag). Somehow think we&#8217;re going to need to have a second stage of parsing. Also what about other document formats (where there happens to be some kind of script tag)?</p>
<p>Wonder if we need to consider some kind of document detection at last. There&#8217;s still the issue with <a href="http://sourceforge.net/tracker/index.php?func=detail&#038;aid=920756&#038;group_id=85372&#038;atid=575984">form tags and XHTML</a>. And if a document does claim to be well formed XML, perhaps we can switch to a real XML parser?</p>
]]></content:encoded>
	</item>
	<item>
		<title>By: Marcus Baker</title>
		<link>http://www.procata.com/blog/archives/2004/10/18/bad-code-smells-in-wact-a-refactoring-review/#comment-286</link>
		<dc:creator>Marcus Baker</dc:creator>
		<pubDate>Mon, 25 Oct 2004 18:31:45 +0000</pubDate>
		<guid isPermaLink="false">http://www.procata.com/blog/archives/2004/10/18/bad-code-smells-in-wact-a-refactoring-review/#comment-286</guid>
		<description>The bit that I&#039;ve never understood is, what is a DataSpace? Do you have a release date planned for the next version?</description>
		<content:encoded><![CDATA[<p>The bit that I&#8217;ve never understood is, what is a DataSpace? Do you have a release date planned for the next version?</p>
]]></content:encoded>
	</item>
	<item>
		<title>By: pachanga</title>
		<link>http://www.procata.com/blog/archives/2004/10/18/bad-code-smells-in-wact-a-refactoring-review/#comment-277</link>
		<dc:creator>pachanga</dc:creator>
		<pubDate>Tue, 19 Oct 2004 11:04:55 +0000</pubDate>
		<guid isPermaLink="false">http://www.procata.com/blog/archives/2004/10/18/bad-code-smells-in-wact-a-refactoring-review/#comment-277</guid>
		<description>Maybe it&#039;s a bit offtopic, yet i find the following feature very important and nice to have in WACT - custom tags loader.
Something like WACT :: registerTagsLoader($loader);
WACT could use some default tags loader if none is registered. 
This way it&#039;ll simplify extending WACT by 3d party software tools...</description>
		<content:encoded><![CDATA[<p>Maybe it&#8217;s a bit offtopic, yet i find the following feature very important and nice to have in WACT &#8211; custom tags loader.<br />
Something like WACT :: registerTagsLoader($loader);<br />
WACT could use some default tags loader if none is registered.<br />
This way it&#8217;ll simplify extending WACT by 3d party software tools&#8230;</p>
]]></content:encoded>
	</item>
	<item>
		<title>By: Jeff Moore's Blog &#187; Status of WACT</title>
		<link>http://www.procata.com/blog/archives/2004/10/18/bad-code-smells-in-wact-a-refactoring-review/#comment-287</link>
		<dc:creator>Jeff Moore's Blog &#187; Status of WACT</dc:creator>
		<pubDate>Wed, 31 Dec 1969 19:00:00 +0000</pubDate>
		<guid isPermaLink="false">http://www.procata.com/blog/archives/2004/10/18/bad-code-smells-in-wact-a-refactoring-review/#comment-287</guid>
		<description>[...] of WACT 	Filed under:  	PHP 	Software Development &#8212; admin @ 2:33 pm  	 	 			My last &lt;a href=&#039;http://www.procata.com/blog/archives/2004/10/18/bad-code-smells-in-wact-a-refactoring-review/&#039;&gt;WACT&lt;/a&gt; post drew a couple questions from Marcus Baker [...]</description>
		<content:encoded><![CDATA[<p>[...] of WACT 	Filed under:  	PHP 	Software Development &#8212; admin @ 2:33 pm  	 	 			My last <a href='http://www.procata.com/blog/archives/2004/10/18/bad-code-smells-in-wact-a-refactoring-review/'>WACT</a> post drew a couple questions from Marcus Baker [...]</p>
]]></content:encoded>
	</item>
</channel>
</rss>

