1. C++ / Говнокод #18014

    +77

    1. 01
    2. 02
    3. 03
    4. 04
    5. 05
    6. 06
    7. 07
    8. 08
    9. 09
    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, 17 Апреля 2015

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

    • ava 3.14159265 17.04.2015 18:14 # +1

      Новый стандарт мозгов не прибавляет.
      Ответить

    • ava guest 17.04.2015 18:26 # 0

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

      • ava guest 17.04.2015 18:33 # +1

        Главное, вовремя всё смыть.
        Ответить

        • ava 1024-- 17.04.2015 18:35 # 0

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

          • ava Stertor 17.04.2015 20:05 # +1

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

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

            • ava Vasiliy 17.04.2015 20:10 # +1

              Я то думал ты Бэтман.
              Ответить

              • ava Stertor 17.04.2015 20:11 # 0

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

          • ava 3.14159265 17.04.2015 20:39 # 0

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

            • ava 1024-- 17.04.2015 20:45 # 0

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

    • ava guest 17.04.2015 18:55 # 0

      А в чём проблема?
      Ответить

      • ava Xom94ok 17.04.2015 20:34 # +2

        Много лишних букв.
        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 17.04.2015 22:46 # 0

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

          • ava bormand 17.04.2015 22:51 # +1

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

            • ava roman-kashitsyn 18.04.2015 01:05 # +3

              > порядок хотят сохранить
              vector + set
              Ответить

            • ava Dummy00001 19.04.2015 11:18 # +3

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

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

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

          • ava Xom94ok 19.04.2015 09:23 # +2

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

          • ava TarasB 19.04.2015 17:11 # 0

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

          • ava laMer007 20.04.2015 13:03 # +2

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

            • ava 3.14159265 20.04.2015 13:08 # +4

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

              • ava kegdan 20.04.2015 20:01 # 0

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

              • ava 3_14dar 25.04.2015 07:14 # 0

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

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

            • ava guest 16.06.2015 05:15 # 0

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

        • ava warzes 15.06.2015 01:12 # 0

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

        • ava guest 16.06.2015 05:16 # 0

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

    • ava LispGovno 14.05.2015 07:47 # 0

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

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