1
al.Add(RandomClass.CreateRandomString());
foreach (var s in al)
    Console.Write("{0}\t", s);

Если этот кусок находится в Main, то все нормально. При перемещении в метод другого класса и последующим вызове метода через экземпляр класса, элементы коллекции неоднократно дублируются при выводе на консоль. Но при этом если запросить кол-во элементов коллекции, выводится правильное число(без учета дублированных членов).

Это работает корректно

Stopwatch stopWatch = new Stopwatch();
ArrayList al = new ArrayList();
Console.Write("Введит количество элементов коллекции: ");
int collectionLength = Int32.Parse(Console.ReadLine());
stopWatch.Start();
for (int i = 0; i < collectionLength; i++)
    al.Add(RandomClass.CreateRandomString());
foreach (var s in al)
    Console.Write("{0}\t", s);
stopWatch.Stop();
Console.WriteLine("Время заполнения коллекции: {0}", stopWatch.ElapsedMilliseconds + " ms");
Console.WriteLine("Количество элементов в коллекции: {0}", collectionLength);

А это нет

Stopwatch stopWatch = new Stopwatch();
ArrayListWrapper al = new ArrayListWrapper();
Console.Write("Введит количество элементов коллекции: ");
int collectionLength = Int32.Parse(Console.ReadLine());
stopWatch.Start();
for (int i = 0; i < collectionLength; i++)
    al.AddElements();
stopWatch.Stop();
Console.WriteLine("Время заполнения коллекции: {0}",      stopWatch.ElapsedMilliseconds + " ms");
Console.WriteLine("Количество элементов в коллекции: {0}", collectionLength);

Код в ArrayListWrapper

class ArrayListWrapper : ICollectionWrapper
{
    ArrayList internalArrayList = new ArrayList();
    public void AddElements()
    {
        internalArrayList.Add(RandomClass.CreateRandomString());
        foreach (var s in internalArrayList)
            Console.Write("{0}\t", s);
    }
4
  • Из main удалили тестовый код? В целом лучше весь код прикладывать 8 апр 2016 в 2:30
  • скорее всего случайно два раза вызываете вывод коллекции. пробовали отладить?
    – user177221
    8 апр 2016 в 5:55
  • Добавил код. Дублирующий код не удалял, а закомментировал
    – cruim
    8 апр 2016 в 6:51
  • код размещен в конструкторе? есть ли потоки?
    – KoVadim
    8 апр 2016 в 6:54

1 ответ 1

2

Ваш код неэквивалентен:

Первый пример collectionLength раз добавляет элемент. Потом выводит все элементы

(n раз добавить)

вывести все

Второй - n раз добавляет элемент, и после каждого добавления выводит все элементы

(n раз)
{
    добавить 1 элемент
    вывести все элементы
}

Т.е. он

  • Добавляет один элемент. Выводит его.
  • Добавляет еще один элемент. Выводит оба элемента.
  • Добавляет еще один элемент. Выводит все три элемента.

Рефакторинг не должен изменять функционал, так что у вас - не рефакторинг :)

Или вынесите foreach с выводом элементов из AddElement, или выводите в AddElement толко последний добавленный.

Кстати, ArrayList - это древнее наследие .NET 1.0, оставленное только ради совместимости. Используйте List<T>.

1
  • спасибо. на счет ArrayList, цель протестировать производительность коллекций.
    – cruim
    8 апр 2016 в 7:26

Ваш ответ

By clicking “Отправить ответ”, you agree to our terms of service and acknowledge that you have read and understand our privacy policy and code of conduct.

Всё ещё ищете ответ? Посмотрите другие вопросы с метками или задайте свой вопрос.