Кресты / Говнокод #18014 Ссылка на оригинал

0

  1. 1
  2. 2
  3. 3
  4. 4
  5. 5
  6. 6
  7. 7
  8. 8
  9. 9
  10. 10
  11. 11
  12. 12
  13. 13
  14. 14
  15. 15
  16. 16
void insertObject(Object object)
{
+	auto found = std::find_if(
+		objects.begin(),
+		objects.end(),
+		[&object](Object otherObject)
+		{
+			return otherObject == object;
+		}
+	);
+
+	if (found != objects.end())
+		return;
+
	objects.push_back(object);
}

Надо было не вставлять в коллекцию дубликаты по какому-то критерию. Человек написал это.

Запостил: atsgovno atsgovno, (Updated )

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

  • ava guest # Ссылка на оригинал 0

    а это считается онанизмом, когда сам себе плюсуешь?
    Ответить

      • ava 1024-- # Ссылка на оригинал 0 ⇈

        Кстати, когда творения Семёна смыли? Он всё же превысил лимит, что Страйкеру пришло уведомление от хостера?
        Ответить

        • ava Stertor # Ссылка на оригинал 0 ⇈

          Возможно, у многих проскальзывала мысль: почто стрикер не банит Стертора? Ведь в сущности я такое же унылое г.. но и спамер, как и Сэм. Хотите узнать, почему?

          Я-Страйкер
          Ответить

            • ava Stertor # Ссылка на оригинал 0 ⇈

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

        • ava 3.14159265 # Ссылка на оригинал 0 ⇈

          Какие именно?
          Его тут постоянно спускают в толчок и выбрасывают в утиль.
          Ответить

          • ava 1024-- # Ссылка на оригинал 0 ⇈

            А вот недавние стоковые про быдло. С патриотических укроучёток.
            Ответить

    • ava Xom94ok # Ссылка на оригинал 0 ⇈

      Много лишних букв.
      void insertObject(Object object)
{
-	auto found = std::find_if(
-		objects.begin(),
-		objects.end(),
-		[&object](Object otherObject)
-		{
-			return otherObject == object;
-		}
-	);
-	if (found != objects.end())
-		return;
-
-	objects.push_back(object);
-
+	if (std::find(objects.cbegin(), objects.cend(), object) == objects.cend())
+		objects.push_back(object);
}
      Ответить

      • ava defecate-plusplus # Ссылка на оригинал 0 ⇈

        возникают вопросы, так ли тут нужен вектор (?), когда все уже придумано за нас - сет/анордеред-сет
        Ответить

        • ava bormand # Ссылка на оригинал 0 ⇈

          Видимо, порядок хотят сохранить. Ну или просто не знали про set.
          Ответить

          • ava Dummy00001 # Ссылка на оригинал 0 ⇈

            > Видимо, порядок хотят сохранить. Ну или просто не знали про set.

            или просто объектов мало, порядок не нужен, и с set'ом не хотели парится.

            или потому что экономят память, и vector самый дешевый контейнер.
            Ответить

        • ava Xom94ok # Ссылка на оригинал 0 ⇈

          Наверное, у обжекта иерархия подклассов, для которой написать operator<() или хеш-функцию не представляется возможным.
          Ответить

        • ava TarasB # Ссылка на оригинал 0 ⇈

          А почему ты думаешь, что это вектор?
          А вдруг автор совсем наркоман?
          Ответить

        • ava laMer007 # Ссылка на оригинал 0 ⇈

          К слову на малом кол-ве элементов сет намного медленнее вектора. Не зря в жаве есть контейнеры хамелионы, меняющие свою структуру динамически в зависимости от кол-ва элементов
          Ответить

          • ava 3.14159265 # Ссылка на оригинал 0 ⇈

            >К слову на малом кол-ве элементов сет намного медленнее вектора.
            Тихо-тихо, сейчас прибегут психи с рассказами про O(N) даже на ДЖВУХ элементах.
            http://govnokod.ru/17919#comment270854
            Ответить

            • ava kegdan # Ссылка на оригинал 0 ⇈

              новый фетиш? скоро в моде - массив листов по два элемента
              Ответить

            • ava 3_14dar # Ссылка на оригинал 0 ⇈

              Бгг, ты мои коменты в избранное сохраняешь?

              На питоне как-то делал тесты, in list и in set в цикле, на одном элементе скорость одинакова, на двух set уже в 2 раза быстрее.
              Ответить

          • ava guest # Ссылка на оригинал 0 ⇈

            Тому до шукати перебором проще чем по хешу в маолых дозах
            Ответить

      • ava warzes # Ссылка на оригинал 0 ⇈

        условие задачи читай.
        >>дубликаты по какому-то критерию.
        Когда начнешь вводить второе условие - получишь больше буков
        Ответить

      • ava guest # Ссылка на оригинал 0 ⇈

        Авто? Разве она не авто по умолчанию? Статикбля? Ты пиздюк может?
        Ответить

  • ava LispGovno # Ссылка на оригинал 0

    > atsgovno
    Ты по что мой ник занял? Теперь у меня нет повода учить этот язык
    Ответить

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

Переведи на "PHP", guest!

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

    ↑
    8
    ↓