1. C# / Говнокод #20239

    +5

    1. 01
    2. 02
    3. 03
    4. 04
    5. 05
    6. 06
    7. 07
    8. 08
    9. 09
    10. 10
    11. 11
    12. 12
    public override TItem[] ToArray()
{
	TItem[] arr = new TItem[Count];
	Int64 i = 0;
	foreach (TItem item in this)
	{
		if (i >= Count) break;
		arr[i] = item;
		i++;
	}
	return arr;
}

    И пофиг, что foreach и так не даст превысить размер коллекции, всё равно проверим!

    Запостил: dm_fomenok, 20 Июня 2016

    Комментарии (13) RSS

    • ava someone 21.06.2016 10:54 # +2

      А вдруг другой поток добавит элементы в коллекцию, пока мы её обходим?
      Ответить

      • ava nihau 21.06.2016 15:16 # +3

        Надо тогда Interlocked.Add(i, 1) делать
        исключение на .MoveNext()
        Ответить

      • ava guest 21.06.2016 17:32 # 0

        И необязательно другой - может автор привык внутри таких циклов удалять элементы, вот и проверяет.
        Ответить

        • ava dm_fomenok 21.06.2016 18:52 # +5

          Если в foreach изменить коллекцию, будет исключение
          Ответить

          • ava gost 22.06.2016 09:08 # +5

            Исключение же бросать надо - а вдруг оно закатится куда-нибудь и потеряется?!
            Ответить

            • ava dev-null 29.06.2016 10:04 # +6

              пх, тоже мне - проблема. Делаешь исключение квадратным и оно уже никуда не закатится.
              Ответить

              • ava bormand 29.06.2016 17:36 # +3

                Зато провалится в квадратное отверстие. Дилема.
                Ответить

                • ava inkanus-gray 29.06.2016 17:54 # +8

                  Вот для чего, оказывается, изобрели треугольник Рело: он и катится плохо, и в отверстие своей формы не проваливается.
                  Ответить

              • ava guestinho 29.06.2016 17:42 # +3

                >>. Делаешь исключение квадратным
                Такое не провалится точно:
                [NSException raise];
                Ответить

      • ava guestinho 29.06.2016 22:30 # +2

        а что, конкарент модификейшен эксепшен не будет?
        Ответить

    • ava Vero92 06.02.2018 18:31 # 0

      Маразм крепчает)))
      Ответить

    Добавить комментарий