Application is fully functional ONLY on my laptop

[juracist]

Application is fully functional only on my laptop, and on other computers it fails with part of its functionality.
It starts on every computer (windows 64- and 32- bit), and some parts of it can be executed, but problem is the main part.
I suspect problem is here, within sorting elements of QList.

for(events_it = eventi.begin(); events_it!=eventi.end(); events_it++) {
            temp_event = *events_it;
            //if type==0, we access size of the event above, and set it to this event
            if(temp_event->getType() == 0) {
                Event *temp_event2;
                events_it2 = events_it;
                events_it2--;
                temp_event2 = *events_it2;
                long s = temp_event2->getSize();
                temp_event->setSize(s);
            }
        }

for(events_it = eventi.begin(); events_it!=eventi.end(); events_it++) {
            temp_event = *events_it;
            //if type==0, we access size of the event above, and set it to this event
            if(temp_event->getType() == 0) {
                Event *temp_event2;
                events_it2 = events_it;
                events_it2--;
                temp_event2 = *events_it2;
                long s = temp_event2->getSize();
                temp_event->setSize(s);
            }
        }

To copy to clipboard, switch view to plain text mode 

These are iterators i use:

QList<Event*>::iterator events_it;
QList<Event*>::iterator events_it2;

QList<Event*>::iterator events_it;
QList<Event*>::iterator events_it2;

To copy to clipboard, switch view to plain text mode 

This is list i use:

QList<Event*> eventi;

QList<Event*> eventi;

To copy to clipboard, switch view to plain text mode 

Object Event, which is inside QList:

class Event
{
public:
    // Constructors
    Event(long a, long s, long t,short tip);
    Event(long a, long t, short tip);
 
private:
    long address;
    long size;
    long time;
    short type;
};

class Event
{
public:
    // Constructors
    Event(long a, long s, long t,short tip);
    Event(long a, long t, short tip);

private:
    long address;
    long size;
    long time;
    short type;
};

To copy to clipboard, switch view to plain text mode 

Application is written under Windows 7 Professional (64-bit).

Build settings:
buildsettings.jpg

[wysota]

What kind of incorrect behaviour do you observe?

[juracist]

If you want to summarize the work of my application..
1) it reads xml, parse it and save into list of Events, then sort this list, calculate points from sorted events, save it to binary file and finally plots the graph from calculated points

OR

2) if there is already binary file created during case 1), points are read from binary file and drawn as graph.


So, on my laptop, app works in both cases of use, but on other computers it can only read correct binary (case 2), but cannot perform case1.
Some steps of case 1 are incorrect, in my opinion, it fails during sorting..

[wysota]

How did you implement sorting?

[juracist]

1st code snippet from 1st post - here's problem, its ok on my laptop and on other computers it fails..

[wysota]

I don't see sorting in there.

[juracist]

sorry, firstly i use QStableSort to sort the list, and then this code snippet to access size property of Event (list element).
Iteration through list is implemented with iterators and for loop, firstly it finds element with property type == 0, then with other iterator it goes up for one position and gets size property from above element and finally sets that size to first iterator (with type==0).

I didn't mention usage of QStableSort - it goes well, i checked. But problems appear in code i pasted here.

[wysota]

sorry, firstly i use QStableSort to sort the list,

What comparison function are you using? Please show us the exact call to qStableSort along with the lessThan implementation you use with it.

[juracist]

qStableSort(eventi.begin(),eventi.end(),sortAddress);

qStableSort(eventi.begin(),eventi.end(),sortAddress);

To copy to clipboard, switch view to plain text mode 

bool sortAddress(Event *left, Event *right){
    return (left->getAddress() < right->getAddress());
}

bool sortAddress(Event *left, Event *right){
    return (left->getAddress() < right->getAddress());
}

To copy to clipboard, switch view to plain text mode 

[wysota]

And what your first code snippet is meant to do? Because it seems it searches for event of type 0 and then assigns it the value of "size" of a previous item. Which by the way will fail immediately if the first element in the list has type = 0.

[juracist]

If you have patience and time, download exe file of my app, and try to run it (only for windows )
Application

Then, open xml file from same folder, and, if everything is ok, graph should be drawn..
.bin file will be created in same folder, but not good file, only with couple of nulls..
I also attached bin file that is created under my enviroment and if you try to open it, it should draw graph..

So, you'll see there is a problem only with reading xml file, not bin file.
Steps while drawing graph from xml is following:
1. parsexml - OK
2. sortAddress - OK
3. getSize(code snippet from above) - FAIL (checked by printing elements from list into text file)
4. sortTime
5. correctOrder (something like getSize)
6. calculatePoints (create points from CORRECT list of elements)
7. drawGraph