Deuxième défi Qt : les résultats de l'application mobile pour hôpital

Adopter une notation différent pour les variables locales et les variables membres d'une classe est une habitude à prendre, cela permet une relecture plus facile et surtout une meilleure compréhension pour des lecteurs extérieurs.

Préférer l'utilisation d'enum plutôt que de bool en paramètres de fonction (surtout constructeur) :

Server(false); 
Server(true);

Quand on commence à avoir plus de deux booléens en paramètres, on est vite perdu.

Server(Mode_Local); 
Server(Mode_Remote);

C'est beaucoup plus lisible. De plus, on peut également utiliser des QFlags avec des bits pour définir plusieurs options exclusives ou pas.

Éviter les #includes inutiles dans les fichier d'en-tête et favoriser la prédéclaration. Par exemple, dans server.h, les deux inclusions (servernode.h et sql.h) peuvent être déplacées dans le .cpp et classe Sql prédéclarées. De plus, la classe QTextStream est inutilisée (penser au nettoyage des includes, lors des nettoyages). Le temps de compilation est rapide sur un petit projet mais c'est une bonne habitude à prendre pour un grand projet. Il est très ennuyant d'attendre que trente fichiers recompilent lorsqu'on a fait une petite modification alors que, au final, seulement quinze fichiers avaient besoin de la déclaration dans l'en-tête.

QSettings settings(QSettings::UserScope, QLatin1String("Trolltech"));

Ça sent le code copier/coller, utiliser sa propre organisation.

if ((config.state() & QNetworkConfiguration::Discovered) != QNetworkConfiguration::Discovered)

Je trouve ça personellement compliqué à lire, j'aurais plus tendance à utiliser :

!config.state().testFlag(QNetworkConfiguration::Discovered)

Ne pas hésiter à utiliser des assertions Q_ASSERT() pour valider les préconditions et post-conditions de certaines fonctions critiques.

Exemple de code sale :

	int cols = rec.count();
	for( int c=0; c<cols; c++ )
	
		for( int r=0; query.next(); r++ ) {
			for( int c=0; c<cols; c++ ) {
				resultList << query.value(c).toString();
			}
		doctorCount++;
		
		// ...

On a ici trois boucles, dont deux avec le même nom de variable et une sans crochet, ce qui est une mauvaise habitude.

Globalement, l'application est esthétique. À l'usage, il appert que l'on peut déplacer certains éléments et que le résultat n'est plus des plus esthétiques, ainsi que le redimensionnement (mauvais comportement sur de petits écrans, soit sur smartphones, tablettes graphiques, etc., pourtant l'objectif du défi).

Quand on redimensionne, on perd une partie importante des fonctionnalités, on doit les garder et n'afficher qu'une liste (ou autre) des items.

On voit que tout n'est pas vraiment bien à l'échelle, on perd des informations, ce n'est pas un problème isolé.

Même dans la barre de boutons. On remarquera aussi l'absence d'une icône pour l'ECG.

Lors d'un déplacement, tout se chevauche, c'est illisible. Dans la gestion des ressources, on peut utiliser la molette de la souris, ce qui donne un résultat figé du même acabit.

Finalement, quand on ajoute de nouveaux éléments, tout n'est pas forcément visible non plus :

À regarder la section de la création de la base et de son remplissage, on est assez loin d'un code lisible :

QSqlQuery query(QSqlDatabase::database());
if ( query.exec("INSERT INTO Users VALUES('Tortue Caroline', '16ce0a28dba46bc6e211389ce8a27cb89d7bc623', 0, 0)") )
    if (query.exec("INSERT INTO Users VALUES('Dr Grocha Roucoud', '16ce0a28dba46bc6e211389ce8a27cb89d7bc623', 1, 0)") )
        if (query.exec("INSERT INTO Users VALUES('Dr Bush Jonnn', '16ce0a28dba46bc6e211389ce8a27cb89d7bc623', 1, 0)") )
            if (query.exec("INSERT INTO Users VALUES('Dr GaBi Duvers', '16ce0a28dba46bc6e211389ce8a27cb89d7bc623', 1, 0)") )
                if (query.exec("INSERT INTO Users VALUES('Litlwayt Spirit', '16ce0a28dba46bc6e211389ce8a27cb89d7bc623', 2, 0)") )
                    if (query.exec("INSERT INTO Users VALUES('Hafidi Abdel', '16ce0a28dba46bc6e211389ce8a27cb89d7bc623', 3, 0)") )
                        if (query.exec("INSERT INTO Users VALUES('admin', 'd033e22ae348aeb5660fc2140aec35850c4da997', 1000, 0)") )
                            if (query.exec("INSERT INTO Users VALUES('doctor', '1f0160076c9f42a157f0a8f0dcc68e02ff69045b', 100, 0)") )
                                if (query.exec("INSERT INTO Users VALUES('nurse', '285f9a003f671c2486a3f87ea1ad5e37699ebc38', 0, 0)") )
                                    return true;

On aurait pu largement éviter une telle imbrication de if et un non-alignement total des requêtes :

if (   query.exec("INSERT INTO Users VALUES('Tortue Caroline', '16ce0a28dba46bc6e211389ce8a27cb89d7bc623', 0, 0)") 
    && query.exec("INSERT INTO Users VALUES('Dr Grocha Roucoud', '16ce0a28dba46bc6e211389ce8a27cb89d7bc623', 1, 0)") 
    && query.exec("INSERT INTO Users VALUES('Dr Bush Jonnn', '16ce0a28dba46bc6e211389ce8a27cb89d7bc623', 1, 0)") 
    && query.exec("INSERT INTO Users VALUES('Dr GaBi Duvers', '16ce0a28dba46bc6e211389ce8a27cb89d7bc623', 1, 0)") 
    && query.exec("INSERT INTO Users VALUES('Litlwayt Spirit', '16ce0a28dba46bc6e211389ce8a27cb89d7bc623', 2, 0)") 
    && query.exec("INSERT INTO Users VALUES('Hafidi Abdel', '16ce0a28dba46bc6e211389ce8a27cb89d7bc623', 3, 0)") 
    && query.exec("INSERT INTO Users VALUES('admin', 'd033e22ae348aeb5660fc2140aec35850c4da997', 1000, 0)") 
    && query.exec("INSERT INTO Users VALUES('doctor', '1f0160076c9f42a157f0a8f0dcc68e02ff69045b', 100, 0)") 
    && query.exec("INSERT INTO Users VALUES('nurse', '285f9a003f671c2486a3f87ea1ad5e37699ebc38', 0, 0)") )
        return true;

On retrouve également pas mal de code commenté, ne laissant que très peu de code réellement compilé dans certaines fonctions, avec un résultat visiblement totalement différent de ce qui est annoncé par le commentaire :

Patient* MedDatabase::patient(int) // Retourne un patient particulier.
{
/*
    QSqlQuery query(QSqlDatabase::database());
    query.prepare("SELECT * FROM Patients WHERE Id=:idPatient");
            query.bindValue(":idPatient", idPatient);
    if ( query.exec() && query.first() )
            return Patient(query.value(0).toInt(), // Id
                                       query.value(1).toString(), // Name
                                       query.value(2).toInt()); // Room
    // TODO
    // age
    // + Item
    // + Version
    else
                    qDebug() << query.lastError().text();*/

    return new Patient(); // invalid patient
}

On remarque aussi que, une fois lancée, l'application consomme toutes les ressources processeur disponibles (sur un octocore) :

Le plus gros reproche à ce niveau : la gestion des ressources des examens. On ne doit pas gérer les ressources soi-même, on peut gérer un examen médical qui contient plusieurs ressources, sinon ce n'est pas utilisable par un médecin sans qu'il perde la majorité de ses cheveux. On doit là choisir le fichier concerné et entrer manuellement la catégorie concernée : pour le premier point, on aurait pu simuler une interaction avec du matériel médical, mais ce n'était pas primordial ; pour le second, c'est inacceptable au niveau de l'ergonomie, on ne doit pas d'abord maîtriser l'application puis être capable de s'en servir.