The Weird and The Wonderful

public void Foo()
        {
            var SomeList = new ArrayList();
            SomeList = getSomeList();

            if (SomeList.Count != 0)
            {
                foreach (string Item in SomeList)
                {
                    var S = Item.Split(',');
                    var S0 = s[0];
                    var S1 = s[1];
                    var S2 = s[2];
                    var S3 = s[3];
                ...
                }
           }
     }

     private static ArrayList getSomeList()
     {
        var List = new ArrayList();
        ...
        List.Add(N1 + "," + N2 + "," + N3 + "," + N3);
        ...
        return List;
     }

1. Useless initialization of SomeList
2. Well, if the count is 0, the foreach won't execute
3. Obviously never heard of out variables, or even returning a struct/class with the parsed data. Instead, he creates a concatenated string, then parses it back out! And not exactly internationalized, the chaos that would occur if one or more of the numbers was 1234,56
4. Why a static method? It's referenced once, in the above call.