Discussion :

[Avatar36]

Bonjour,

J'obtiens l'erreur suivante: ASSERT failure in QList<T>::operator[]: "index out of range"
Voila le code qui vient avec:

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
    QList< QList<int > > distance;
    int i, j, cost;
 
    for (i=0; i<string1.size();i++){
        distance[i][0] = i;
    }

Pourtant je ne vois rien de choquant. Le programme plante à la ligne distance[i][0] = i pour i=0. Qu'est-ce qui se passe?

Merci de votre aide.

Cordialement

[mintho carmo]

Pourtant, l'heure est assez explicite. Qu'est ce que tu ne comprends pas dans "out of range" ?
Quelle est la taille de tes listes ? Est ce que tu accèdes à un élément existant ?

[Avatar36]

Ce que j'ai mis c'est le tout début de mon code. Il n'y a rien avant. J'ai compris ce que signifie "out of range". Ce que je ne comprends pas c'est que j'ai cette erreur alors que i vaut 0 :/
La liste est créée à la ligne QList< QList<int > > distance; donc non, je n'accède pas à un élément existant.
Les listes font une taille de 5.

EDIT: J'ai essayé de copier ce bout de code dans un projet vierge => même bug

[mintho carmo]

La liste est créée à la ligne QList< QList<int > > distance; donc non, je n'accède pas à un élément existant.
Les listes font une taille de 5.

A quel moment tu donnes une taille de 5 a ta liste dans ton code ? (HS : tu ne viendrais pas d'un autre langage, dans lequel les listes se dimensionnent automatiquement ? Ce n'est pas le cas en C++)

Si tu accèdes à un élément qui n'existe pas, c'est normal d'avoir une erreur

[Avatar36]

C'est ce que je m'étais dit aussi mais dans la doc de la classe QList, ils n'indiquent à aucun moment la taille de ce QList. (HS: et faisant beaucoup de VBA en ce moment j'avoue que ça ne m'a pas choqué. Damn reflexes) Du coup je suis censé indiquer la taille des QList à quel endroit?

[mintho carmo]

Tu peux connaître la taille d'un conteneur en utilisant size(). Et une bonne pratique est de toujours tester la taille d'un conteneur avant d'utiliser [] (ce n'est pas spécifique à Qt, c'est valable aussi pour tous les conteneurs du C++)

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
// version Qt
Q_ASSERT(list.size() >= i) // ou assert
list[i]

Pour la taille de la liste :

• sois tu définies une taille par défaut dans le constructeur
• sois tu redimenssionnes avec resize()
• sois tu utilies une fonction qui adapte la taille (push_back, append, insert, etc)

[Avatar36]

ok merci, je vais tester ça

Le truc c'est que dans la doc, à aucun moment ils ne spécifient de taille pour une QList, même si ça me semble évident. Enfin bref, j'ai résolu mon problème en faisant un tableau à deux dimensions de int du c++ sans utiliser Qt. Et ça marche.

Cordialement

[mintho carmo]

Le truc c'est que dans la doc, à aucun moment ils ne spécifient de taille pour une QList, même si ça me semble évident. Enfin bref, j'ai résolu mon problème en faisant un tableau à deux dimensions de int du c++ sans utiliser Qt. Et ça marche.

Si. La doc dit :

Constructs an empty list.

pour le constructeur par défaut. Vide = 0 élément. Donc si tu essaies d'accéder au premier élément d'une liste qui en contient aucun, tu as un bug. Rien de surprenant à cela

[Avatar36]

ah oui effectivement. Moi j'avais lu que la partie:

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
 if (list[0] == "Bob")
     list[0] = "Robert";

alors qu'avant il y a:

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
QList<QString> list;
 list << "one" << "two" << "three";

Dans tous les cas, ça ne rend pas le remplissage par les indices très intuitif. A mon avis pour mon cas, mieux vaut utiliser le tableau à deux dimensions tel que je l'ai fait. Surtout quand les données sont des int.

[mintho carmo]

C'est le comportement classique de [], c'est pareil pour les autres conteneurs de Qt ou de la STL.

C'est parce que tu n'es pas encore habitué au C++. Dans les autres langages, si tu tentes d'accéder à un élément qui n'existe pas, on va le créer pour toi. En C++, on va considérer que c'est une erreur