chalarangelo / 30-seconds-of-csharp

Short C# code snippets for all your development needs

Hi there 👋

I'm Angelos Chalaris, a web developer from Athens, Greece. I love programming both as a job and as a hobby and I spend a lot of time writing code and testing out new ideas. I work mainly with web technologies such as JavaScript, HTML, CSS, Node.js, React and SCSS, and I occasionally dabble in Python, Astro and Svelte.

My work 🔭

• I work as a frontend engineer for Skroutz - website
• I am the creator of 30 seconds of code - website
• I code a lot, mainly in JavaScript - view repositories

Contact 📫

You can reach me via email at [email protected]

30-seconds-of-csharp's Issues

Palindrome

Hi, i would like to add my PR
Returns true if the given string is a palindrome, false otherwise.

Generalize array/list snippets

Currently certain snippets can only apply to arrays or lists. However, Stringify showcases a generalized example for IEnumerable. We should investigate the following snippets for a generalized version:

• Chunk
• All
• None
• DistinctValues

Related milestone: 2020 Sprint 1

PadNumber and string interpolation

Description

The PadNumber snippet uses string interpolation to convert an integer to a string with leading zeroes to a specified length. I suggest using concatenation instead of interpolation.

Here is the current snippet:

public static string PadNumber(int n, int length)
{
    return n.ToString($"D{length}");
}

String interpolation adds overhead including extra memory use and extra CPU time. In this case, the interpolation doesn't help anything, so it's overhead with no benefit.

The method below uses a little less time and memory, and is slightly simpler because the user does not have to understand C# string interpolation. Obviously, the user will encounter string interpolation elsewhere but that should be when faced with a situation where interpolation is more useful.

public static string PadNumber(int n, int length)
{
    return n.ToString("D" + length.ToString());
}

There are ways to make this more performant, but that's not usually necessary when using a function like this.

Merge Type tag into Utility

Currently, both tags are quite empty. It is preferable to move all the Type snippets into Utility, at least for now.

Fix typos in snippet descriptions/explanations/examples

Most snippets currently use True and False in descriptions, explanations and examples instead of true and false (uppercase vs lowercase). This might confuse users. We should update all existing snippets to replace these typos with the correct form.