Common mistakes of developer

String concatenation instead of StringBuilder

String concatenation works in such a way that every time when you add something to a string, a new address in the memory is being allocated. The previous string is copied to a new location with the newly added part. This is inefficient. On the other hand we have StringBuilder which keeps the same position in the memory without performing the copy operation.

//INCORRECT

List<string> values = new List<string> { "This ", "is ", "Abrar ", "!" };

string outputValue = string.Empty;

foreach (var value in values)

{

outputValue += value;

}

//CORRECT

StringBuilder outputValueBuilder = new StringBuilder();

foreach (var value in values)

{

outputValueBuilder.Append(value);

}

LINQ – ‘Where’ with ‘First’ instead of FirstOrDefault

A lot of programmers find a certain set of elements by means of ‘Where’ and then return the first occurrence. This is inappropriate, because the ‘First’ method can be also applied with the ‘Where’ condition. What’s more, it shouldn’t be taken for granted that the value will always be found. If “First” is used when no value is found, an exception will be thrown. Thus, it’s better to use FirstOrDefault instead. When using FirstOrDefault, if no value has been found, the default value for this type will be returned and no exception will be thrown.

//INCORRECT

List<int> numbers = new List<int> { 1, 4, 5, 6, 2, 3, 4, 5, 6, 3, 6, 23, 23 };


return numbers.Where(x => x == 6).First();

//PARTLY CORRECT

return numbers.First(x =>x==6);

//CORRECT

return numbers.FirstOrDefault(x => x == 6);

Linq Do not use OrderBy before Where

//INCORRECT

var allIndiaEmployee =EmployeeList.OrderBy(x => x.EmployeeName)

.Where(x => x.Country == Country.India);

//CORRECT

var allIndiaEmployee = EmployeeList.Where(x => x.Country == Country.India)

.OrderBy(x => x.EmployeeName);

Linq Difference between First() and Single()

Always remember that First() returns the very first item of the sequence, if no item exists it throws InvalidOperationException, whereas Single() Returns the only item in the sequence, if no item exists it throws InvalidOperationException,and if more than one item exists, it also throws InvalidOperationException.

Casting by means of ‘(T)’ instead of ‘as (T)’ when possibly not castable

//INCORRECT

var chair = (Chair)objFurniture;

//CORRECT

var chair = objFurniture as Chair;

Incorrect exceptions re-throwing

C# programmers usually forget that when they throw an exception using „throw ex” they loose the stack trace. It is then considerably harder to debug an application and to achieve appropriate log messages. When simply using „throw” no data is lost and the whole exception together with the stack trace can be easily retrieved.

//INCORRECT

try

{

//some code that can throw exception [...]

}

catch (Exception ex)

{

//some exception logic [...]

throw ex;

}

//CORRECT

try

{

//some code that can throw exception [...]

}

catch (Exception ex)

{

//some exception logic [...]

throw;

}

Take advantage of string.IsNullOrEmpty() and string.IsNullOrWhiteSpace()

// INCORRECT:

if (name != null && name != string.Empty)

{

[...]

}

//CORRECT

if (string.IsNullOrEmpty(name))

{

[...]

}

Sometimes we want to make sure that a string is not only not empty and not null, but also does not comprise of whitespaces. In such situation we could of course use the following code:

// INCORRECT:

if (string.IsNullOrEmpty(name.Trim()))

{

[...]

}

//CORRECT

if (string.IsNullOrWhiteSpace(name))

{

[...]

}