KL
Konstantin LazarevJan. 31, 2017, 9:34 a.m.

QNetworkAccessManager определение позиции сайта в Google по запросу

Code Review

Написал приложение по определению позиции сайта в выдаче по запросу и определение ТИЦ первой 10. Хочу получить комментарии к коду и какие элементы кода программы стоит доработать.

position.h
We recommend hosting TIMEWEB
We recommend hosting TIMEWEB
Stable hosting, on which the social network EVILEG is located. For projects on Django we recommend VDS hosting.

Do you like it? Share on social networks!

6
KL
  • Jan. 31, 2017, 9:37 a.m.
#ifndef POSITION_H
#define POSITION_H

#include <QMainWindow>
#include "network_my.h"
#include "makesaytslist.h"

namespace Ui {
class Position;
}

class Position : public QMainWindow
{
    Q_OBJECT

public:
    explicit Position(QWidget *parent = 0);
    ~Position();

private slots:
    void on_pushButton_clicked();
    void setResults();
    void setImage();


private:
    Ui::Position *ui;
    network_my Network;
    MakeSaytsList Make;
};

#endif // POSITION_H

    KL
    • Jan. 31, 2017, 9:38 a.m.
    #include "position.h"
    #include "ui_position.h"
    #include <QDebug>
    Position::Position(QWidget *parent) :
        QMainWindow(parent),
        ui(new Ui::Position)
    {
        ui->setupUi(this);
        ui->lineEdit_2->setText("gazetax.ru");
        ui->comboBox->addItem("google.ru");
           ui->comboBox->addItem("google.com.ua");
           connect(&Network,SIGNAL(PageDownload()),this,SLOT(setResults()));
           connect(&Network,SIGNAL(ImageDownload()),this,SLOT(setImage()));
    }
    
    Position::~Position()
    {
        delete ui;
    }
    
    void Position::on_pushButton_clicked()
    {
    
        Network.setMainPage(Make.MakeGoogleQuery(ui->comboBox->currentText(), ui->lineEdit->text()));
    }
    
    void Position::setResults()
    {
        QStringList ListSayt;
        ui->plainTextEdit->clear();   // очистка поля
            ListSayt=Network.getListSaytsGoogle();    // Возвращает список сайтов из выдачи Google
            for(int i=0; i<10; i++)
            {
               QString number;
               number=QString::number(1+i);
                ui->plainTextEdit->appendPlainText(number+" "+ListSayt[i]);
            }
            int counter_2=0;
                for(int i=0; i<ListSayt.size(); i++)  // Определяет позицию искомого сайта
                {
                    if(ListSayt[i]==ui->lineEdit_2->text())
                    {
                         counter_2++;
                       ui->lcdNumber->display(++i);
                       break;
                    }
                }
                if(counter_2==0)
                {
                    ui->lcdNumber->display(0);
                }
                Network.setImageForTic();
    }
    
    void Position::setImage()
    {
    
            ui->label->setPixmap(Network.getImage(0));
            ui->label_2->setPixmap(Network.getImage(1));
            ui->label_3->setPixmap(Network.getImage(2));
            ui->label_4->setPixmap(Network.getImage(3));
            ui->label_5->setPixmap(Network.getImage(4));
            ui->label_6->setPixmap(Network.getImage(5));
            ui->label_7->setPixmap(Network.getImage(6));
            ui->label_8->setPixmap(Network.getImage(7));
            ui->label_9->setPixmap(Network.getImage(8));
            ui->label_10->setPixmap(Network.getImage(9));
    
    }
    
    
      KL
      • Jan. 31, 2017, 9:39 a.m.
      #ifndef NETWORK_MY_H
      #define NETWORK_MY_H
      #include <QObject>
      #include <QNetworkAccessManager>
      #include <QNetworkRequest>
      #include <makesaytslist.h>
      #include <QPixmap>
      class network_my: public QObject
      {
          Q_OBJECT
      public:
          network_my(QObject *parent=0);
          void setMainPage(QString NameSayte); // Парсит выбраный вами файл или запрос
          QStringList getListSaytsGoogle();
          void setImageForTic();
          QPixmap getImage(int i);
      
      private:
          QString MainPage; //  Содержимое спарсеного документа
          QNetworkAccessManager manager;  // Класс для работы с сетью
          QNetworkRequest request;
          MakeSaytsList Make;
          QVector<QPixmap> pixmap;// Вектор картинок с тицом сайтов
          QStringList SaytListForTic; // Список сайтов из выдачи гугла
          int counter; //Счетчик для загрузки изображений
      private slots:
          void ParsingMainPage(QNetworkReply *reply);
          void DownloadImage(QNetworkReply*reply);
          void DisconnectSignals();
          void DownloadImageForTic(int i);
      signals:
          void PageDownload();
          void ImageDownload();
      };
      
      #endif // NETWORK_MY_H
      
      
        KL
        • Jan. 31, 2017, 9:39 a.m.
        #include "network_my.h"
        #include <QUrl>
        #include <QDebug>
        #include <QNetworkReply>
        network_my::network_my(QObject *parent):QObject(parent)
        {
        
        }
        
        void network_my::setMainPage(QString NameSayte)
        {
            DisconnectSignals();
            connect(&manager,SIGNAL(finished(QNetworkReply*)),SLOT(ParsingMainPage(QNetworkReply*)));
            QUrl url;
               url.setUrl(NameSayte);
               request.setUrl(url);
               manager.get(request);
        }
        
        QStringList network_my::getListSaytsGoogle()
        {
            return Make.SaytsList(MainPage);
        }
        
        void network_my::setImageForTic()
        {
              DisconnectSignals();
             connect(&manager,SIGNAL(finished(QNetworkReply*)),this,SLOT(DownloadImage(QNetworkReply*)));
             SaytListForTic=Make.SaytsList(MainPage);
             pixmap.clear();
             counter=0;
             DownloadImageForTic(counter);
        }
        
        QPixmap network_my::getImage(int i)
        {
            return pixmap.at(i);
        }
        
        void network_my::ParsingMainPage(QNetworkReply* reply)
        {
            if(reply->error())
            {
                        qDebug() << "ERROR";
                        qDebug() << reply->errorString(); //проверка полученого текста на ошибки
                    }
            else {
                        MainPage=QString::fromUtf8(reply->readAll());
                        reply->deleteLater();
                        emit PageDownload();
            }
        }
        
        void network_my::DownloadImage(QNetworkReply *reply)
        {
            QPixmap picture;
               picture.loadFromData(reply->readAll());
               pixmap.push_back(picture);
               counter++;
               DownloadImageForTic(counter);
               reply->deleteLater();
        }
        
        void network_my::DisconnectSignals()
        {
            disconnect(&manager,SIGNAL(finished(QNetworkReply*)),this,SLOT(ParsingMainPage(QNetworkReply*)));
            disconnect(&manager,SIGNAL(finished(QNetworkReply*)),this,SLOT(DownloadImage(QNetworkReply*)));
        }
        
        void network_my::DownloadImageForTic(int i)
        {
            QUrl url;
            if(i<10)
               {
               url.setUrl("http://www.yandex.ru/cycounter?"+SaytListForTic[i]+"");
               request.setUrl(url);
               manager.get(request);
               }
               else
               {
                counter=0;
                emit ImageDownload();
                /*
                    urlGoogle.setUrl( Make.MakeGoogleQueryAdvertising(Google_Region, Query_User));
                     request.setUrl(urlGoogle);
                    managerAdvertising.get(request);
                   counter=0;
                   */
               }
        
        }
        
        
          Evgenii Legotckoi
          • Jan. 31, 2017, 11:12 a.m.
          • The answer was marked as a solution.

          1. В Qt используется стиль Camel Style для написания названий классов и методов. Стоит придерживаться этого стиля. Поэтому лучше переименовть network_my в NetworkMy или MyNetwork

          2.

          connect(&Network,SIGNAL(PageDownload()),this,SLOT(setResults()));
          

          Это устаревший синтаксис сигналов и слотов, в котором не отлавливаются ошибки на этапе компиляции. В крупном проекте это чревато нудным и долгим поиском проблемы. Используйте новый синтаксис на указателях.

          connect(button, &QPushButton::clicked, this, &MainWindow::slotButton);
          

          3. Вот это просто некрасиво выглядит. Я бы переписал с динамическим созданием компонентов.

          ui->label->setPixmap(Network.getImage(0));
          ui->label_2->setPixmap(Network.getImage(1));
          ui->label_3->setPixmap(Network.getImage(2));
          ui->label_4->setPixmap(Network.getImage(3));
          ui->label_5->setPixmap(Network.getImage(4));
          ui->label_6->setPixmap(Network.getImage(5));
          ui->label_7->setPixmap(Network.getImage(6));
          ui->label_8->setPixmap(Network.getImage(7));
          ui->label_9->setPixmap(Network.getImage(8));
          ui->label_10->setPixmap(Network.getImage(9));
          

          4. Перемудрили вот c этим кодом:

          int counter_2=0;
                      for(int i=0; i<ListSayt.size(); i++)  // Определяет позицию искомого сайта
                      {
                          if(ListSayt[i]==ui->lineEdit_2->text())
                          {
                               counter_2++;
                             ui->lcdNumber->display(++i);
                             break;
                          }
                      }
                      if(counter_2==0)
                      {
                          ui->lcdNumber->display(0);
                      }
          

          Вот так будет явно проще и более красиво:

          ui->lcdNumber->display(ListSayt.indexOf(ui->lineEdit_2->text()) + 1 );
          

          5. И здесь тоже много лишнего:

          QString number;
                     number=QString::number(1+i);
                      ui->plainTextEdit->appendPlainText(number+" "+ListSayt[i]);
          

          Лучше так:

          ui->plainTextEdit->appendPlainText(QString::number(1+i)+" "+ListSayt[i]);
          

          6. Ну и код стайл. Грязновато написано, отступы, скобки и т.д. по всему коду. Много перечислять.

          7. Если покопаться, то можно ещё много нарыть думаю, но пока и этого хватит.

            KL
            • Jan. 31, 2017, 11:48 a.m.

            Спасибо, попробую исправить. Без помощи знающих людей сложно чему то учиться. Благодарен вам за помощь!

              Comments

              Only authorized users can post comments.
              Please, Log in or Sign up
              AD

              C ++ - Test 004. Pointers, Arrays and Loops

              • Result:50points,
              • Rating points-4
              m

              C ++ - Test 004. Pointers, Arrays and Loops

              • Result:80points,
              • Rating points4
              m

              C ++ - Test 004. Pointers, Arrays and Loops

              • Result:20points,
              • Rating points-10
              Last comments
              Evgenii Legotckoi
              Evgenii LegotckoiOct. 31, 2024, 2:37 p.m.
              Django - Lesson 064. How to write a Python Markdown extension Добрый день. Да, можно. Либо через такие же плагины, либо с постобработкой через python библиотеку Beautiful Soup
              A
              ALO1ZEOct. 19, 2024, 8:19 a.m.
              Fb3 file reader on Qt Creator Подскажите как это запустить? Я не шарю в программировании и кодинге. Скачал и установаил Qt, но куча ошибок выдается и не запустить. А очень надо fb3 переконвертировать в html
              ИМ
              Игорь МаксимовOct. 5, 2024, 7:51 a.m.
              Django - Lesson 064. How to write a Python Markdown extension Приветствую Евгений! У меня вопрос. Можно ли вставлять свои классы в разметку редактора markdown? Допустим имея стандартную разметку: <ul> <li></li> <li></l…
              d
              dblas5July 5, 2024, 11:02 a.m.
              QML - Lesson 016. SQLite database and the working with it in QML Qt Здравствуйте, возникает такая проблема (я новичок): ApplicationWindow неизвестный элемент. (М300) для TextField и Button аналогично. Могу предположить, что из-за более новой верси…
              k
              kmssrFeb. 8, 2024, 6:43 p.m.
              Qt Linux - Lesson 001. Autorun Qt application under Linux как сделать автозапуск для флэтпака, который не даёт создавать файлы в ~/.config - вот это вопрос ))
              Now discuss on the forum
              Evgenii Legotckoi
              Evgenii LegotckoiJune 24, 2024, 3:11 p.m.
              добавить qlineseries в функции Я тут. Работы оень много. Отправил его в бан.
              t
              tonypeachey1Nov. 15, 2024, 6:04 a.m.
              google domain [url=https://google.com/]domain[/url] domain [http://www.example.com link title]
              NSProject
              NSProjectJune 4, 2022, 3:49 a.m.
              Всё ещё разбираюсь с кешем. В следствии прочтения данной статьи. Я принял для себя решение сделать кеширование свойств менеджера модели LikeDislike. И так как установка evileg_core для меня не была возможна, ибо он писался…
              9
              9AnonimOct. 25, 2024, 9:10 a.m.
              Машина тьюринга // Начальное состояние 0 0, ,<,1 // Переход в состояние 1 при пустом символе 0,0,>,0 // Остаемся в состоянии 0, двигаясь вправо при встрече 0 0,1,>…

              Follow us in social networks