Discussion :

[Maiyar]

Bonjour,

Je voudrai qu'on m'éclair sur ce code , précisément je voudrais repérer les erreurs dans ce code SVP :
import java.io.FileNotFoundException;
import java.net.Socket;

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
public class Imap {
	public static void main(String[] args) {
		new Thread(new Runnables() {
			public static void start() {
				ServerSocket ss = new socket(1234);
				while (true) {
					Socket s = ss.Listen();
					try {
						BufferReader os = new BufferReader(new InputSreamReader(s.getInputStream()));
						while((i = is.readline()) != null) {
							is.add(line);
						}
					} catch (IOexception ex) {
						;
					}
				}
			}
 
		}).run();
	}
try {
	FileOutputStream a = new fileOutPutStream("message.txt");
	PrintWriter os = new PrintWriter(a,true);
	While (true){
		for(string x : is) {
			for(int b = mi; b < fa; b++) {
				try {
					socket s = new socket(x, b);
					os.println(x +b);
				}catch (IOException ex) {;}
			}
		} is.clear();}
	} catch (InterruptedException ) | FileNotFoundException ex) {ex.printStackTrace();
} 
}}

Bon fin week end

[professeur shadoko]

Ben tu compiles et tu vois toutes les fotes d'aurtografe! (majuscules, pluriel intempestif, etc.)

[Maiyar]

oui mais je parlais de la logique du codes .. je viens de voir que y'avait des erreurs de majuscules justement voici la correction effectuée:

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
45
46
47
48
49
50
51
52
53
54
55
package java;
 
import java.io.BufferedReader;
import java.io.FileNotFoundException;
import java.io.FileOutputStream;
import java.io.IOException;
import java.io.InputStreamReader;
import java.io.PrintWriter;
import java.net.ServerSocket;
import java.net.Socket;
 
public class Imp {
	public static void main(String[] args) {
		String i;
		new Thread(new Runnable() {
			public void start() {
				ServerSocket ss = new ServerSocket(1234);
				while (true) {
					Socket s = ss.accept();
					try {
						BufferedReader os = new BufferedReader(new InputStreamReader(s.getInputStream()));
						while((i = is.readLine()) != null) {
							is.add(line);
						}
					} catch (IOException ex) {
						;
					}
				}
			}
 
			@Override
			public void run() {
				// TODO Auto-generated method stub
 
			}
 
		}).run();
 
try {
	FileOutputStream a = new FileOutputStream("message.txt");
	PrintWriter is = new PrintWriter(a,true);
	while (true){
		for(String x : is) {
			int fa;
			int mi;
			for(int b = mi; b < fa; b++) {
				try {
					Socket s = new Socket(x, b);
					os.println(x +b);
				}catch (IOException ex) {;}
			}
		} is.clear(); }
	} catch (InterruptedException | FileNotFoundException ex) {ex.printStackTrace();
} 
}}

[PhilippeGibault]

Ce code a plein de problèmes, car il ne respecte pas un tas de bonne pratique à appliquer.

Quelques exemples de ce qu'il ne faut pas faire:
- Toujours fermer les flux (BufferReader,FileOutputStream,PrintWriter). A partir de Java 7, le faire avec un try-ressources.

Code :

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

try(FileOutputStream a = new fileOutPutStream("message.txt");PrintWriter os = new PrintWriter(a,true)){

-Toujours traiter une exception. Pour du code test (et seulement du code test), par exemple:

Code :

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

catch (IOException ex) {ex.printStackTrace();}

Après, les Thread, c'est un peu compliqué.
Tu trouvera des indications ici:
https://jmdoudoux.developpez.com/cou...ds.php#threads

En général, il faut éviter la classe Thread et préférer le framework Executor (https://jmdoudoux.developpez.com/cou...r.php#executor).
Je te recommande de faire une classe implémentant Runnable (Au moins pour la lisibilité du code) et la lancer par un Executor.

Par ailleurs, dans l'interface Runnable, c'est la méthode run() qu'il faut surcharger (comme pour Thread d'ailleurs).

Enfin, il faut éviter ça:

Code :

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

while (true) {

[thelvin]

Hello,

oui enfin, plus important, ça compile pas.

Code :

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

for(String x : is) {

"is" est de type PrintStream. Quelqu'un m'explique ce que ça veut dire, itérer sur un PrintStream ? Cette ligne n'a aucun sens.

Code :

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

os.println(x +b);

... Et c'est quoi, ça, "os" ?

On n'arrivera à rien en passant des heures à essayer de deviner ce que ce programme est censé faire.

Il faut apprendre comment fonctionnent les choses, d'abord, et construire ses propres programmes avec ce qu'on a appris, ensuite. Là c'est des bouts de machin inventés et sans queue ni tête. Ca n'ira nulle part. Il faut effacer et recommencer du début, en ne faisant que des choses qu'on a apprises.