[C++]Qt class: QTcpSocket

[Teo@Unix]

Altra questione sulle librerie Qt....
premetto che non ho verificato il seguente codice... ma la mia domanda è... come controllo che "connectToHost()" va a buon fine? dato che è void?

QHostAddress hostaddr;
QHostAddress Localhost;
QTcpSocket sock;
hostaddr.setAddress(Host);
Localhost = QHostAddress(QHostAddress::LocalHost).toString();
sock = new QTcpSocket(this);
sock.waitForConnected(timeout);
for(unsigned int curport = Sport; curport <= Eport;curport++)
{
sock.abort();
sock.connectToHost(hostaddr,curport);
}

Come sempre, grazie.

[Teo@Unix]

Altra questione sulle librerie Qt....
premetto che non ho verificato il seguente codice... ma la mia domanda è... come controllo che "connectToHost()" va a buon fine? dato che è void?

QHostAddress hostaddr;
QHostAddress Localhost;
QTcpSocket sock;
hostaddr.setAddress(Host);
Localhost = QHostAddress(QHostAddress::LocalHost).toString();
sock = new QTcpSocket(this);
sock.waitForConnected(timeout);
for(unsigned int curport = Sport; curport <= Eport;curport++)
{
sock.abort();
sock.connectToHost(hostaddr,curport);
}

Come sempre, grazie.

Dovrei aver risolto così: "QTcpSocket.connectToHost()" emette diversi segnali a seconda di ciò che accade, quindi li gestisco collegandoli a delle funzioni con connet() .... scusate ma mi dimentico che qui si parla con i segnali ..... :D

[Teo@Unix]

.. invece ... una funzione della libreria Qt che mi fa il lavoro di getservbyport() esiste? .....

[cionci]

.. invece ... una funzione della libreria Qt che mi fa il lavoro di getservbyport() esiste? .....
E a che ti serve ? La puoi usare direttamente. C'è sia su Windows che su Linux e Unix.

[Teo@Unix]

già ... :doh: ... tutte queste nuove informazioni mi mandano in confusione.....

riguardo invece a quel frammento di codice di prima....
cè qualcosa che mi stà sfuggendo...... come vedi c'è un ciclo for che cicla su connect(), il problema è che mentre il programma stà eseguendo il for l'applicazione non risponde, quindi qualsiasi bottone "stop" non funziona.... come faccio se voglio pemettere all'utente di interrompere il ciclo??:confused:

un codice di prova:
void NetScan::BeginScanning(unsigned int Sport,unsigned int Eport, unsigned short timeout,QString Host)
{
ui->dialoglabel->setText(tr("Scansione di %1").arg(Host));
QHostAddress hostaddr;
QHostAddress Localhost;
hostaddr.setAddress(Host);
Localhost = QHostAddress(QHostAddress::LocalHost).toString();

for(curport = Sport; curport <= Eport;curport++)
{
sock->abort();
sock->connectToHost(hostaddr,curport);
if(sock->waitForConnected(timeout*1000))
{
ui->outplainTextEdit->setPlainText((ui->outplainTextEdit->toPlainText()).append("Porta %1 aperta\n").arg(curport));
}
}
ui->outplainTextEdit->setPlainText((ui->outplainTextEdit->toPlainText()).append("\n *** Scansione terminata *** \n"));
}

anche i sttaggi dei label e text box vengono settati solo al termine del ciclo.... nel mentre l'applicazione rimane nell'odioso stato "(non risponde)"... dove sbaglio?
Grazie!

[cionci]

Usa un thread ;) Metti tutto in un QThread.

[Teo@Unix]

mumble ... mumble ... ma è una classe a se stante come faccio ad inserirlo? E come faccio a passare i parametri....

ad esempio se volessi eseguire la funzione descritta in un processo separato come farei?

edit: o meglio, la dichiarazione va fatta nell'header file del programma, ma il thread come può accedere ai membri della mi classe?

è da inserire nell'header? :

namespace Ui {
class NetScan;
}

class NetScan : public QDialog {
Q_OBJECT
public:

.....................
.....................

void BeginScanning(unsigned int,unsigned int, unsigned short,QString);
};

// Dichirazione come classe derivata da QThread
class scan : public QThread {
public:
void run(unsigned int,unsigned int, unsigned short,QString);
};

e in .cpp:

void scan::run(unsigned int Sport,unsigned int Eport, unsigned short timeout,QString Host)
{
ui->dialoglabel->setText(tr("Scansione di %1").arg(Host));
QHostAddress hostaddr;
QHostAddress Localhost;
hostaddr.setAddress(Host);
Localhost = QHostAddress(QHostAddress::LocalHost).toString();

for(curport = Sport; curport <= Eport;curport++)
{
sock->abort();
sock->connectToHost(hostaddr,curport);
if(sock->waitForConnected(timeout*1000))
{
ui->outplainTextEdit->setPlainText((ui->outplainTextEdit->toPlainText()).append("Porta %1 aperta\n").arg(curport));
}
}
ui->outplainTextEdit->setPlainText((ui->outplainTextEdit->toPlainText()).append("\n *** Scansione terminata *** \n"));
}

ma giustamente il compilatore mi avvisa:
"D:/NetScanner/netscan.cpp:78: error: 'ui' was not declared in this scope"
ecc ecc.....

[cionci]

Passa ui nel costruttore di scan ;)

[Teo@Unix]

ok ma...

ma dall'altra classe come posso chiamare il thread?
relativamente al codice di prima,

nella classe NetScan la nuova classe derivata da QThread non è visibile come chiamo run()? ....:mbe:

[cionci]

Come non è visibile ?
La includi e poi crei un membro privato di tipo scan e dopo dai il run quando più ti piace.

[Teo@Unix]

Aspè probabile sto facendo casotto...
come passeresti il costruttore.. io per ora ho fatto questo:

namespace Ui {
class NetScan;
class scan;
}

class scan : public QThread {
Q_OBJECT
public:
scan(NetScan); //Qui come lo passo?
~scan();

void run(unsigned int,unsigned int, unsigned short,QString);
};

class NetScan : public QDialog {
Q_OBJECT
public:
NetScan(QWidget *parent = 0);
~NetScan();

[....]

};

e poi come sarebbe l'implementazione del costruttore...

scan::scan( // ?) { }

Sui libri non ho trovato riferimenti a queste situazioni.. almeno ciò che ho letto io ...:(

[cionci]

No, nel namespace non devi mettere niente.
Devi mettere scan un membro scan in NetScan e poi inizializzare il costruttore tramite la lista di inizializzazione del costruttore di NetScan andando a passare il puntatore a ui. ui non è il namespace, ma è una variabile che devi passare anche al costruttore di NetScan, visto che contiene le istanze di tutti gli oggetti presenti nel form.
Io sinceramente cercherei di separare l'interfaccia grafica dal lavoro vero e proprio. Puoi definire delle variabili condivise da passare al thread in cui raccogliere i dati delle scansioni e a cui accedi tramite costrutti che garantiscono la mutua esclusione (QT ne ha tanti, ad esempio la QMutex).
Dopo da NetScan penserai ad aggiornare la grafica, ad esempio ogni X secondi.

[Teo@Unix]

quindi per non iniziare ad imparare metodi sbagliati mi conviene dichiarare 2 classi; una relativa all'interfaccia grafica ed una per le funzioni. Utilizzando quindi QMutex?!.. che dovrò andarmi a vedere ....

grazie, vedo cosa riesco a fare....
purtroppo il massimo che avevo fatto in questo ambito era utilizzare fork() ..... P.S. queste gui mi stanno mettendo in ginocchio:D

[cionci]

E' sempre meglio tenere separate le classi delle GUI da quelle che fanno il lavoro vero e proprio.

[Teo@Unix]

Sai mica indicarmi qualche libro? io ne ho trovati alcuni in Inglese, trattano le librerie Qt in generale, non saprei...
mi pare dicevi che esiste qualcosa in italiano...

[cionci]

In Italiano non credo che esistano libri gratuiti. In inglese sì: http://www.qtrac.eu/C++-GUI-Programming-with-Qt-4-1st-ed.zip
Anche se è basato sulle QT 4 potrebbe andare bene.

[Teo@Unix]

Thank you.

Invece per il problema di prima dovrei aver capito....

l'unica cosa.. ho un dubbio su dove dichiarare le variabili globali, queste variabili a cui dovranno accedere le due classi contemporaneamente con mutex dove sarebbe più giusto metterle secondo te?

[cionci]

Non devono essere globali.
Hai due soluzioni. O dichiari QMutex e la struttura dati per lo scambio all'interno di NetScan come variabili membro o ti fai una classe che li contiene con i vari metodi di accesso. La seconda è la soluzione che mi piace di più.
Anche nel secondo caso devi dichiarare una variabile membro (come puntatore o no dipende un po' dall'uso che ne fai).
In entrambi i casi devi passare i puntatori al costruttore di scan, della mutex e della struttura dat con il primo metodo, dell'istanza della classe con il secondo.

[Teo@Unix]

Fantastico :cool:

ho creato una terza classe, contiene variabili private e metodi pubblici.
Queste variabili devono essere lette o modificate dalle altre due classi.

La struttura QMutex, è corretto crearla all'interno della classe che contiene i dati?
#include <QString>
#include <QMutex>

class charon
{
private:
unsigned int Sport, Eport, curPort; //Da porta, a porta, porta corrente
short int timeout; //Timeout su connect()
QString Host; //Indirizzo IP o nome dell'host da scansire
QString BufResult; //Lo utilizza il thread per scriverci i risultati
QMutex Mutex; //Garantisce mutua esclusione in accesso alle variabili
public:
charon();
QString get_Host();
void set_Host(QString);
QString get_BufResult();
void set_BufResult(QString);
unsigned int get_Sport();
void set_Sport(unsigned int);
unsigned int get_Eport();
void set_Eport(unsigned int);
unsigned int get_curPort();
void set_curPort(unsigned int);
short int get_timeout();
void set_timeout(short int);
};

Poi nel file cpp implemento ogni metodo tipo:

void charon::set_BufResult(QString text)
{
Mutex.lock();
text.append(text);
Mutex.unlock();
}

Le due classi avranno dichiarato un membro "charon dati" ok?
Cosa che ancora non mi è chiarissima è dove poi inserire "#include "charon.h" e l'altra. E dove fare il passaggio ai costruttori, lo devo fare nel main.cpp?

Grazie.

[cionci]

charon *data;

Dovrà essere dichiarata come membro di NetScan.
All'interno di NetScan dovrai ad un certo punto richiamare il costruttore di charon:

data = new charon(...tutti i vari parametri...);

Poi ad un certo punto dovrai chiamare il costruttore di scan:

mScan = new scan(data);

[Teo@Unix]

Ok, ora mi è chiaro dovrei aver fatto correttamente, debuggando mi sono assicurato che le classi accedono agli stessi dati, quel che non mi torna è cha la mia amica gui non si accorge che il thread termina, io ho usato connent in questo modo:
connect(MyPortScanner,SIGNAL(terminated()),this,SLOT(UpdateResult()));

è il metodo giusto per procedere?

[cionci]

Mi incolli l'header della gui ?

[Teo@Unix]

eccolo

#ifndef MAINWINDOW_H
#define MAINWINDOW_H

#include <QMainWindow>
#include <QMessageBox>
#include <QTimer>
#include "charon.h"
#include "scanport.h"

namespace Ui {
class MainWindow;
}

class MainWindow : public QMainWindow {
Q_OBJECT
public:
MainWindow(QWidget *parent = 0);
~MainWindow();
enum Mode { ScanMode, StanbyMode };

protected:
void changeEvent(QEvent *e);

private:
Ui::MainWindow *ui;
// Dati
charon *dati;
// Thread
ScanPort *MyPortScanner;
// Funzioni
void SetInterface(Mode);

private slots:
void InitScanner();
void UpdateResult();
};

#endif // MAINWINDOW_H

già che ci sono metto anche il cpp,

#include "mainwindow.h"
#include "ui_mainwindow.h"

MainWindow::MainWindow(QWidget *parent) :
QMainWindow(parent),
ui(new Ui::MainWindow)
{
ui->setupUi(this);
dati = new charon;
MyPortScanner = new ScanPort(dati);
// Connetto pulsanti e slot
connect(ui->startpushButton,SIGNAL(clicked()),this,SLOT(InitScanner()));
// Connetto eventi del thread e slot
connect(MyPortScanner,SIGNAL(terminated()),this,SLOT(UpdateResult()));
// Setto l'interfaccia
SetInterface(StanbyMode);
}

MainWindow::~MainWindow()
{
delete ui;
}

void MainWindow::changeEvent(QEvent *e)
{
QMainWindow::changeEvent(e);
switch (e->type()) {
case QEvent::LanguageChange:
ui->retranslateUi(this);
break;
default:
break;
}
}

void MainWindow::SetInterface(Mode currentmode)
{
switch(currentmode)
{
case ScanMode:
{
ui->startpushButton->setEnabled(false);
ui->stoppushButton->setEnabled(true);
ui->hostlineEdit->setEnabled(false);
ui->sportspinBox->setEnabled(false);
ui->eportspinBox->setEnabled(false);
ui->timeoutspinBox->setEnabled(false);
ui->captionlabel->setText(tr("Scansione di \"%1\"").arg(dati->get_Host()));
break;
}
case StanbyMode:
{
ui->startpushButton->setEnabled(true);
ui->stoppushButton->setEnabled(false);
ui->hostlineEdit->setEnabled(true);
ui->timeoutspinBox->setEnabled(true);
ui->sportspinBox->setEnabled(true);
ui->eportspinBox->setEnabled(true);
ui->captionlabel->clear();
break;
}
}
}

void MainWindow::InitScanner()
{
QTimer time;
//Controllo del campo host name
if(ui->hostlineEdit->text().isEmpty())
{
QMessageBox::information(this,tr("Campo host vuoto"),tr("Devi specificare un indirizzo o un nome FQDN."));
return;
}
//Controllo che il range delle porte sia valido
if(ui->sportspinBox->value() > ui->eportspinBox->value())
{
QMessageBox::information(this,tr("Range di porte non valido"),tr("Il range di porte dichiarato non può essere negativo"));
return;
}
// Preparo dati
dati->set_Sport(ui->sportspinBox->value());
dati->set_Eport(ui->eportspinBox->value());
dati->set_Host(ui->hostlineEdit->text());
dati->set_timeout(ui->timeoutspinBox->value());
// Inizio Thread
SetInterface(ScanMode);
MyPortScanner->run();
}

void MainWindow::UpdateResult()
{
// Guardo nel buffer se ci sono nuovi dati
ui->outplainTextEdit->setPlainText(ui->outplainTextEdit->toPlainText().append(dati->get_BufResult()));
}

edit:
tra l'altro è parecchio strano, ora ho inserito anche lo slot per il pulsante "stop", questo lavora correttamente andano ad interrompermi il thread e a chiamare UpdateResult() e SetInterface() che mi aggiorna l'interfaccia in modo corretto.

Ora poi.....
sono a casa e sul pc di casa.... la gui ha ripreso a non rispondere quando arriva a Thread->run()... :sbavvv:

[Teo@Unix]

debuggando sembra che quando arrivo a thread->run() il programma si comporti come se fosse una fuzione...

[cionci]

Per far partire il thread non devi chiamare il metodo run, ma il metodo start.

[Teo@Unix]

Acciderbolina...

funziona!
resta il problema che la gui non si accorge che il thread finisce ....
ho inserito lo slot per il tasto stop, l'ho fatto in questo modo: sembra ok

void MainWindow::TerminateScanner()
{
MyPortScanner->wait();
MyPortScanner->terminate();
SetInterface(StanbyMode);
dati->set_BufResult(tr("\n-----> Scansione interrotta dall'utente\n"));
UpdateResult();
}

mi sorge un dubbio, che io abbia sbagliato l'implementazione di run()?
ecco il codice:
header:
#ifndef SCANPORT_H
#define SCANPORT_H

#include <QThread>
#include <QTcpSocket>
#include <QHostAddress>
#include "charon.h"

class ScanPort : public QThread //Thread ScanPort eseguirà la scansione
{
private:
charon *dati;
public:
ScanPort(charon*); //Costruttore
void run(); //Funzione per la scansione delle porte
};

#endif // SCANPORT_H
cpp:
void ScanPort::run()
{
QTcpSocket sock;
QHostAddress remoteHost;
QHostAddress localHost;
remoteHost.setAddress(dati->get_Host());
localHost = QHostAddress(QHostAddress::LocalHost).toString();

unsigned int curPort;
unsigned int Sport = dati->get_Sport();
unsigned int Eport = dati->get_Eport();
short int timeout = dati->get_timeout();

for(curPort = Sport; curPort <= Eport; curPort++)
{
sock.abort();
sock.connectToHost(remoteHost,curPort);
if(sock.waitForConnected(timeout*1000))
{
dati->set_BufResult(tr("Porta %1 aperta\n").arg(curPort));
}
}
dati->set_BufResult(tr("\n *** Scansione completata *** \n"));
}

[cionci]

Così funziona:

connect(&thread, SIGNAL(started()), this, SLOT(showMaximized()));
connect(&thread, SIGNAL(finished()), this, SLOT(showNormal()));

[Teo@Unix]

Nel senso che devo usare il segnale finished()? O intendevi un'altra cosa?

edit:
mi rispondo da solo, Si devo usare finished(). A questo punto dovrebbe funzionare, mi manca da definire qualcosina, ma posso dire di aver fatto qualcosa che abbia una grafica.:D

Il tuo supporto mi è stato molto d'aiuto.
Ti ringrazio per la continua disponibilità.:)

adesso studio :D