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

    +121

    1. 01
    2. 02
    3. 03
    4. 04
    5. 05
    6. 06
    7. 07
    8. 08
    9. 09
    10. 10
    int finda(string[] strl, string a)
{
	int ii=0;
	foreach(var i in strl)
	{
		if(a==i)
			return ii;
		ii++;//Тут нужно оптимизировать!!!
	};
}

    Запостил: Говногость, 25 Декабря 2010

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

    • ava TarasB 25.12.2010 18:57 # +2

      Вернуть количество строк в массиве, идущих до данной - это требовалось?
      Если так, то особого говна тут не вижу.
      Ну можно вместо ретурна поставить брек, а ретурн в конце. Да, тут же не разобран случай, когда данной строки в массиве нет - тогда функция возвращает неопределённость.
      Ответить

      • ava guest 25.12.2010 19:07 # +2

        Вообще тут получают индекс элемента a в массиве, наверное.

        Делаеться так:
        int index=strl.IndexOf(a);

        Так, что говно на лицо.
        Ответить

        • ava TarasB 25.12.2010 19:17 # +2

          Тогда зачем форич, для индекса как раз честный цикл лучше был бы.
          Ответить

        • ava TarasB 25.12.2010 21:08 # +1

          А вообще, если надо искать строку в большом массиве строк, значит нехрен было заводить массив, надо было сразу карту заводить.
          Ответить

    • ava koodeer 25.12.2010 19:36 # 0

      Фейк! Не скомпилится. По причине, озвученной TarasB: не хватает ретурна в конце.
      Ответить

      • ava guest 25.12.2010 23:43 # 0

        Говногость, вроде, говорил, что по памяти все говнокоды пишет. Может на работе Интернета нет? О_о Или так занят, что не успевает зайти в Интернет на работе? о_О
        Ответить

    • ava abatishchev 27.12.2010 11:05 # 0

      тут даже именование переменных - гавно
      Ответить

    • ava guest 10.05.2011 15:56 # +1 

      ii++;//Тут нужно оптимизировать!!!
      Ага, хороший коммент. Надо написать ++ii; :D
      Ответить

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