Click here to Skip to main content

Welcome to the Lounge

   

For lazing about and discussing anything in a software developer's life that takes your fancy.
The Lounge is rated PG. If you're about to post something you wouldn't want your kid sister to read then don't post it. No flame wars, no abusive conduct and please don't post ads.

Technical discussions are welcome, but if you need specific help please ask your question here.


 
GeneralRe: Do you know Akpors? Pinmemberjeron12-May-13 9:07 
GeneralRe: Do you know Akpors? PinmentorKeith Barrow2-May-13 12:41 
GeneralRe: Do you know Akpors? PinmemberBobJanova3-May-13 0:30 
RantAnother open source rant PinprotectorMarc Clifton2-May-13 8:22 
This time, my sights are on HtmlAgilityPack[^].
 
Besides the fact that there is no documentation and the one example provided has an absurd quote-placement error, so obviously the code was never tested, there's this:
 
/// <summary>
/// Selects a list of nodes matching the <see cref="XPath"/> expression.
/// </summary>
/// <param name="xpath">The XPath expression.</param>
/// <returns>An <see cref="HtmlNodeCollection"/> containing a collection of nodes matching the <see cref="XPath"/> query, or <c>null</c> if no node matched the XPath expression.</returns>
public HtmlNodeCollection SelectNodes(string xpath)
 
What? You return a null if no matches are found???
 
Which of course would blow up their "example" if no "href" tags exist:
 
HtmlDocument doc = new HtmlDocument();
 doc.Load("file.htm");
 foreach(HtmlNode link in doc.DocumentElement.SelectNodes("//a[@href"])
 {
 
Why is it that the quality of what "we" as supposed professionals produce is so obviously bad?
 
On a positive note, someone did put together a HAPExplorer (albeit in WPF, why???) that at least provides some working examples, and lo-and-behold, it does compile (after I nuked the test project with an NUnit dependency), but on a bad note, the latest source download complained about a missing .cs file, so the test stuff doesn't build anyways. Sigh | :sigh:
 
Marc

GeneralRe: Another open source rant PinmemberCDP18022-May-13 8:30 
GeneralRe: Another open source rant PinprotectorAspDotNetDev2-May-13 8:33 
GeneralRe: Another open source rant PinmemberJoe Simes2-May-13 8:52 
GeneralRe: Another open source rant PinprofessionalSoMad2-May-13 9:04 
GeneralRe: Another open source rant PinmemberAlbert Holguin2-May-13 9:38 
GeneralRe: Another open source rant PinmvpEddy Vluggen2-May-13 12:27 

General General    News News    Suggestion Suggestion    Question Question    Bug Bug    Answer Answer    Joke Joke    Rant Rant    Admin Admin   

Use Ctrl+Left/Right to switch messages, Ctrl+Up/Down to switch threads, Ctrl+Shift+Left/Right to switch pages.


Advertise | Privacy | Mobile
Web01 | 2.8.150123.1 | Last Updated 25 Jan 2015
Copyright © CodeProject, 1999-2015
All Rights Reserved. Terms of Service
Layout: fixed | fluid