KL
Konstantin Lazarev31 января 2017 г. 9:34

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

Code Review

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

position.h
Рекомендуем хостинг TIMEWEB
Рекомендуем хостинг TIMEWEB
Стабильный хостинг, на котором располагается социальная сеть EVILEG. Для проектов на Django рекомендуем VDS хостинг.

Вам это нравится? Поделитесь в социальных сетях!

6
KL
  • 31 января 2017 г. 9:37
#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
    • 31 января 2017 г. 9:38
    #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
      • 31 января 2017 г. 9:39
      #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
        • 31 января 2017 г. 9:39
        #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
          • 31 января 2017 г. 11:12
          • Ответ был помечен как решение.

          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
            • 31 января 2017 г. 11:48

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

              Комментарии

              Только авторизованные пользователи могут публиковать комментарии.
              Пожалуйста, авторизуйтесь или зарегистрируйтесь
              AD

              C++ - Тест 004. Указатели, Массивы и Циклы

              • Результат:50баллов,
              • Очки рейтинга-4
              m
              • molni99
              • 26 октября 2024 г. 1:37

              C++ - Тест 004. Указатели, Массивы и Циклы

              • Результат:80баллов,
              • Очки рейтинга4
              m
              • molni99
              • 26 октября 2024 г. 1:29

              C++ - Тест 004. Указатели, Массивы и Циклы

              • Результат:20баллов,
              • Очки рейтинга-10
              Последние комментарии
              i
              innorwall11 ноября 2024 г. 22:12
              Django - Урок 055. Как написать функционал auto populate field Freckles because of several brand names retin a, atralin buy generic priligy
              i
              innorwall11 ноября 2024 г. 18:23
              QML - Урок 035. Использование перечислений в QML без 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
              innorwall11 ноября 2024 г. 15:50
              Qt/C++ - Урок 052. Кастомизация Qt Аудио плеера в стиле 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
              innorwall11 ноября 2024 г. 14:19
              Алгоритм сортировки кучей The role of raloxifene in preventing breast cancer priligy precio
              i
              innorwall11 ноября 2024 г. 13:55
              PyQt5 - Урок 006. Работа с QTableWidget buy priligy 60 mg 53 have been reported by Javanovic Santa et al
              Сейчас обсуждают на форуме
              i
              innorwall11 ноября 2024 г. 20:56
              добавить qlineseries в функции buy priligy senior brother Chu He, whom he had known for many years
              i
              innorwall11 ноября 2024 г. 10:55
              Всё ещё разбираюсь с кешем. 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
              9Anonim25 октября 2024 г. 9:10
              Машина тьюринга // Начальное состояние 0 0, ,<,1 // Переход в состояние 1 при пустом символе 0,0,>,0 // Остаемся в состоянии 0, двигаясь вправо при встрече 0 0,1,>…
              ИМ
              Игорь Максимов3 октября 2024 г. 4:05
              Реализация навигации по разделам Спасибо Евгений!

              Следите за нами в социальных сетях