Classe Java - erreurs

Version imprimable

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:

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:oops:

31/03/2019, 20h11
professeur shadoko

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

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:

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();
} 
}}

01/04/2019, 17h26
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:

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:

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:

while (true) {
02/04/2019, 00h42
thelvin

Hello,

oui enfin, plus important, ça compile pas.

Code:

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:

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.