Create a matrix from a .csv file

[mrjj]

@AliM93
Hi
Well if you want to pop it up anyway, you could leave it as QDialog and draw directly to it but
could be more handy if it was standalone and you would add it to a QDialog but this is not critical.

well you just inherit QWidget instead of QDialog and change in cpp where it calls base in constructor.
xxx) : QDialog(parent) {} -> ) : QWidget(parent) {}

class GridWidget : public QWidget

[AliM93]

This post is deleted!

[mrjj]

@AliM93
Ah sorry my bad.
You have an UI file with it and inside that it says its still a QDialog so that what it complains about
So you must open that UI in text editor and change type ( Creator should be close)
or remove that it has one.
That is matrix.h line 23
and also the setupUI thing and the ui thing line 20

Its hard to show when you use images for code.

So do you need UI file ?
To add other widgets on top of grid ?

[AliM93]

@mrjj what if i do from new? maybe more clear for me to learn something. and sorry fro the images, up to now i will paste the code.

[mrjj]

@AliM93
Yes lets do a clean widget. I think its good to learn as Creator can really help in this.
give me 5. ill make some shots.

[mrjj]

1:

Choose New file or Project

2:

Select C++ class

3:
Give it a name and set base class to Qwidget

press NEXT, and then next again.

4:
we now have the new class and its cpp and .h added to the project

Lets add PaintEvent and MousePress

Right click the name of the class in the .h file (MatrixWidget)
and select the Refactor menu and add virtual function

Now type Paint in top
and check the paintEvent function
make sure the Insert option is as shown

Dont press OK yet!

now type Press in top (where you had paint)
and find the MousePress function and check that.

Now Press ok.

Then we are back to the code with our new functions.

Press F4 to switch to cpp file and see it has given you the bodies also.

Now you made your custom widget. ( it might seems a lot first time, but its very fast when used to it :)

So next part is to add code to paintEvent to draw something.
We also need your old reader code to get the data and stuff into our structure.

[mrjj]

hi
I hope the above did work for you.
If yes. Next step is the structure to hold the data.

struct DataPoint {
    int x;
    int y;
    bool value;
    QColor DrawColor; // we change this from selected / not selected.
    bool isSelected; // we set this when we click on it.
};

const int max_x = 14;
const int max_y = 21;

class MatrixWidget : public QWidget
{
    Q_OBJECT
    DataPoint Data[max_x][max_y];  /// this is where we want to load the data to
public:
    explicit MatrixWidget(QWidget *parent = nullptr);

signals:


    // QWidget interface
protected:
    virtual void mousePressEvent(QMouseEvent *event) override;
    virtual void paintEvent(QPaintEvent *event) override;
};

[AliM93]

sorry ok i put the function

[mrjj]

@AliM93
Thats the Insert Option part

Default its on something else.

But no worries.

Do you have them in .h with body also like

If yes, then click on the function name and go to the refactor menu again ( or press alt+enter)

and ask it to move it to .CPP
do for both functions.

[AliM93]

@mrjj the code that you wrote me i have to write into .h or .cpp? into the .h i suppose
we need my previous code to read the data from csv?

[mrjj]

@AliM93 said in Create a matrix from a .csv file:

• the code that you wrote me i have to write into .h or .cpp? into the .h i suppose
  Yes the DataPoint class goes into .h as shown higher up.

we need my previous code to read the data from csv?

Yes, this has to be changed to add the values to the
DataPoint Data[max_x][max_y];
please post it.

[AliM93]

  csvModel = new QStandardItemModel(this);
  csvModel->setColumnCount(3);
  csvModel->setHorizontalHeaderLabels(QStringList() <<"x" <<y"" << "bool");
  ui->tableView_1->setModel(csvModel);
  
  QFile file("/home/alice/catkin_ws/src/agree_gui/resources/Punti_tappetino.csv");
  if(!file.open(QFile::ReadOnly | QFile::Text))
  {
    qDebug()<< "FIle not exist";
    
  }
  else {
    QTextStream in(&file);
    while (!in.atEnd())
    {
      QString line = in.readLine();
      QList<QStandardItem *>standardItemsList;
      for(QString item:line.split(";"))
        
    }
    csvModel ->insertRow(csvModel->rowCount(), standardItemsList);
  }
  file.close();
```this is in the .cpp file, and in the .h i put 

QStandardItemsModel *csvModel;

there are also some include in the cpp : 

#include <QTableView>

#include <iostream>
#include <fstream>
#include <sstream>

and om the .h 

#include <QStandardItemModel>

[mrjj]

Hi
Super
Now we add a new function to our new class.
lets call it LoadData.

add these to the .cpp 
#include <QFile>
#include <QTextStream>
#include <QDebug>

// this is changed version of your reader code
void MatrixWidget::LoadData()
{
    QFile file("/home/alice/catkin_ws/src/agree_gui/resources/Punti_tappetino.csv");
    if (!file.open(QFile::ReadOnly | QFile::Text)) {
        qDebug() << "FIle not exist";

    } else {
        QTextStream in(&file);
        while (!in.atEnd()) {
            QString line = in.readLine();
            QStringList list = line.split(";");
            int x = list.at(0).toInt();
            int y = list.at(1).toInt();
            bool value = list.at(2).toInt();
            if ( x < max_x && y < max_y) // to make sure we dont crash
                Data[x][y].value = value; // here we store value
            else
                qDebug() << "x or y bigger than matrix!";
        }

    }
    file.close();
}

Do you follow so far ?
I have not tried this before so not sure how easy its to follow :)

[AliM93]

@mrjj yes i understand all the flow! super clear, so far.

[mrjj]

@AliM93
Super!
Could you open Punti_tappetino.csv in textEditor and copy it to say
https://paste.ofcode.org/
and give me url ?
Then i also have some real data to read it to check loaddata works :)

[AliM93]

@mrjj Done! are you seing it?

[mrjj]

@AliM93
where is the url ?

you have to press paste it and then copy the adress from the top of browser
like
https://paste.ofcode.org/vYJiPZBPqzaDNArjpqPYE7
(just test)

[AliM93]

@mrjj https://paste.ofcode.org/XLhSxBmaWcaKj2L9CapQ3s

sorry my bad

i have also to declare the function iLoadData in the .h file, right? in the private, protected or in the public slot?

[mrjj]

@AliM93
ah better :)
yes also add to public section.
we might call it from outside so its public.

[AliM93]

@mrjj ok! but i get some trouble with the struct. invalid use of non-static data member.