Question: I'm working on a Kata on CodeWars in which I have to count the amount of repetitions each letter has in a string. The amount of repetitions should be stored in an int array.

The algorithm I've written seems to almost work, however I get a weird output that I can't explain. I might be missing something in the code.

```
static void Main(string[] args)
{
    string str = "abcdef";
    string input = str.ToLower();
    int count = 0;

    string[] arrayInput = Regex.Split(input, string.Empty);
    string[] alphabet = Regex.Split("abcdefghijklmnopqrstuvwxyz", string.Empty);
    int[] amounts = new int[input.Length];

    foreach (string letter in alphabet)
    {
        for (int x = 0; x < input.Length; x++)
        {
            if (arrayInput[x] == letter)
            {
               amounts[x]++;
            }
        }
    }

    foreach (int amount in amounts)
    {
        Console.Write(amount + ", ");
    }
    Console.ReadKey();
}

```

Output: 

> 
> "2, 1, 1, 1, 1, 1,"
> 
> 
> 

Expected: 

> 
> "1, 1, 1, 1, 1, 1,"
> 
> 
> 

since each letter only appears once in the string.

Answer: 
I think you have made a mistake:

```
int[] amounts = new int[input.Length];

```

Should be

```
int[] amounts = new int[26];

```

And also your loops aren't quite right.

You don't need to split the strings into string arrays. You can just use the string iterator to get each char. Also, if you were doing this on very large strings your solution would be inefficient as for every char you are iterating through the entire alphabet which isn't needed.

you can simplify what you've written considerably:

```
string input = "abcdef";
int[] counts = new int[26]; 
foreach (var ch in input)
{
    var c = char.ToLower(ch);
    if (c >= 'a' && c <= 'z')
        counts[c - 'a']++;
}

```
When you print the values after the split you will observer the extra space. It seems that is causing issue.

```js
using System;
using System.Text.RegularExpressions;     
public class Program
{
 public static void Main()
 {
    string str = "abcdef";
    string input = str.ToLower();
    int count = 0;

    char[] arrayInput = input.ToCharArray();
    char[] alphabet = "abcdefghijklmnopqrstuvwxyz".ToCharArray();
    int[] amounts = new int[input.Length];

 foreach (char letter in alphabet)
 Console.Write(letter + ", ");
 // observe the first letter here when you use regx.split
 //, a, b, c, d, e, f, g, h, i, j, k, l, m, n, o, p, q, r, s, t, u, v, w, x, y, z, 

 foreach (char inputWord in arrayInput)
 Console.Write(inputWord + ", ");
 // you get extra space in the start hence there is issue when you use regx.split
 //, a, b, c, d, e, f,

    foreach (var letter in alphabet)
    {
        for (int x = 0; x < input.Length; x++)
        {
            if (arrayInput[x] == letter)
            {
               amounts[x] = amounts[x] + 1;
            }
        }
    }

    foreach (int amount in amounts)
    {
        Console.Write(amount + ", ");
    }

 }
}
```

stick to basic: use string.ToCharArray()