<?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: Рецензирование кода (code review).</title>
	<atom:link href="http://blog.not-a-kernel-guy.com/2007/02/21/151/feed" rel="self" type="application/rss+xml" />
	<link>http://blog.not-a-kernel-guy.com/2007/02/21/151</link>
	<description>... также известный как &#34;Not a kernel guy&#34;</description>
	<lastBuildDate>Thu, 11 Mar 2010 19:19:40 -0800</lastBuildDate>
	<generator>http://wordpress.org/?v=2.8.6</generator>
	<sy:updatePeriod>hourly</sy:updatePeriod>
	<sy:updateFrequency>1</sy:updateFrequency>
		<item>
		<title>By: Not a kernel guy</title>
		<link>http://blog.not-a-kernel-guy.com/2007/02/21/151#comment-1888</link>
		<dc:creator>Not a kernel guy</dc:creator>
		<pubDate>Thu, 22 Feb 2007 16:10:28 +0000</pubDate>
		<guid isPermaLink="false">http://blog.not-a-kernel-guy.com/2007/02/21/151#comment-1888</guid>
		<description>&#160;&lt;blockquote&gt;2 Dmitriy
программ Диффов в общем - то навалом, мне например нравится связка
SVN + TRACK, и версионность и просмотр дифов и багтрекинг и планирование, плюс еще можно к TRACKу много чего наворотить.&lt;/blockquote&gt;

+ 1 за Subversion &amp; Trac. Я как раз пишу следующий пост о том, как организовать рецензирование в Trac.</description>
		<content:encoded><![CDATA[<p>&nbsp;<br />
<blockquote>2 Dmitriy<br />
программ Диффов в общем &#8211; то навалом, мне например нравится связка<br />
SVN + TRACK, и версионность и просмотр дифов и багтрекинг и планирование, плюс еще можно к TRACKу много чего наворотить.</p></blockquote>
<p>+ 1 за Subversion &#038; Trac. Я как раз пишу следующий пост о том, как организовать рецензирование в Trac.</p>
]]></content:encoded>
	</item>
	<item>
		<title>By: Not a kernel guy</title>
		<link>http://blog.not-a-kernel-guy.com/2007/02/21/151#comment-1887</link>
		<dc:creator>Not a kernel guy</dc:creator>
		<pubDate>Thu, 22 Feb 2007 16:09:09 +0000</pubDate>
		<guid isPermaLink="false">http://blog.not-a-kernel-guy.com/2007/02/21/151#comment-1887</guid>
		<description>&#160;
&lt;blockquote&gt;
&lt;blockquote&gt;Так в этом всё и дело. Код должен быть таким, чтобы автор для его понимания не требовался.&lt;/blockquote&gt;

конечно да, но если код такой, какой он есть, то разбираться лучше с автором :).
&lt;/blockquote&gt;


Но если код &quot;такой, какой он есть&quot;, то код не пройдет review. Автор конечно может наплевать на мнение рецнгзента и не исправлять такой код, но на моеё памяти такого не бывало.</description>
		<content:encoded><![CDATA[<p>&nbsp;</p>
<blockquote>
<blockquote><p>Так в этом всё и дело. Код должен быть таким, чтобы автор для его понимания не требовался.</p></blockquote>
<p>конечно да, но если код такой, какой он есть, то разбираться лучше с автором <img src='http://blog.not-a-kernel-guy.com/wp-includes/images/smilies/icon_smile.gif' alt=':)' class='wp-smiley' /> .
</p></blockquote>
<p>Но если код &#8220;такой, какой он есть&#8221;, то код не пройдет review. Автор конечно может наплевать на мнение рецнгзента и не исправлять такой код, но на моеё памяти такого не бывало.</p>
]]></content:encoded>
	</item>
	<item>
		<title>By: ryfm</title>
		<link>http://blog.not-a-kernel-guy.com/2007/02/21/151#comment-1871</link>
		<dc:creator>ryfm</dc:creator>
		<pubDate>Thu, 22 Feb 2007 09:21:06 +0000</pubDate>
		<guid isPermaLink="false">http://blog.not-a-kernel-guy.com/2007/02/21/151#comment-1871</guid>
		<description>2 Dmitriy
программ Диффов в общем - то навалом, мне например нравится связка
SVN + TRACK, и версионность и просмотр дифов и багтрекинг и планирование, плюс еще можно к TRACKу много чего наворотить.</description>
		<content:encoded><![CDATA[<p>2 Dmitriy<br />
программ Диффов в общем &#8211; то навалом, мне например нравится связка<br />
SVN + TRACK, и версионность и просмотр дифов и багтрекинг и планирование, плюс еще можно к TRACKу много чего наворотить.</p>
]]></content:encoded>
	</item>
	<item>
		<title>By: ryfm</title>
		<link>http://blog.not-a-kernel-guy.com/2007/02/21/151#comment-1870</link>
		<dc:creator>ryfm</dc:creator>
		<pubDate>Thu, 22 Feb 2007 09:18:46 +0000</pubDate>
		<guid isPermaLink="false">http://blog.not-a-kernel-guy.com/2007/02/21/151#comment-1870</guid>
		<description>&#160;&lt;blockquote&gt;Так в этом всё и дело. Код должен быть таким, чтобы автор для его понимания не требовался.&lt;/blockquote&gt;
конечно да, но если код такой, какой он есть, то разбираться лучше с автором :).</description>
		<content:encoded><![CDATA[<p>&nbsp;<br />
<blockquote>Так в этом всё и дело. Код должен быть таким, чтобы автор для его понимания не требовался.</p></blockquote>
<p>конечно да, но если код такой, какой он есть, то разбираться лучше с автором <img src='http://blog.not-a-kernel-guy.com/wp-includes/images/smilies/icon_smile.gif' alt=':)' class='wp-smiley' /> .</p>
]]></content:encoded>
	</item>
	<item>
		<title>By: ryfm</title>
		<link>http://blog.not-a-kernel-guy.com/2007/02/21/151#comment-1869</link>
		<dc:creator>ryfm</dc:creator>
		<pubDate>Thu, 22 Feb 2007 09:16:41 +0000</pubDate>
		<guid isPermaLink="false">http://blog.not-a-kernel-guy.com/2007/02/21/151#comment-1869</guid>
		<description>to G2 
там просто еще несколько циклов добавляющих условие фильтрации, а в конце лишний OR просто отрезается.</description>
		<content:encoded><![CDATA[<p>to G2<br />
там просто еще несколько циклов добавляющих условие фильтрации, а в конце лишний OR просто отрезается.</p>
]]></content:encoded>
	</item>
	<item>
		<title>By: Dmitry</title>
		<link>http://blog.not-a-kernel-guy.com/2007/02/21/151#comment-1868</link>
		<dc:creator>Dmitry</dc:creator>
		<pubDate>Thu, 22 Feb 2007 08:58:58 +0000</pubDate>
		<guid isPermaLink="false">http://blog.not-a-kernel-guy.com/2007/02/21/151#comment-1868</guid>
		<description>В последнем абзаце говорится о удобных инстументах, может назовете парочку для примера?</description>
		<content:encoded><![CDATA[<p>В последнем абзаце говорится о удобных инстументах, может назовете парочку для примера?</p>
]]></content:encoded>
	</item>
	<item>
		<title>By: G2</title>
		<link>http://blog.not-a-kernel-guy.com/2007/02/21/151#comment-1863</link>
		<dc:creator>G2</dc:creator>
		<pubDate>Thu, 22 Feb 2007 07:56:01 +0000</pubDate>
		<guid isPermaLink="false">http://blog.not-a-kernel-guy.com/2007/02/21/151#comment-1863</guid>
		<description>вылилось в 

&lt;pre&gt;&lt;code class=&quot;cpp&quot;&gt;foreach (DictionaryEntry entry in csyndbf)
{
    tablesFilter += String.Format(”@Dict = ‘{0}’ or “, entry.Key);
}
&lt;/code&gt;&lt;/pre&gt;


Фильтр будет правильно работать, если в конце будет ‘’…or @Dict = ‘key’ or “</description>
		<content:encoded><![CDATA[<p>вылилось в </p>
<pre><code class="cpp">foreach (DictionaryEntry entry in csyndbf)
{
    tablesFilter += String.Format(”@Dict = ‘{0}’ or “, entry.Key);
}
</code></pre>
<p>Фильтр будет правильно работать, если в конце будет ‘’…or @Dict = ‘key’ or “</p>
]]></content:encoded>
	</item>
	<item>
		<title>By: Not a kernel guy</title>
		<link>http://blog.not-a-kernel-guy.com/2007/02/21/151#comment-1861</link>
		<dc:creator>Not a kernel guy</dc:creator>
		<pubDate>Thu, 22 Feb 2007 05:26:11 +0000</pubDate>
		<guid isPermaLink="false">http://blog.not-a-kernel-guy.com/2007/02/21/151#comment-1861</guid>
		<description>&#160;&lt;blockquote&gt;тем более в таком случае сидя рядом с автором быстрее понимаешь код.&lt;/blockquote&gt;

Так в этом всё и дело. Код должен быть таким, чтобы автор для его понимания не требовался.</description>
		<content:encoded><![CDATA[<p>&nbsp;<br />
<blockquote>тем более в таком случае сидя рядом с автором быстрее понимаешь код.</p></blockquote>
<p>Так в этом всё и дело. Код должен быть таким, чтобы автор для его понимания не требовался.</p>
]]></content:encoded>
	</item>
	<item>
		<title>By: ryfm</title>
		<link>http://blog.not-a-kernel-guy.com/2007/02/21/151#comment-1860</link>
		<dc:creator>ryfm</dc:creator>
		<pubDate>Thu, 22 Feb 2007 05:24:51 +0000</pubDate>
		<guid isPermaLink="false">http://blog.not-a-kernel-guy.com/2007/02/21/151#comment-1860</guid>
		<description>не согласен с тем, что вариант когда оба двое сидят за одним монитором плох, мне по-крайней мере больше нравится чем диффы, тем более в таком случае сидя рядом с автором быстрее понимаешь код.

из вчерашнего :)

&lt;pre&gt;&lt;code class=&quot;cpp&quot;&gt;IDictionaryEnumerator TN = csyndbf.GetEnumerator();
string tablefilt = &quot;&quot;;
bool flag = true;
while (TN.MoveNext())
{
    if (flag)
    {
        flag = false;
        tablefilt += String.Format(&quot;@Dict = &#039;{0}&#039;&quot;, TN.Key);
    }
    tablefilt += String.Format(&quot; or @Dict = &#039;{0}&#039;&quot;, TN.Key);
}
&lt;/code&gt;&lt;/pre&gt;

вылилось в 

&lt;pre&gt;&lt;code class=&quot;cpp&quot;&gt;foreach (DictionaryEntry entry in csyndbf)
{
    tablesFilter += 
        String.Format(&quot;@Dict = &#039;{0}&#039; or &quot;, entry.Key);
}
&lt;/code&gt;&lt;/pre&gt;

хотя тут конечно и понимать нечего, просто человек по жизни пишет на sql а тут понадобилось модуль сваять на шарпе :)</description>
		<content:encoded><![CDATA[<p>не согласен с тем, что вариант когда оба двое сидят за одним монитором плох, мне по-крайней мере больше нравится чем диффы, тем более в таком случае сидя рядом с автором быстрее понимаешь код.</p>
<p>из вчерашнего <img src='http://blog.not-a-kernel-guy.com/wp-includes/images/smilies/icon_smile.gif' alt=':)' class='wp-smiley' /> </p>
<pre><code class="cpp">IDictionaryEnumerator TN = csyndbf.GetEnumerator();
string tablefilt = "";
bool flag = true;
while (TN.MoveNext())
{
    if (flag)
    {
        flag = false;
        tablefilt += String.Format("@Dict = '{0}'", TN.Key);
    }
    tablefilt += String.Format(" or @Dict = '{0}'", TN.Key);
}
</code></pre>
<p>вылилось в </p>
<pre><code class="cpp">foreach (DictionaryEntry entry in csyndbf)
{
    tablesFilter +=
        String.Format("@Dict = '{0}' or ", entry.Key);
}
</code></pre>
<p>хотя тут конечно и понимать нечего, просто человек по жизни пишет на sql а тут понадобилось модуль сваять на шарпе <img src='http://blog.not-a-kernel-guy.com/wp-includes/images/smilies/icon_smile.gif' alt=':)' class='wp-smiley' /> </p>
]]></content:encoded>
	</item>
	<item>
		<title>By: Not a kernel guy</title>
		<link>http://blog.not-a-kernel-guy.com/2007/02/21/151#comment-1849</link>
		<dc:creator>Not a kernel guy</dc:creator>
		<pubDate>Thu, 22 Feb 2007 03:36:32 +0000</pubDate>
		<guid isPermaLink="false">http://blog.not-a-kernel-guy.com/2007/02/21/151#comment-1849</guid>
		<description>Согласен. Особенно когда обсуждается не то, где правильней всего ставить скобки и нужно ли использовать венгерскую нотацию, а то как следует использовать стандартные структуры и алгоритмы.</description>
		<content:encoded><![CDATA[<p>Согласен. Особенно когда обсуждается не то, где правильней всего ставить скобки и нужно ли использовать венгерскую нотацию, а то как следует использовать стандартные структуры и алгоритмы.</p>
]]></content:encoded>
	</item>
</channel>
</rss>
