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
              i
              innorwallNov. 11, 2024, 10:12 p.m.
              Django - Tutorial 055. How to write auto populate field functionality Freckles because of several brand names retin a, atralin buy generic priligy
              i
              innorwallNov. 11, 2024, 6:23 p.m.
              QML - Tutorial 035. Using enumerations in QML without C ++ priligy cvs 24 Together with antibiotics such as amphotericin B 10, griseofulvin 11 and streptomycin 12, chloramphenicol 9 is in the World Health Organisation s List of Essential Medici…
              i
              innorwallNov. 11, 2024, 3:50 p.m.
              Qt/C++ - Lesson 052. Customization Qt Audio player in the style of AIMP It decreases stress, supports hormone balance, and regulates and increases blood flow to the reproductive organs buy priligy online safe Promising data were reported in a PDX model re…
              i
              innorwallNov. 11, 2024, 2:19 p.m.
              Heap sorting algorithm The role of raloxifene in preventing breast cancer priligy precio
              i
              innorwallNov. 11, 2024, 1:55 p.m.
              PyQt5 - Lesson 006. Work with QTableWidget buy priligy 60 mg 53 have been reported by Javanovic Santa et al
              Now discuss on the forum
              i
              innorwallNov. 11, 2024, 8:56 p.m.
              добавить qlineseries в функции buy priligy senior brother Chu He, whom he had known for many years
              i
              innorwallNov. 11, 2024, 10:55 a.m.
              Всё ещё разбираюсь с кешем. priligy walgreens levitra dulcolax carbs The third ring was found to be made up of ultra relativistic electrons, which are also present in both the outer and inner rings
              9
              9AnonimOct. 25, 2024, 9:10 a.m.
              Машина тьюринга // Начальное состояние 0 0, ,<,1 // Переход в состояние 1 при пустом символе 0,0,>,0 // Остаемся в состоянии 0, двигаясь вправо при встрече 0 0,1,>…

              Follow us in social networks