Trouver les valeurs min et max dans un fichier CSV

Version imprimable

Bonjour à tous

Merci de prendre quelques instants pour me lire :D
Bon tout d'abord je vous préviens je suis vraiment une brêle en C++ j'ai commencé y'a quelques jours, donc je comprends pas toujours ce que je fais :ptdr:
Je travaille sur QT Creator 5, windows 7.

Voila la situation, j'ai une interface Arduino qui envoi une trame de 3 valeurs par le port série. C'est un fait la position d'un codeur et 2 tensions.
Je suis donc arrivé à me connecter, récupérer la trame, découper la trame en valeur unitaire pour les afficher dans le programme.
J'ai créée une fonction pour sauvegarder ces valeurs dans un fichier csv. J'ai également ajouter une fenêtre avec le tracage du graphique en temps réel ( QCPCurve)

Tout ça fonctionne parfaitement, j'arrive maintenant à lire le fichier csv, et séparer les 2 colonnes pour les afficher dans 2 "textEdit" différents
Mon problème est que j'ai besoin de chercher dans chaque colonne la valeur min et max.
Avant de poster j'ai procéder à des heures de recherches, sur différents sujets et des heures d'essai de code "au feeling"...mais ça ne fonctionne pas j'aurai vraiment besoin d'un coup de main
J'ai essayé ça ?! ( encore une fois je comprends pas tjs ce que je fais alors soyez indulgent :oops: )
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 20 21 22 23 24 25 26 27 28 int main() { QString fichier2= "c:/qt/serial.csv"; QFile file(fichier2); if(!file.open(QIODevice::ReadOnly | QIODevice::Text)) file.readLine(); while (!file.atEnd()); QByteArray line = file.readLine(); QByteArrayList fields {line.split(',')}; QList<QByteArray> column2 {fields [2] }; QList<int> list; std::sort(list.begin(), list.end()); int iMax = list[0]; //set min and max as the first element int iMin = list[0]; for (int i=1; i>iMax; i++) if (list[i] < iMin) iMin = list[i]; else { if (list[i] > iMax) iMax = list[i]; } return 0; }
Merci de votre aide précieuse

Bon j'ai avancé un peu cet après midi

Donc j'ai fait des essais pour voir si j'accedais bien au fichier et à la bon colonne. J'ai donc fait des essais en virant le fichier, ou en changeant les valeurs et ça fonctionne.
Enfin l'accès est correct.

Comme je le disais j'arrive bien à afficher la colonne souhaitez dans le textEdit, par contre le retour de la fonction cout me donne n'importe quoi je pense qu'il manque une étape
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 20 21 22 23 24 25 26 27 void Dialog::on_pushButton_clicked() { QString fichier2= "c:/qt/serial.csv"; QFile file(fichier2); if(!file.open(QIODevice::ReadOnly | QIODevice::Text)) return; file.readLine(); while (!file.atEnd()) { QByteArray line = file.readLine(); QByteArrayList fields {line.split(';')}; QByteArray column2 {fields [1] }; QDataStream in(column2); auto mm = std::minmax_element(column2.begin(), column2.end()); std::cout << "min is " << *mm.first; std::cout << "max is " << *mm.second; { ui->textEdit->append(fields[2]); }
Ca fait des heures que j'y suis et je commence à baisser les armes là :weird:

13/04/2020, 01h11
dalfab

Bonjour,

Tu as appelé une variable column2. Penses-tu avoir récupéré le liste des nombres qui sont dans la colonne 2, si c'était le cas, la fonction std::minmax_element() permettrait en effet de trouver les positions des valeurs extrêmes.
Si on s'intéresse au type de cette variable : QByteArray, on voit que c'est constitué de quelques caractères (ceux que tu as vu quand tu as cherché à afficher cette colonne.) On est donc loin d'une liste de nombres, il manque donc au moins 2 choses :
- convertir le champ lu en un nombre.
- stocker chacun des nombres dans une liste ou un vector.
- une fois le fichier entièrement parcouru, tu pourras utiliser std::minmax_element().

Bonsoir

Je planche avec Ben sur ce projet avec le même niveau de base en QT/C++. Mais à 2 on avance bien :mrgreen:

Alors merci dalfab de confirmer ce que j'ai en tête pour la conversion des champs en nombres et aussi le fait que fonction nous renvoyer une reponse par ligne et non une pour la colonne global.

Donc j'ai essayé de lire ligne par ligne mon csv, chaque ligne remplissant une case d'un vector<QstringList> appelé tabfichier2 afin de plus passer par la variable QByteArray

Je me dis que si on arrive a ranger nos valeurs de cette manière , on devrai pouvoir les convertir en nombre (grace à "todouble" peut etre bien ? ) et ensuite les traiter avec std::minmax_element() (ou par la méthode avec itération )

Mais jai encore une erreur sur la dernière ligne no viable overloaded'=' ???:aie::aie:
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 14 void MainWindow::on_OUVRIR_clicked() { QFile fichier2("D:/Mes Documents/Arduino/PROJETS YANN/BANC AAC/LOGICIEL/BANC_ARBRE_A_CAMES_V1_0_2/serial2.csv"); if(fichier2.open(QIODevice::ReadOnly|QIODevice::Text)) { QString line; for(int i=0; !(line = fichier2.readLine()).isEmpty(); i++) { tabfichier2[i] = line.split(';'); } }
merci à vous

Là aussi, il y a une étape de brûlée.
On ne peut pas convertir par un simple égal 2 objets complexes. D'autant que la simple conversion d'un QByteArray en QString n'est déjà pas si immédiate et pourrait nécessiter des précisions (est-ce que fichier contenait l'Ascii, de l'utf-8, ... la QString nécessite une information pour que les bytes lu sont compris comme des caractères).
Le moyen simple pourrait être (sous réserve car je ne connais du tout Qt):
Code:

1 2 3 4 5 6 7 8 9 10 QList<double> column2; while (!file.atEnd()) { QByteArray line{ file.readLine() }; QByteArrayList fields{ line.split(';') }; column2.append( fields[1].toDouble() ); // les 2 actions que je préconisais } auto mm = std::minmax_element(column2.begin(), column2.end()); std::cout << "min of column 2 is " << *mm.first; std::cout << "max of column 2 is " << *mm.second;

1 pièce(s) jointe(s)

Tout d'abord merci de votre réponse qui semble tout à fait logique, j'ai essayé votre code et j'ai essayé pas mal de chose mais celle qui semble se rapprocher le plus de ma demande est celle ci
Code:

1 2 3 4 5 6 7 8 QByteArray line = file.readLine(); QByteArrayList fields {line.split(';')}; QByteArray column2 {fields [3]}; column2 = column2.simplified(); auto mm = std::max_element(column2.begin(), column2.end()); std::cout << "max" << mm;
donc il va bien chercher cette colonne, mais il me sort tout la réponse en ligne à ligne, j'ai pensé que c'était dû au \n en fin de ligne donc je l'ai enlevé .simplified mais non

J'arrive pas à convertir ça, ni en QString ou autre ....

Voila ce que j'ai comme retour

Pièce jointe 555686

bon jai un debut de solution je pense

là , à chaque boucle il me sort bien la bonne valeur de la bonne colonne et converti en type double :D

il faut desormais que je remplisse un tableau ou une liste avec la valeur finale de la boucle . ensuite on pourra utiliser minmax jespere.

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
//ouverture d'un fichier de donnée type csv et traitement
void MainWindow::on_OUVRIR_clicked()
{
 
QFile file("D:/Mes Documents/Arduino/PROJETS YANN/BANC AAC/LOGICIEL/BANC_ARBRE_A_CAMES_V1_0_2/serial2.csv");
 
    if(file.open(QIODevice::ReadOnly|QIODevice::Text))
 
    QTextStream stream(&file);
 
    while(!file.atEnd())
 
    {
        QString line = file.readLine();
             QStringList list = line.split(";");
 
             QStringList columns2 = {list[2]};
             colonne = columns2[0].toDouble();
 
    qDebug() << colonne;
 
    }
 
 
 
}

1 pièce(s) jointe(s)

On touche au but :) cela fonctionne enfin :aie::aie:

N'ayant pas reussi a utiliser la fonction std::min max j'ai contourner le probleme en utilisant std::sort
Après si vous avez la syntaxe exact pour utiliser directement std::minmax on est preneur :mrgreen::mrgreen:

merci a vous

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
void MainWindow::on_OUVRIR_clicked()
{
QList<double>listefinale;
QFile file("D:/Mes Documents/Arduino/PROJETS YANN/BANC AAC/LOGICIEL/BANC_ARBRE_A_CAMES_V1_0_2/serial.csv");
 
    if(file.open(QIODevice::ReadOnly|QIODevice::Text))
 
    QTextStream stream(&file);
 
    while(!file.atEnd())
 
    {
        QString line = file.readLine();
//on separe les valeur de la  ligne a chaque :
        QStringList list = line.split(";");
 
//on extrait la 4eme valeur de la ligne :
             QStringList columns2 = {list[3]};
 
//on transforme la valeur en variable double :
             colonne = columns2[0].toDouble();
 
// on  insere la variable dans la liste :
 
             listefinale<< colonne ;
 
 }
 
// on range la liste dans l'ordre croissant:
 
std::sort(listefinale.begin(), listefinale.end());
 
// 1ere valeur = min et derniere valeur = max :
 
Imin= listefinale.first();
Imax= listefinale.last();
 
    qDebug()<< listefinale;
    qDebug()<<"Valeur min de la liste :"<<Imin;
    qDebug()<<"Valeur max de la liste :"<<Imax;

Pièce jointe 555840

après réflexion voici comment evité de passer par l'etape de classement croissant pour extraire le min et le max , avec un petit algo tout bête:

Code:

1
2
3
4
5
6
7
8
9
10
 
        Imin = listefinale[0];
        Imax = listefinale[0];
        for (int i=1; i<listefinale.size(); i++)
        {
          if (listefinale[i] < Imin)
            Imin = listefinale[i];
        if (listefinale[i] > Imax)
            Imax = listefinale[i];
        }

Sinon, pour utiliser la STL, tu pourrais essayer ceci, avec deux itérateurs :

Code:

auto [min, max] = std::minmax_element(listefinale.begin(), listefinale.end());

(Ça vient de https://en.cppreference.com/w/cpp/al...minmax_element.) Deux avantages : moins de code à écrire, moins de code à indenter :mrgreen:. A priori, les deux fonctions ont la même complexité, mais c'est probable que la fonction de la STL sera mieux optimisée. Attention que cette fonction renvoie deux itérateurs, pas directement les valeurs.

Sinon, vraiment, pense à indenter ton code correctement. En l'état, c'est assez difficile à lire. Par exemple, pour le dernier bout de code :
Code:

1 2 3 4 5 6 7 8 9 Imin = listefinale[0]; Imax = listefinale[0]; for (int i=1; i<listefinale.size(); i++) { if (listefinale[i] < Imin) Imin = listefinale[i]; if (listefinale[i] > Imax) Imax = listefinale[i]; }

17/04/2020, 11h45
YannGTT

Merci Dourouc05. Je vais egalement essayer cette méthode. Je n'avais pas compris que cela renvoyer un itérateur et non la valeur .

Pour l'écriture je comprend , mais étant novice en programmation ce n'est pas un réflexe. Mais il faut que je m'y force car effectivement c'est plus lisible . ;)
17/04/2020, 16h56
dourouc05

Citation:

Envoyé par YannGTT

Pour l'écriture je comprend , mais étant novice en programmation ce n'est pas un réflexe. Mais il faut que je m'y force car effectivement c'est plus lisible . ;)

Normalement, Qt Creator s'en charge pour toi, comme tous les environnements de développement (EDI) dignes de ce nom.
28/04/2020, 03h13
YannGTT

Bonsoir

Désormais je sais récupérer les valeur min et max de la liste , mais là j'ai besoin de connaitre leur position dans la liste (enfin juste la position de la max ):roll:.
Donc logiquement la valeur de l'index [i]?
Jai essayé différentes solutions avec std::min max mais rien ne fonctionne :calim2::calim2:

je suis bloqué dessus et sans ca impossible de déclenchés les calculs suivant :oops::oops:

Merci à vous

Citation:

Envoyé par YannGTT

Donc logiquement la valeur de l'index [i]?
Jai essayé différentes solutions avec std::min max mais rien ne fonctionne :calim2::calim2:

je suis bloqué dessus et sans ca impossible de déclenchés les calculs suivant :oops::oops:

Es-tu sérieux ? :weird: :weird: :weird:

Parce que c'est très très très difficile :ptdr: :ptdr: :ptdr: :ptdr: :ptdr:

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
        value_min = listefinale[0];
        index_min = 0;
        value_max = listefinale[0];
        index_max = 0;
 
        for (int i=1; i < listefinale.size(); i++) {
			if (listefinale[i] < value_min) {
				value_min = listefinale[i];
				index_min = i;
                        }
			if (listefinale[i] > value_max){
				value_max = listefinale[i];
				index_max = i;
                        }
        }

Version avec plus d'accès tableau :

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
        index_min = 0;
        index_max = 0;
 
        for (int i=1; i < listefinale.size(); i++) {
			if (listefinale[i] < listefinale[index_min]) {
				index_min = i;
                        }
			if (listefinale[i] > listefinale[index_max]){
				index_max = i;
                        }
        }
 
        value_min = listefinale[index_min];
        value_max = listefinale[index_max];

Sinon un exemple avec std::min_element (pour retourner un itérateur) et ensuite utiliser std::distance (<- 2 liens cplusplus.com en anglais)

Code:

1
2
3
4
5
6
        std::vector<int> v{3, 1, 4, 1, 5, 9};
 
//      We need to get the min value
        std::vector<int>::iterator result = std::min_element(std::begin(v), std::end(v));
//      Then we get the index of the value in the array
        std::cout << "min element at: " << std::distance(std::begin(v), result);

28/04/2020, 10h42
YannGTT

Merci à toi foetus ;)

Cela parait surement une évidence pour un dévellopeur chevronné , mais n'oubliez pas que je n'avais JAMAIS lu ou ecrit une ligne de code il y a encore 1mois:roll::roll: .
Le confinement m'autorise à passer énormement d'heures ( cf l'heure de mon dernier post , les nuits blanches s'accumules:mouarf:) à explorer le net et à rechercher / appprendre les bases de ce langage mais cela n'est pas du tout simple même avec une grosse motivation et de la persévérence.

je viens de tester et cela ne fonctionne pas :(
la valeur max est bien la bonne mais la position qu'il me renvoie est la dernière de la liste :weird::weird:

alors après avoir lu et relue https://qt.developpez.com/doc/4.7/ql...erator-typedef j'ai trouver la solution :ptdr::ptdr::

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
Imin = listeVcapteur1[0];
    Imax = listeVcapteur1[0];
 
 
        for (int i=0; i<listeVcapteur1.size(); i++)
 
            {
              if (listeVcapteur1[i] < Imin)
                  Imin = listeVcapteur1[i];
 
              if (listeVcapteur1[i] >Imax)
                  Imax = listeVcapteur1[i];
 
            }
 
        int i=listeVcapteur1.indexOf(Imax);
            if (i!= -1)
                index_max = i;
 
                qDebug()<<"postion Imax est ="<<i;

28/04/2020, 18h25
dourouc05

Citation:

Envoyé par foetus

Parce que c'est très très très difficile :ptdr: :ptdr: :ptdr: :ptdr: :ptdr:

Sauf que tu proposes une solution en C, bien dégueulasse et probablement pas aussi efficace que ça pourrait l'être :aie:. À part à la toute fin de ton message, tu encourages du code extrêmement douteux en C++.

Sinon, un itérateur, c'est pareil qu'un index, en plus général. La solution proposée par foetus tout à la fin (à base de std::distance) permet de passer d'un itérateur à une position, c'est l'idéal en C++ pour profiter de toutes les fonctionnalités du langage. Là, tu auras un code très court, sans aucune boucle écrite par toi, relativement clair (beaucoup plus que la boucle), bien mieux optimisé que ce que tu aurais fait seul dans ton coin.

Fuseau horaire GMT +2. Il est actuellement 09h53.