Discussion :

[ChrissRaz]

Bonjour à tous!!
ceci est mon premier discussion dans un forum car à chaque problème que je rencontre je trouve toujours une solution existante mais maintenant j'suis bloqué et je trouve pas de solution.
Bon voilà mon problème, je développe une application qui marche avec un fichier de configuration formaté en xml et j'aimerai utiliser QXmlStreamWriter pour modifier le fichier mais le fichier s'ouvre pas.
voici mon code=>

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
 
HXmlManager::HXmlManager()
{
    //":/flies/conf" est un fichier ressource de l'application 
    configurationFile = new QFile(":/files/conf");
 
 
    if (!configurationFile->open(QIODevice::ReadOnly))
    {
        qDebug("not  Opened");
        emit fileNotOpened("Erreur d'ouverture du fichier de configuration!");
        configurationFile->close();
        configurationDom= new QDomDocument();
        return;
    }
 
    configurationDom= new QDomDocument();
    configurationDom->setContent(configurationFile);
    configurationFile->close();
    configurationDomElement = new QDomElement(configurationDom->documentElement());
 
 
    configurationFile->open(QFile::WriteOnly | QFile::Text);//problème: impossible d'ouvrir le fichier ici!
 
    outXml = new QXmlStreamWriter();
    outXml->setDevice(configurationFile);
 
    outXml->writeStartElement("database");//juste pour le test
    outXml->writeAttribute("DBName","ho");
    outXml->writeEndElement();
 
    configurationFile->close();
}

j'ai essayé d'enlever l'ouverture pour la première ouverture mais ça montre toujours à la sortie de l'aplication "QIODevice::write (QFile, ":\files\conf"): device not open".
J'ulitilise Qt5.11.2
Besoin d'aide urgent, merci

[dourouc05]

Déjà, tu ne vérifies pas le code de retour de ton deuxième QIODevice::open. Ensuite, ça me paraît un peu tordu d'ouvrir deux fois le même fichier : ne pourrais-tu pas directement l'ouvrir en lecture et écriture, une seule fois ?

Autre source de problèmes : tu sembles lire un fichier dans tes ressources (QRC), sauf que ces ressources sont compilées dans l'exécutable et ne sont donc pas modifiables.

[ChrissRaz]

Salut!
Merci #dourouc05 , enfaite j'ai pas encore mis la vérification de l'ouverture vu que j'ai fait des petits test avant de faire le code complet mais bon c'est fait en ce moment mais c'est quand même pas ça qui règlera le problème Je crois le problème viens du fait qu'on peut pas modifier un fichier ressource du coup je vais mettre le fichier dans un dossier de mon programme et l'ouvrir en lecture et écriture dans un tel situation(quand je pense au temps que j'ai perdu pour résoudre ça je me rend compte à quel point je suis débutant ). je sais pas comment taguer ici
je vais marquer résolu maintenant, merci pour l'aide

[koala01]

Salut,

Ensuite, ça me paraît un peu tordu d'ouvrir deux fois le même fichier : ne pourrais-tu pas directement l'ouvrir en lecture et écriture, une seule fois ?

En fait, bien pire que cela ...

Ce qui me paraît particulièrement tordu, c'est de vouloir faire de la lecture ET de l'écriture dans la même fonction

SRP nous conseille en effet d'avoir:

1. une fonction pour charger le contenu du fichier en mémoire
2. une (ou plusieurs) fonction(s) pour modifier le contenu qui se trouve en mémoire
3. une fonction pour enregistrer les modifications

Cela permet de rendre chaque fonction beaucoup plus simple à comprendre, vu qu'elle ne s'occupe que d'une seule chose, le code beaucoup plus explicite, vu qu'il prendrait une forme proche de

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
auto content = read(filename),
modify(content);
write(filename, content);

Et, enfin, cela présente l'énorme avantage (ou inconvénient selon le cas) de laisser le fichier à disposition de tout autre processus qui voudrait y accéder entre le read et le write

Par contre, que l'on ne vienne pas me dire que cela réduira les performances :

Entre le XML et le traitement DOM, qui n'est clairement pas ce qui se fait de plus léger, les accès au disque dur, qui sont clairement parmi les accès les plus lents que l'on peut trouver, et l'utilisation potentielle d'une IHM (vu que l'on est dans la partie Qt) qui risque de nécessiter une action de la part de l'utilisateur, ce n'est pas le fait d'appeler trois malheureuses fonctions au lieux d'une seule qui fera une différence