Comments on: Guarding Against Multiple Empty Strings

By: Piyush

Piyush — Thu, 06 Jan 2011 10:37:33 +0000

You can also use yourstring.Any() to check whether it contains any elements.

By: Ben Griswold

Ben Griswold — Thu, 18 Mar 2010 22:55:42 +0000

@DeveloperIT – Thanks for the comment. There are multiple issues with my experimental code. Happy for your feedback.

By: DeveloperIT

DeveloperIT — Thu, 18 Mar 2010 22:32:50 +0000

@discordinoffice – You’re right…. and let’s say you have string A containing 1 000 000 characters, string B 1 000 000 more and string c is empty (not null). Then you have a mutiplication of 1 000 000 by a 1 00 000 then by 0. I don’t know if .net does “looks” at all params befrore doing the actual operation, but if not, you have a lack of performance, especially if you’re running that block into a loop. I, as a best pratice, would always consider usign String.IsNullOrEmpty and avoid using Length if not absolutly necessary. What do you think ?

By: Ben Griswold

Ben Griswold — Wed, 10 Jun 2009 03:04:51 +0000

@discordinoffice – Hey, you’re right. Assuming that null values were possible, the latter approach would bomb. Probably not the best approach after all. Thanks.

By: discordinoffice

discordinoffice — Wed, 10 Jun 2009 01:50:24 +0000

In the latter example, it seems worth noting that the “settled on” approach will throw a NullReferenceException if any of the variables being checked are set to NULL, instead of “” (or empty string). For this reason alone the first, more verbose, approach should ultimately be utilized.