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

    +11

    1. 1
    2. 2
    3. 3
    4. 4
    5. 5
    6. 6
    7. 7
    8. 8
    void Aligner::set_genomic_sequences(vector< pair <string,string> > nt_genomic_seq){
	this->nt_genomic_sequences = *(new forward_list<pair<string,string>>);
	this->int_genomic_sequences = *(new forward_list<pair<string,string>>);
	for(vector<pair<string,string>>::const_iterator iter = nt_genomic_seq.begin() ; iter != nt_genomic_seq.end() ; iter++){
		nt_genomic_sequences.emplace_front((*iter).first,(*iter).second);
		int_genomic_sequences.emplace_front((*iter).first , nt2int((*iter).second));
	}
}

    Типичный академический код из https://bitbucket.org/yuvalel/repgenhmm. Могу только предположить, что авторы скопипастили код из Java, а потом разыменовывали указатели, пока не скомпилировалось.

    Запостил: Yuuri, 11 Декабря 2015

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

    • ava Soul_re@ver 11.12.2015 17:26 # +12 

      *(new ...)
      — оператор утечки памяти.
      Ответить

    • ava bormand 11.12.2015 17:45 # +1

      demonic_sequences
      Ответить

    • ava inkanus-gray 11.12.2015 18:12 # +2

      Офтопик: https://pic.twitter.com/2zTMwQg83t

      Что это за оператор?
      Ответить

    • ava Yuuri 13.12.2015 22:15 # +5

      Коллеги тут оставили на полдня эту софтину что-то крупное посчитать. Каждые полчаса радовались: «О, съедено уже NN гигабайт!»
      Ответить

      • ava 3_14dar 13.12.2015 22:23 # +4

        Это такой прогрессбар теперь?
        Ответить

        • ava Soul_re@ver 13.12.2015 23:29 # +4

          Всё для удобства пользователей: прогрессбар в мониторинге ресурсов.
          Ответить

          • ava bormand 13.12.2015 23:32 # +2

            И даже график прогресса от времени.
            Ответить

          • ava 1024-- 13.12.2015 23:33 # +1

            Рекомендую использовать ещё и график загрузки процессора. Так можно показать продвинутым пользователям статус выполнения, а не просто текущее состояние.
            А нет, график для памяти и так показывается. Но всё равно серьёзные люди должны расширять возможности своего ПО.
            Ответить

            • ava bormand 13.12.2015 23:41 # +1

              С процессором сложно. Надо аккуратно задержки высчитывать... Зато чем ближе к концу - тем быстрее.
              Ответить

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