Updated HttpUtility ParseQueryString code snippets for C# and VB

[JoanGil]

Summary

Fixes #9251

This pull request enhances the C# and VB language examples on the HttpUtility.ParseQueryString page. Following a discussion with @MichaelDeutschCoding, I've introduced two updated versions of the code to replace the outdated examples that did not adhere to clean code principles.

Thanks @gewarren and @carlossanlop for guiding me!

[ghost]

Tagging subscribers to this area: @dotnet/ncl
See info in area-owners.md if you want to be subscribed.

Issue Details

---

Summary

Fixes #9251

This pull request enhances the C# and VB language examples on the HttpUtility.ParseQueryString page. Following a discussion with @MichaelDeutschCoding, I've introduced two updated versions of the code to replace the outdated examples that did not adhere to clean code principles.

Thanks @gewarren and @carlossanlop for guiding me!

Author:	JoanGil
Assignees:	-
Labels:	area-System.Net.Http, community-contribution
Milestone:	-

[JoanGil]

@dotnet-policy-service agree

[JoanGil]

Hi @Rick-Anderson, @carlossanlop! Any news on this one?

[JoanGil]

Hi guys @Rick-Anderson, @carlossanlop. It's been a month with no response, could you take a look at this one? Some feedback would be really appreciated! 😁

[Rick-Anderson]

File

Status

Preview URL

ms.author

ms.prod

ms.technology

ms.service

ms.subservice

Details

snippets/csharp/VS_Snippets_WebNet/HttpUtility_ParseQueryString/cs/httputility_parsequerystring.aspx Succeeded n/a (file deleted or renamed)
snippets/csharp/VS_Snippets_WebNet/HttpUtility_ParseQueryString/cs/httputility_parsequerystring.cs Succeeded View
snippets/visualbasic/VS_Snippets_WebNet/HttpUtility_ParseQueryString/vb/httputility_parsequerystring.aspx Succeeded n/a (file deleted or renamed)
snippets/visualbasic/VS_Snippets_WebNet/HttpUtility_ParseQueryString/vb/httputility_parsequerystring.vb Succeeded View
xml/System.Web/HttpUtility.xml Succeeded View

[JoanGil]

File

Status

Preview URL

ms.author

ms.prod

ms.technology

ms.service

ms.subservice

Details

snippets/csharp/VS_Snippets_WebNet/HttpUtility_ParseQueryString/cs/httputility_parsequerystring.aspx Succeeded n/a (file deleted or renamed) snippets/csharp/VS_Snippets_WebNet/HttpUtility_ParseQueryString/cs/httputility_parsequerystring.cs Succeeded View snippets/visualbasic/VS_Snippets_WebNet/HttpUtility_ParseQueryString/vb/httputility_parsequerystring.aspx Succeeded n/a (file deleted or renamed) snippets/visualbasic/VS_Snippets_WebNet/HttpUtility_ParseQueryString/vb/httputility_parsequerystring.vb Succeeded View xml/System.Web/HttpUtility.xml Succeeded View

Is this something that I have to solve? Everything seems to be success and I don't have access to most of the files. Anything I can do?

[carlossanlop]

Sorry for the late response.

@gewarren is the main contact for this repo, she can help review. @Rick-Anderson since you own System.Web, can you also please take a look?

Is this something that I have to solve? Everything seems to be success and I don't have access to most of the files. Anything I can do?

The CI passed, those are succeeded validation results, you don't need to do anything else for now. This now needs a code review.

[gewarren]

@JoanGil The Try.NET snippet fails with these errors when I preview it in the browser (you won't have permissions to do that unfortunately):

program.cs(4,1): error CS0825: The contextual keyword 'var' may only appear within a local variable declaration or in script code
program.cs(4,5): error CS0116: A namespace cannot directly contain members such as fields or methods
program.cs(5,1): error CS0825: The contextual keyword 'var' may only appear within a local variable declaration or in script code
program.cs(5,5): error CS0116: A namespace cannot directly contain members such as fields or methods
program.cs(5,17): error CS0103: The name 'url' does not exist in the current context
program.cs(8,1): error CS0825: The contextual keyword 'var' may only appear within a local variable declaration or in script code
program.cs(8,53): error CS0103: The name 'parsedUrl' does not exist in the current context
program.cs(11,1): error CS1022: Type or namespace definition, or end-of-file expected
program.cs(11,17): error CS8124: Tuple must contain at least two elements.
program.cs(11,17): error CS1026: ) expected
program.cs(11,17): error CS1022: Type or namespace definition, or end-of-file expected
program.cs(11,37): error CS0116: A namespace cannot directly contain members such as fields or methods
program.cs(11,44): error CS1022: Type or namespace definition, or end-of-file expected
program.cs(13,13): error CS0116: A namespace cannot directly contain members such as fields or methods
program.cs(13,23): error CS1031: Type expected
program.cs(13,23): error CS8124: Tuple must contain at least two elements.
program.cs(13,23): error CS1026: ) expected
program.cs(13,23): error CS1022: Type or namespace definition, or end-of-file expected
program.cs(14,1): error CS1022: Type or namespace definition, or end-of-file expected

[JoanGil]

@JoanGil The Try.NET snippet fails with these errors when I preview it in the browser (you won't have permissions to do that unfortunately):

program.cs(4,1): error CS0825: The contextual keyword 'var' may only appear within a local variable declaration or in script code program.cs(4,5): error CS0116: A namespace cannot directly contain members such as fields or methods program.cs(5,1): error CS0825: The contextual keyword 'var' may only appear within a local variable declaration or in script code program.cs(5,5): error CS0116: A namespace cannot directly contain members such as fields or methods program.cs(5,17): error CS0103: The name 'url' does not exist in the current context program.cs(8,1): error CS0825: The contextual keyword 'var' may only appear within a local variable declaration or in script code program.cs(8,53): error CS0103: The name 'parsedUrl' does not exist in the current context program.cs(11,1): error CS1022: Type or namespace definition, or end-of-file expected program.cs(11,17): error CS8124: Tuple must contain at least two elements. program.cs(11,17): error CS1026: ) expected program.cs(11,17): error CS1022: Type or namespace definition, or end-of-file expected program.cs(11,37): error CS0116: A namespace cannot directly contain members such as fields or methods program.cs(11,44): error CS1022: Type or namespace definition, or end-of-file expected program.cs(13,13): error CS0116: A namespace cannot directly contain members such as fields or methods program.cs(13,23): error CS1031: Type expected program.cs(13,23): error CS8124: Tuple must contain at least two elements. program.cs(13,23): error CS1026: ) expected program.cs(13,23): error CS1022: Type or namespace definition, or end-of-file expected program.cs(14,1): error CS1022: Type or namespace definition, or end-of-file expected

Okay, I have updated the code, wrapping it inside a Program and Main. Hopefully this will solve the C# and VB problems. Could you check again @gewarren. Thanks!

[learn-build-service-prod]

Learn Build status updates of commit d4f906e:

✅ Validation status: passed

File	Status	Preview URL	Details
snippets/csharp/VS_Snippets_WebNet/HttpUtility_ParseQueryString/cs/httputility_parsequerystring.aspx	✅Succeeded	n/a (file deleted or renamed)
snippets/csharp/VS_Snippets_WebNet/HttpUtility_ParseQueryString/cs/httputility_parsequerystring.cs	✅Succeeded	View
snippets/visualbasic/VS_Snippets_WebNet/HttpUtility_ParseQueryString/vb/httputility_parsequerystring.aspx	✅Succeeded	n/a (file deleted or renamed)
snippets/visualbasic/VS_Snippets_WebNet/HttpUtility_ParseQueryString/vb/httputility_parsequerystring.vb	✅Succeeded	View
xml/System.Web/HttpUtility.xml	✅Succeeded	View

For more details, please refer to the build report.

For any questions, please:

• Try searching the learn.microsoft.com contributor guides
• Post your question in the Learn support channel

[gewarren]

It works now; thanks @JoanGil!