why is this code bad?
<p>Below is a code in C# for rss reader, why this code is bad? this class generates a list of the 5 most recent posts, sorted by title. What do you use to analyze code in C#?</p>

<pre><code>    static Story[] Parse(string content)
    {
        var items = new List&lt;string&gt;();
        int start = 0;
        while (true)
        {

            var nextItemStart = content.IndexOf("&lt;item&gt;", start);
            var nextItemEnd = content.IndexOf("&lt;/item&gt;", nextItemStart);
            if (nextItemStart &lt; 0 || nextItemEnd &lt; 0) break;

            String nextItem = content.Substring(nextItemStart, nextItemEnd + 7 - nextItemStart);
            items.Add(nextItem);
            start = nextItemEnd;
        }

        var stories = new List&lt;Story&gt;();
        for (byte i = 0; i &lt; items.Count; i++)
        {
            stories.Add(new Story()
            {
                title = Regex.Match(items[i], "(?&lt;=&lt;title&gt;).*(?=&lt;/title&gt;)").Value,
                link = Regex.Match(items[i], "(?&lt;=&lt;link&gt;).*(?=&lt;/link&gt;)").Value,
                date = Regex.Match(items[i], "(?&lt;=&lt;pubDate&gt;).*(?=&lt;/pubdate&gt;)").Value
            });
        }

        return stories.ToArray();
    }
</code></pre>
