Envoi de fichier par socket

Version imprimable

2 pièce(s) jointe(s)

bnjr :P
j ai besoin d aide je veux envoyer des fichiers dans un réseau (utilisant les sockets)
mais j ai tester toutes les solutions et r1 ne fonctionne c ma derniere tentative et elle ne fonctionne pas :? :(
le code suivant est dans le tread de client lorsqu il reçoit un Objet de type FICH (FICH classe serialiser contient un attribut fe type File)

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
if (o instanceof FICH) {
                F = (FICH) o;
                try {
                    fi = F.FIN;
                    f=new FileInputStream(fi);
                    fo=new FileOutputStream(fc.c.num);
                    byte[] buf = new byte[1024];
                    f.read();
 
                   System.out.println("reçu  "+fi);
                int n;int i=1;
                    System.out.println("debut d ecriture"); 
                     while((n=f.read(buf))==1024){                        
                          fo.write(buf,0,n);
                          System.out.println("ecriture N° "+i);
                            i++;
        }
//        //et la on ajoute les bytes plus petits que 1024 ) la fin            
                    fo.write(buf,0,n);
                    System.out.println("la dernier ecriture");
                    f.close();
                    fo.close();
                } catch (Exception ex) {
                    System.out.println(ex + " erreur d ecriture");
                    ex.printStackTrace();
                }

c'est l'affichage dans la console du client

Pièce jointe 177939

et le fichier contient les byte mais il m affich ça

Pièce jointe 177940
svp aidez moi:roll::roll::roll::roll:
merci

17/05/2015, 13h04
tchize_

Code:

while((n=f.read(buf))==1024)

Le test correct est

Code:

while((n=f.read(buf))>=0)

on teste tant que la valeur de retour n'est pas négative. Seule une valeur de -1 indique une fin de fichier.

Ensuite sérialiser un objet de type File est inutile. Tout ce que ton code fait actuellement c'est copier une fichier d'un dossier du client vers un autre dossier du client. La seule chose qui est transférée entre le serveur et le client, c'est le nom du fichier. File ne contient pas le contenu du fichier, juste son chemin d'accès. Si tu veux transférer un fichier du serveur vers les clients, tu lit les bytes coté serveur, tu écrit ces bytes sur la socket, tu relit ces bytes coté client et tu les écrit dans le fichier d'arrivée.

PS: un effort pour nommer correctement tes variables dans ton code en respectant les conventions java, et un autre effort pour écrire autre chose que du language SMS dans tes questions, ce ne serait pas du luxe.
17/05/2015, 13h21
houdacat

merci
j'ai essai de faire cette solution de lire les byte puis les écrire dans la socket coté serveur et les reçu du coté client puis les écrire dans le fichier d'arrivée mais
le thread de client reçoit des objets donc cette solution ma générer une exception

et a propos le code du test il ne marche pas
17/05/2015, 13h35
tchize_

Le code du test marche très bien, c'est le reste de ton code qui fait n'importe quoi. Poste nous ton serveur et ton client corrigés pour qu'on vois tes erreurs.

j'ai essayé cette solution est elle fonctionne

Code:

1
2
3
4
5
6
7
8
9
public class CONTENU implements Serializable{
 
    int compteur;
    Vector tabbit=new Vector();
    CONTENU(){
 
}
 
}

coté serveur : (lorsqu'il reçoit une demande de telecharegement )

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
//////la reception du nom de fichier voulez le telecharger
                   if (o instanceof fichier) {
                     f = (fichier) o;
                try {
                    fis=new FileInputStream(f.chemin);
                       buf = new byte[1024];
                       int n = 0;
                       int compteur=0;
                       FCONTENU=new CONTENU();
                        while ((n = fis.read(buf)) >= 0) {
                            FCONTENU.tabbit.add(buf);
                            FCONTENU.compteur=compteur;
                            compteur++;
                            buf = new byte[1024];
 
                        }
                        System.out.println("terminée !");
                        fs.gc.sortie.writeObject(FCONTENU);
                        System.out.println("ENVOYEE !"); 
                        fis.close();
                } catch (Exception ex) {
                    System.out.println(ex+" erreur d envoi ");
                    ex.printStackTrace();
                }
 
                }

coté client : (la reception du fichier)

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
 if (o instanceof  CONTENU) {
                FCONTENU =(CONTENU)o;
                 System.out.println("reçu !");
                try {
                    fos=new FileOutputStream(fc.c.num);
                   for(int i=0;i<FCONTENU.tabbit.size();i++){
                               buf  = (byte[]) FCONTENU.tabbit.elementAt(i);
                               fos.write(buf);
 
                   } 
                    System.out.println("terminée !");
                   fos.close();
                } catch (Exception ex) {
                    System.out.println(ex + " erreur d ecriture");
                    ex.printStackTrace();
                }
            }

18/05/2015, 23h30
thelvin
Ça marche pas si bien que ça.
- Essaie avec des fichiers textes, tu verras qu'ils se retrouvent avec des trucs tout bizarres à la fin.
  
  C'est bien d'avoir une variable n dans while ((n = fis.read(buf)) >= 0) { mais tu dois aussi t'en servir, sinon c'est comme si elle était pas là.
- Essaie avec des fichiers de 500Mo. Ça fait mal, hein ?
  
  C'est parce que tu le charges entièrement en mémoire. Ce n'est pas malin de faire ça avec des fichiers, il faut les envoyer par bouts.
  Par exemple rien ne t'oblige à faire en sorte qu'un fichier soit envoyé en entier sous forme d'un seul objet. Tu pourrais avoir un objet "là j'envoie un fichier" puis plein d'objet "un bout du fichier" puis un objet "le fichier est fini".
19/05/2015, 00h26
tchize_

Ouais, mais non, même en envoyant bout par bout, avec un OnbjetOutputStream, ça va faire mal à la mémoire car ObjectOutputStream et ObjhectInputStream gardent un cache de tous les objets qui passent par là. Ce n'est tout simplement pas fait pour envoyer des fichiers. Point. Si tu veux envoyer un fichier tu prend la socket, tu prend l'outputstream de la socket et t'envoie le tout là dedans, sans créer un ObjectOutputStream qui sert à rien.

Pour lire un fichier entièrement:

http://java.developpez.com/faq/java/...FICHIER_copier

Pour créer un socket, tu as visiblement déjà le code. Il te faut juste supprimer l'objetOutputStream qui se trouve dessus, et écrire toi même les données dans la socket. Eventuellement, la classe DataOutputStream / DataInputStream t'aidera à standardiser l'envoi des chaine de caractère et nombres faisant partie de ton protocole.