Discussion :

[comateen]

Bonjour à tous,


dans un petit logiciel que je programme l'utilisateur peut choisir un fichier via un jchoosefile, le chemin d'accès est stockée dans une variable "newDoc"

cependant lors de la commande :

Code :

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

Runtime.getRuntime().exec("cmd /C start " + newDoc);

j'ai pu constater que si le nom du fichier contenant un espace, par exemple : Annexe 1.docx (ou pdf, doc, xls, xlsx), document plein de blabla, la commande plante.

j'ai testé :

Code :

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

Runtime.getRuntime().exec("cmd /C start " + "\"" + newDoc + "\"");

ça ne marche pas, ça provoque juste l'ouverture de la fenêtre cmd dos mais ça n'ouvre pas le document.

Une idée de comment je peux résoudre ce souci?

Merci à tous

[joel.drigo]

Salut,

Le premier paramètre entre guillemets de la commande start est le titre de la fenêtre. Tu dois voir que la fenêtre qui s'ouvre a pour titre le nom de ton fichier. Il faut donc insérer entre start et le nom de ton fichier entre guillemet, un titre en guillemet (au minimum "").

Il est plus simple de gérer ce genre d'aspect en utilisant la méthode qui utilise un tableau en paramètre :

Code :

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

1
2
3
 
String[] command = {"cmd", "/C",  "start", "\"\"", newDoc};
Runtime.getRuntime().exec(command);

Ici, peu import que newDoc contienne des espaces ou non : ce sera géré automatiquement.

Remarque que

Code :

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

1
2
3
 
String[] command = {"cmd", "/C",  newDoc};
Runtime.getRuntime().exec(command);

fonctionne aussi.


Par ailleurs, il est te sera plus simple de gérer des appels externe par la classe ProcessBuilder (au lieu de Runtime.getRuntime().exec()), notamment en ce qui concerne les traitements des flux.


[Edit]Effectivement comme le dit @adiGuba, pour ouvrir un doc, il vaut mieux utiliser Desktop.open()

[adiGuba]

Salut,


Déjà il faut éviter d'utiliser Runtime.getRuntime() et privilégier ProcessBuilder qui découpe explicitement les paramètres et surtout de mieux gérer les flux d'E/S.
(actuellement il est possible qu'un programme se bloque vu que tu ne traite pas les flux)
Ce qui donnera pour ta commande :

Code :

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

1
2
3
	new ProcessBuilder("cmd", "/C", "start", file.toString())
		.inheritIO() // utilise les flux d'entrée/sorties vers ceux du programme appelant
		.start(); // lance le process

Mais çà ne marche toujours pas : ton problème ne vient pas de Java mais de ta commande.
Même si tu la tapes directement en ligne de commande cela ne marche pas.

start attend en premier paramètre le nom de la fenêtre.
Normalement ce paramètre est optionnel mais le fait de ne pas le spécifier peut poser problème : on dirait que si le premier paramètre contient un espace il considère que c'est le titre de la fenêtre et non pas le nom de la commande à exécuter...

Il faut donc spécifier un titre :

Code :

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

1
2
3
	new ProcessBuilder("cmd", "/C", "start", " ", file.toString())
		.inheritIO()
		.start();

Mais dans tous les cas le meilleur moyen pour ouvrir un fichier reste d'utiliser la classe Desktop, et en plus c'est portable :

Code :

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

1
2
3
4
5
6
7
	public static void openDocument(File file) throws IOException {
		if (Desktop.isDesktopSupported() && Desktop.getDesktop().isSupported(Action.OPEN)) {
			Desktop.getDesktop().open(file);
		} else {
			throw new IOException("Desktop is not supported");
		}
	}

a++

[comateen]

merci beaucoup, me voilà donc un peu moins bête et je peux continuer le développement de mon outil de comparaison de document