[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]

Originariamente inviato da 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 .....

[Teo@Unix]

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

[cionci]

Originariamente inviato da Teo@Unix

.. 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à ... ... 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??

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()? ....

[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

[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-Programm...t-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

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);