<?xml version="1.0" encoding="utf-8"?>
<rss version="2.0" xmlns:content="http://purl.org/rss/1.0/modules/content/">
    <channel>
        <title>DOM Scripting comments: Tell me my typos</title>
        <link>http://domscripting.com/blog/display/29</link>
        <description>The book is going to be reprinted and I need to know what needs fixing.</description>
        <language>en</language>
        <item>
            <title>Jeremy Keith</title>
            <link>http://domscripting.com/blog/display.php/29#comment122</link>
            <content:encoded><![CDATA[<p>Guy, we should definitely get together for a beer sometime.</p>

<p>Thanks for pointing out that typo for page 107. Unfortunately it&#8217;s too late now to get it into the reprint but I&#8217;ll add to my list.</p>

<p>As for the whole toggling of the &quot;odd&quot; variable, yes, I did do it for pedagogical, rather than coding reasons.</p>

<p>You&#8217;re right: the changes you suggest would be more elegant, but then I would also have to explain exactly what was going on. By simply writing code that&#8217;s close to the English, &quot;if odd is equal to true&#8230;&quot;, the flow of the logic is fairly clear without any extra explanation.</p>

<p>There&#8217;s quite a few places in the book where the code could be shortened and improved considerably, but I generally chose readability and clarity at the expense of code elegance.</p>

<p>Also, I&#8217;m sure many programmers would balk at my constant flaunting of structured programming best practices. Again, it was readability I was striving for.</p>

<p>Now, about that beer&#8230;</p>
]]></content:encoded>
            <pubDate>Wed, 09 Nov 2005 20:24:15 GMT</pubDate>
        </item>
        <item>
            <title>Guy McCusker</title>
            <link>http://domscripting.com/blog/display.php/29#comment114</link>
            <content:encoded><![CDATA[<p>I guess I am too late with these but I only found the book in Brighton Waterstone&#8217;s this weekend.</p>

<p>p107 there&#8217;s a line</p>

<p>if (whichpic.getAttribue(&quot;title&quot;))</p>

<p>which should clearly say &quot;getAttribute&quot;.</p>

<p>p196, nothing incorrect but you&#8217;ve done something that makes me shout at my year 1 programming undergrads:</p>

<p>if (odd == true) </p>

<p>should just be</p>

<p>if (odd)</p>

<p>This may be a pedagogical decision on your part I suppose but I strongly object to using == to check boolean data.</p>

<p>Also I&#8217;d prefer the two lines</p>

<p>odd = false;
and
odd = true;</p>

<p>to come outside the if-statement and coexist as</p>

<p>odd = !odd;</p>

<p>which makes it clear that you&#8217;re toggling.</p>

<p>Very nice book, clear explanations etc. I want to buy you a beer in Brighton sometime.</p>

<p>Guy.</p>
]]></content:encoded>
            <pubDate>Tue, 08 Nov 2005 14:19:39 GMT</pubDate>
        </item>
        <item>
            <title>Kevin</title>
            <link>http://domscripting.com/blog/display.php/29#comment112</link>
            <content:encoded><![CDATA[<p>Probably too late to add (and also pretty minor)&#8230;.but I just got your book the other day. I noticed that at the end of chpt 4 you provide a picture of how the image gallery looks after the mark up is applied. I believe that by using the float: left; in the li tag, the placeholder image will float left as well, unless the float is cleared (atleast this is how it acted in my browser..safari for mac). I just created a .seperate { clear: both;} and applied the class prior to the placeholder. This way the placeholder clears the float and will align under the links, not next to them.</p>

<p>Enjoying the book very much.</p>
]]></content:encoded>
            <pubDate>Sun, 06 Nov 2005 04:03:54 GMT</pubDate>
        </item>
        <item>
            <title>Shane Bonham</title>
            <link>http://domscripting.com/blog/display.php/29#comment103</link>
            <content:encoded><![CDATA[<p>Pretty minor, but page 25 (of my edition) lists the arithmetic operators and refers to the operator for division as a backslash.  I believe that should be a <em>forward</em>-slash (or simply, slash).</p>
]]></content:encoded>
            <pubDate>Mon, 31 Oct 2005 21:37:12 GMT</pubDate>
        </item>
        <item>
            <title>Dustin</title>
            <link>http://domscripting.com/blog/display.php/29#comment99</link>
            <content:encoded><![CDATA[<p>Doh. I&#8217;ll need to finish quickly then. I have some other typos as well&#8230;just hold&#8230;one&#8230;more&#8230;second&#8230;</p>
]]></content:encoded>
            <pubDate>Thu, 27 Oct 2005 06:00:33 GMT</pubDate>
        </item>
   </channel>
</rss>