Problème d'erreur code

Version imprimable

Bonjour

Dans ce code :

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
public class telechargement extends Activity {
 
	public void onCreate(Bundle savedInstanceState) {
        super.onCreate(savedInstanceState);
        setContentView(R.layout.telecharge);
        UrlHelper();
	}
	private void UrlHelper() {
		public void downloadFile(String urlStr){
			downloadFile(urlStr, null);
		}
 
	}
	private void downloadFile(String urlStr , File destFilePath)throws IOException, URISyntaxException {
		// TODO Auto-generated method stub
 
	}
 
}

Sur cette ligne public void downloadFile(String urlStr){
j'ai downloadFile et ) de surligné
le message d'erreur est le suivant

Code:

1
2
3
4
Multiple markers at this line
	- void is an invalid type for the variable 
	 downloadFile
	- Syntax error on token ")", ; expected

Pouvez-vous m'indiquer l'erreur d'écriture
Merci

Salut,

Pourquoi tu as deux méthodes imbriquer ?

Code:

1
2
3
4
5
6
7
 
private void UrlHelper() {
		public void downloadFile(String urlStr){
			downloadFile(urlStr, null);
		}
 
	}

C'est pas possible de faire ça, que je sache tu as pas le droit de déclarer comme ça une méthode dans une autre. Faut revoir ton code et comment tu appelle tes methodes.

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
 
public class telechargement extends Activity {
 
	public void onCreate(Bundle savedInstanceState) {
        super.onCreate(savedInstanceState);
        setContentView(R.layout.telecharge);
        UrlHelper();
	}
 
	private void UrlHelper() {
	    downloadFile("test");
	}
 
         public void downloadFile(String urlStr){
	    downloadFile(urlStr, null);
         }
 
	private void downloadFile(String urlStr , File destFilePath)throws IOException, URISyntaxException {
		// TODO Auto-generated method stub
 
	}
 
}

Ce code marchera mieux mais je sais pas s'il correspond à ce que tu veux faire.

10/02/2011, 17h30
_Xavier_

Code:

- Syntax error on token ")", ; expected

Une parenthèse ")" de trop.
11/02/2011, 11h24
Vanessa48

Bonjour

Nakilu je veux pouvoir au lancement de mon appli télécharger un fichier de mise à jour (comme dans la FAQ télécharger un fichier)

dans cette méthode

Code:

private void downloadFile(String urlStr , File destFilePath)throws IOException, URISyntaxException {

C'est pas comme cela ?

Bonjour Vanessa48

Si tu veux que ton appli télécharge un fichier au lancement, il faut créer une méthode qui télécharge ton fichier par exemple downloadFile et l’appeler dans la méthode OnCreate de ton application. C'est aussi simple que ça
Code:

1 2 3 4 5 6 7 8 9 10 11 12 @Override protected final void onCreate(Bundle savedInstanceState) { //Ton code downloadFile("str", dest); } private void downloadFile(String urlStr , File destFilePath)throws IOException, URISyntaxException { //Ton code de telechargement }
Voila , en espérant que ça t'aide :)

J'ai corrigé mon code comme suit :

Code:

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
public void run() {
    		//Téléchargement du fichier
    		public void getFile(Url u) throws IOException{
    			URL u = new URL("http://www.mysite.com/BDSQLite/mybase");
    			URLConnection uc = u.openConnection();
    			uc.setUseCaches(true);
    			int FileLenght = uc.getContentLength();
    			if (FileLenght == -1){
    				Toast.makeText(miseAjour.this, "Erreur de chargement", 1000).show();
    				{
    				try	
    				}
    				InputStream myInput = uc.getInputStream();
    				String outFileName="/data/data/com.van.myappli/mybase";
    				FileOutputStream myOutPut = new myOutPut(outFileName);
    				byte[] buff = new byte[1024];
    				int I = myInput.read(buff);
    				while(I>0)
    				{
    					myOutPut.write(buff, 0, I);
    					I = myInput.read(buff);
    				}
    				myOutPut.flush();
    				myOutPut.close();
    				{
    					catch(Exception e)
    				}
    				Toast.makeText(miseAjour.this, e.toString, 1000).show();
    			}
    		}
    		//Télécharement terminé 
    		handler.sendEmptyMessage(0);
    	}
       	private Handler handler = new Handler() {
     		public void handleMessage(Message msg) {
    			Toast.makeText(miseAjour.this, "Chargement terminé ...", 1000).show();
    			dialog.dismiss();
    		}
    	};
    }

Les caractères indiqués en rouge sont surlignés
je n'arrive pas à corriger
Pouvez-vous m'aider
Merci

11/02/2011, 15h33
nbenbourahla

C'est quoi ton erreur exactement ? Tu peux copier le message stp ?

Syntaxe incorrecte sur le try-catch:
Code:

1 2 3 4 5 6 7 try { //ici le code qui est susceptible de lancer une exception } catch(Exception ex){ //Le traitement de l'exception }
Corrige ton placement des accolades.

11/02/2011, 15h56
Vanessa48
Apres corrections j'ai

Code:

public void run() {
Code:

1 2 public void getFile(Url u) throws IOException{ URL u = new URL("http://www.mysite.com/BDSQLite/mybase");
Code:

FileOutputStream myOutPut = new myOutPut(outFileName);

Code:

handler.sendEmptyMessage(0);
11/02/2011, 16h07
Feanorin
Code:

1 2 public void getFile(Url u) throws IOException{ URL u = new URL("http://www.mysite.com/BDSQLite/mybase");
En premier lieu Url n'existe pas mais plutôt URL.
Après URL u ne peut pas être ici car la variable u existe déjà dans cette fonction puisque tu l'as dans la déclaration de celle-ci

Code:

FileOutputStream myOutPut = new myOutPut(outFileName);

La ce serait plutôt

Code:

FileOutputStream myOutPut = new FileOutputStream(outFileName);

Code:

handler.sendEmptyMessage(0);

La c'est parceque le compilateur est perdu , dû à la première erreur qu'il t'as signalé

Code:

public void run() {

puisque tu as un problème dans celle ci

Maintenant je te conseille de relire attentivement la doc de java que tu trouveras ici .
11/02/2011, 16h10
yaraco
Il y a l'air d'avoir pas mal d'erreurs de cohérence.

Par exemple pour l'url: tu donnes une url en paramètre à ta fonction, et tu re-déclares à l'intérieur une url avec le même nom.
Soit tu réutilises la même url, soit tu en déclares une autre.

Ensuite pour le myOutPut, myOutPut c'est le nom que tu donnes à ton FileOutputStream. Tu ne peux donc pas créer un nouveau myOutPut mais un FileOutputStream.

Ça serait donc quelque chose du genre:
Code:

1 2 FileOutputStream myOutPut = new FileOutputStream(outFileName);
Mais la question serait plutôt quel IDE utilises-tu?
Par exemple sous netbeans, l'erreur de l'URL aurait été indiquée par "u" is already defined in ...
Pour voir ça, il suffit de laisser la souris sur le mot souligné.

Cela t'aidera à résoudre les problèmes de syntaxe.

EDIT: Tient, Feanorin écrit plus vite que moi :)

J'ai corrigé mon code en tenant compte de vos remarques
mais j'ai toujours void run() {

u est bien comme cela ? : u = new URL("http://www.mysite.com/BDSQLite/mybase");

Code:

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
public void run() {
    		//Téléchargement du fichier
    		public void getFile(URL u) throws IOException{
    			u = new URL("http://www.mysite.com/BDSQLite/mybase");
    			URLConnection uc = u.openConnection();
    			uc.setUseCaches(true);
    			int FileLenght = uc.getContentLength();
    			if (FileLenght == -1){
    				Toast.makeText(miseAjour.this, "Erreur de chargement", 1000).show();
    				{
    				try	{
    				InputStream myInput = uc.getInputStream();
    				String outFileName="/data/data/com.van.myappli/mybase";
    				FileOutputStream myOutPut = new FileOutputStream(outFileName);
    				byte[] buff = new byte[1024];
    				int I = myInput.read(buff);
    				while(I>0) {
    					myOutPut.write(buff, 0, I);
    					I = myInput.read(buff);
    				}
    				myOutPut.flush();
    				myOutPut.close();
    				}
    				catch(Exception e){
 
    				Toast.makeText(miseAjour.this, e.toString(), 1000).show();
    			}
    		}
    		//Télécharement terminé 
    		handler.sendEmptyMessage(0);
    		}
    	};

11/02/2011, 16h43
yaraco

Pas regardé plus que ça, mais à première vue il y a d'abord un souci d'accolades. Par exemple tu en as une ouvrante au dessus du try qui n'a pas l'air de correspondre à grand chose.

Vérifie tes accolades ouvrantes/fermantes, les IDE colorient généralement les accolades reliées quand on passe le curseur dessus.
12/02/2011, 08h44
Vanessa48

Merci à tous pour votre aide
l'erreur est résolue