[Solved] Duplicate finder

[giesbert]

memcpy copies in memory and does not compare them.

AFAIK, he wanted to search for duplicates, so hashes would be faster. YOu don't want to do a full compare for all files with all files....

[ixSci]

Gerolf, thank you for the function name correction.
And yes, you are right about the question. I've misinterpreted OP goal :)

[alex.dadaev]

How do you think, is it correct?
@
while(it.hasNext())
{
it.next();
if(it.peekPrevious().key()==it.peekNext().key())
std::cout<<it.peekPrevious().value()<<"="
<<it.peekNext().value()<<std::endl;
}
@

[goetz]

No it is not correct.

It compares only adjacent entries in your container.

If your container is a map (QMap or QHash) and you use insert() to populate it then you will get no duplicates at all, since every key occurs only once, hence the keys at different positions are all distinct.

You must use insertMulti() and values() to get a list of all entries with the same hash value. Or use QMultiMap/QMultiHash with the before mentioned methods.

[alex.dadaev]

Yes, you're right, thanks

[alex.dadaev]

i can't understand why it's not working :(
@
while(it.hasNext())
{
it.next();
if(it.key()==it.peekNext().key()) {
std::cout << "i've got you" << std::endl;
}
}
@

PS: i've used insertMulti() to add item to hash, as you said

[alex.dadaev]

maybe like this?
@int compare_flag;

while(it.hasNext())
{
    it.next();
    compare_flag = QString::compare(it.key(),it.peekNext().key(),Qt::CaseSensitive);
                   if(compare_flag==0) {
        std::cout << "i've got you" << std::endl;
    }
}@

[goetz]

The keys in a (hash) map are always distinct. You will never find two identical keys so your comparison will never be true.

And even if you had identical keys in your container you would only find them if they are adjacent in the list.

But I'm going to have a kind of déjà-vu...

To make things clearer for us to understand: You do have a multi hash/multi map. What do you put in there and what do you expect to come out?

[alex.dadaev]

@QHash<QString,int> FilesHash;@
QString key is MD5
int value - just a number of file

on output i want to see the names of similar files

[goetz]

Ok, let's make things clearer step by step. Seems that you should make yourself comfortable with the concepts of a map.

A map (QHash is one) stores values associated with keys. Every key only exists once in the map - I wrote that several times, let's prove it:

@
QHash<QString, int> myHash;
myHash.insert("abc", 2);
myHash.insert("def", 3);
myHash.insert("abc", 5);

qDebug() << "hash keys:" << myHash.keys();

QHash<QString, int> myMultiHash;
myMultiHash.insertMulti("abc", 2);
myMultiHash.insertMulti("def", 3);
myMultiHash.insertMulti("abc", 5);

qDebug() << "multi hash keys:" << myMultiHash.keys();
@

What will the output be?

What will happen if you compare every key with every other?

[alex.dadaev]

insertMulti allows you to store items with similar keys

[alex.dadaev]

without overwriting them

[alex.dadaev]

what do you think about it?
@
bool ok;
QHashIterator<QString,int> it(FilesHash);
QHashIterator<QString,int> begin(FilesHash);
QHashIterator<QString,int> end(FilesHash);
while(it.hasNext()) {
it.next();
begin = qLowerBound(FilesHash.begin(), FilesHash.end(), it.key());
end = qUpperBound(begin, FilesHash.end(), it.key());
iter = begin;
while(iter!=end) {
if(*i=*it) {
ok = true;
} else { ok = false; }
}
}
@

[alex.dadaev]

why i cannot do like this?
@QHashIterator<QString,int> iter(FilesHash);

while(it.hasNext()) {
    it.next();
    iter = qBinaryFind(FilesHash.begin(), FilesHash.end(), it.key());
}@

[alex.dadaev]

please note that the problem has been solved :)

[giesbert]

you can do it on your own:
go to your first post and click edit :-)
and edit the title.