Discussion :

[Ben.r]

Bonjour la communauté,
Je fais appel à vous car j'ai un problème avec des fichiers .dat.

Je cherche à effacer une ligne selectionnée par l'utilisateur dans un fichier .dat

voici le code de ma méthode :

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
34
35
36
37
38
39
40
41
42
43
44
 
public void effacerLigne(String nomDuFichier, String ligne){
                        FileInputStream fIn = null;
			InputStreamReader isr = null;
			BufferedReader br = null;
 
			FileOutputStream fOut = null;
			OutputStreamWriter osw = null;
 
 
			try {
				   fIn = c.openFileInput(nomDuFichier);
				   isr = new InputStreamReader(fIn);
				   br = new BufferedReader(isr);
				   fOut = c.openFileOutput("t" + nomDuFichier, Context.MODE_APPEND);
				   osw = new OutputStreamWriter(fOut);			
 
					String line;         
				while ((line = br.readLine()) != null) {
 
				        if (!line.equals(ligne)) { 
				        	osw.write(line + '\n');	
				        	osw.flush();
 
				        }
				        else if (line.equals(ligne)){
				        	Toast.makeText(c.getApplicationContext(), line + " " + ligne, Toast.LENGTH_LONG).show();
				        }
				}
				br.close();
				isr.close();
				fIn.close();
			    osw.close();
			    fOut.close();
 
 
		      File ancien = new File(nomDuFichier);
		      File nouveau = new File("t" + nomDuFichier);
		      ancien.delete();
		      nouveau.renameTo(ancien);
 
			}catch(Exception e) { 
				Toast.makeText(c.getApplicationContext(), e.toString(), Toast.LENGTH_LONG).show(); 
			}

Tout se passe bien sauf à la fin,
Je souhaite renommer le nouveau fichier crée avec la ligne en moins, par le nom de l'ancien, mais cela ne fonctionne pas.
Quelqu'un a t'il une idée?

[Simon MARQUIS]

As-tu essayé de ne pas faire appel à ancien.delete(); ?

[Ben.r]

Bonjour Simon,

Oui à la ligne 39

Et le renamTo ne change rien, car si je mets un toast derriere pour connaitre le nouveau nom du nouveau Fichier il s'appelle toujours "t" + quelquechose...
Je pense que le problème viens de l'instanciation des objets File...

[Simon MARQUIS]

Attention, j'ai bien dis : ne pas faire appel à ancien.delete();

[Ben.r]

Ah oui pardon!
Je viens d'essayer, même résultat...

[Simon MARQUIS]

Sinon, tu peux essayer d'une manière différente :
Tu copie le fichier 'Fichier' vers un nouveau fichier 'SauvegardeFichier'.
Tu écris dans le fichier 'Fichier' (ne pas utiliser le mode append).
Si quelque chose échoue, il te suffit de renommer 'SauvegardeFichier' en 'Fichier'.
Si tout est ok, il te reste à supprimer le fichier 'SauvegardeFichier'.

[nicroman]

Attention... File() est un faux amis... L'objet ne représente pas un fichier, mais le chemin d'un fichier....

Et entre openFileOutput(name) (j'imagine que "c" est un contexte), et new File(name) il y a une *grosse* différence....

Je te conseille de transformer ta fonction en un truc genre:

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
 
public void effacerLigne(File fichier, String ligne) {
 
    File tempFile = File.createTempFile("temp",null,fichier.getParent());
    boolean succeeded = false;
 
    try {
        src = new BufferedReader(new FileReader(fichier));
        dest = new FileWriter(tempFile);
 
        ... le code de remplacement ...
 
        succeeded = true;
    } finally {
        if (src != null) src.close();
        if (dest != null) dest.close();
    }
 
    if (succeeded) {
       fichier.delete();
       tempFile.renameTo(fichier);
    } else {
       tempFile.delete();
    }
}

Non seulement il marchera sur n'importe quel fichier (et pas seulement dans les fichiers "privés" du context), mais cela évitera de se trimbaler deux manières de référencer le fichier.

Ensuite il faut découvrir comment passer du filename local à un File:

Code :

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

1
2
3
4
5
 
public void effacerLigne(String filename, String ligne) {
    File localFile = new File(c.getFilesDir(),filename);
    effacerLigne(localFile,ligne);
}

Et le tour devrait être joué !

[Ben.r]

Bonjour Nicroman, et MERCI pour ta réponse!!!

Je me doutes bien que le problème viens de l'objet File...

Je vais essayer ça tout de suite!!

[Ben.r]

ça marche parfaitement !!!
Et c'est plutôt astucieux finalement.
Et merci pour les précisions sur l'objet File que j'ai du mal à bien cerner.

je rectifie juste un bout de code, à la ligne 4 :

File tempFile = File.createTempFile("temp",null,fichier.getParent());

c'est getParentFile() la méthode

Merci encore Nicroman