std :: map’s find не может найти, но элемент в карте при ручном сканировании

Я использую std :: map и список, чтобы отслеживать оконные элементы и связанные с ними оценки. Когда окно заполнено, я хочу удалить элемент из очереди Windows и удалить его с карты. Поскольку могут быть дубликаты, карта отслеживает, сколько раз встречался каждый элемент в окне. Я также использую упорядоченную карту, чтобы я мог получать минимальные значения в данном окне.

Моя проблема в том, что find () возвращает end (), когда это не ожидается.
И когда я перебираю карту, я нахожу этот элемент присутствующим. Я не хочу жертвовать логарифмической сложностью использования карты.

tl; dr: std :: map говорит, что элемент отсутствует на карте. Сканирование вручную говорит, что это так.

[Редактировать: предложение Брайана Чена исправило карту. Спасибо!]
#include <cstdint>
#include <cstdio>
#include <cinttypes>
#include <map>
#include <list>
#include <vector>

#include "util.h"#include "kmerutil.h"
namespace kpg {

struct elscore_t {
uint64_t el_, score_;
INLINE elscore_t(uint64_t el, uint64_t score): el_(el), score_(score) {
LOG_ASSERT(el == el_);
LOG_ASSERT(score == score_);
}
INLINE elscore_t(): el_(0), score_(0) {}
inline bool operator <(const elscore_t &other) const {
return score_ < other.score_ || el_ < other.el_; // Lexicographic is tie-breaker.
}
inline bool operator ==(const elscore_t &other) const {
return score_ == other.score_ && el_ == other.el_; // Lexicographic is tie-breaker.
}
std::string to_string() const {
return std::to_string(el_) + "," + std::to_string(score_);
}
};

struct esq_t: public std::list<elscore_t> {
};

typedef std::map<elscore_t, unsigned> esmap_t;

class qmap_t {
// I could make this more efficient by using pointers instead of
// elscore_t structs.
// *maybe* TODO
// Could also easily templatify this module for other windowing tasks.
esq_t list_;
#if !NDEBUG
public:
esmap_t map_;
private:
#else
esmap_t map_;
#endif
const size_t wsz_;  // window size to keep
public:
void add(const elscore_t &el) {
auto it(map_.upper_bound(el));
if(it->first == el) ++it->second;
else map_.emplace(el, 1);
}
void del(const elscore_t &el) {
auto f(map_.find(el));
if(f == map_.end()) {
LOG_DEBUG("map failed :(\n");
for(f = map_.begin(); f != map_.end(); ++f)
if(f->first == el)
break;
}
LOG_ASSERT(f != map_.end());
if(--f->second <= 0)
map_.erase(f);
}
uint64_t next_value(const uint64_t el, const uint64_t score) {
list_.emplace_back(el, score);
LOG_ASSERT(list_.back().el_ == el);
LOG_ASSERT(list_.back().score_ == score);
add(list_.back());
if(list_.size() > wsz_) {
//fprintf(stderr, "list size: %zu. wsz: %zu\n", list_.size(), wsz_);
//map_.del(list_.front());
del(list_.front());
list_.pop_front();
}
LOG_ASSERT(list_.size() <= wsz_);
return list_.size() == wsz_ ? map_.begin()->first.el_: BF;
// Signal a window that is not filled by 0xFFFFFFFFFFFFFFFF
}
qmap_t(size_t wsz): wsz_(wsz) {
}
void reset() {
list_.clear();
map_.clear();
}
};

}

0

Решение

Как Т.С. указал в своем ответе ваш operator< не является правильным.

Ты можешь использовать станд :: галстука сделать лексикографическое сравнение

return std::tie(score_, el_) < std::tie(other.score_, other.el_);

В противном случае вы можете сделать

if (score_ == other.score_) {
return el_ < other.el_; // use el_ to compare only if score_ are same
}
return score_ < other.score_;
0

Другие решения

Это недопустимый строгий слабый порядок:

return score_ < other.score_ || el_ < other.el_;

У тебя есть elscore_t(0, 1) < elscore_t(1, 0) а также elscore_t(1, 0) < elscore_t(0, 1),

1

По вопросам рекламы [email protected]