Си диез / Говнокод #5056 Ссылка на оригинал

0

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

Запостил: Говногость Говногость, (Updated )

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

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

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

      Так, что говно на лицо.
      Ответить
      • Тогда зачем форич, для индекса как раз честный цикл лучше был бы.
        Ответить
      • А вообще, если надо искать строку в большом массиве строк, значит нехрен было заводить массив, надо было сразу карту заводить.
        Ответить
  • Фейк! Не скомпилится. По причине, озвученной TarasB: не хватает ретурна в конце.
    Ответить
    • Говногость, вроде, говорил, что по памяти все говнокоды пишет. Может на работе Интернета нет? О_о Или так занят, что не успевает зайти в Интернет на работе? о_О
      Ответить
  • ii++;//Тут нужно оптимизировать!!!
    Ага, хороший коммент. Надо написать ++ii; 😀
    Ответить

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

Я, guest, находясь в здравом уме и твердой памяти, торжественно заявляю:

    А не использовать ли нам bbcode?


    8