Discussion :

[lemfi]

Bonjour,

j'utilise une servlet pour afficher un tableau d'images, au niveau du code HTML voici ce que j'ai :

Code :

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

1
2
3
4
5
 
<img src="/context/image.do?path=\Photo 001.jpg" />
<img src="/context/image.do?path=\Photo 002.jpg" />
<img src="/context/image.do?path=\Photo 003.jpg" />
...

ma servlet se contente de lire l'attribut path et d'afficher l'image correspondante, ci-dessous le code (en gros) :

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
 
try {
 
 File filename = null;
 String path = imageForm.getPath();
 filename = new File(path);    
 response.setContentType("application/octet-stream");
 response.setHeader("Content-Disposition", "inline; 
 OutputStream fout = response.getOutputStream();
 ImageStream is = ImageStream.getInstance();
 is.write(fout, filename);    
 
} catch (IOException e) {
 // gestion de lexception
}

Tout ceci marche SAUF que parfois, alors que le chemin est différent je vais avoir à l'affichage deux fois la même image. Je suppose qu'il y a un problème de synchronisme car cette servlet est appelée plusieurs fois très rapidement sur la même page mais je ne sais pas comment corriger cela.

Pouvez-vous m'aider ?

[lemfi]

Vraiment personne n'a d'idée ?

Comment puis-je faire pour que ma servlet soit "synchronisée", et ne soit pas appelée deux fois en même temps, car je pense que c'est ça le problème ...

J'envoie deux requêtes avec des paramètres différents et pourtant en réponse j'ai deux fois la même réponse, en l'occurence, la même image affichée deux fois.

[fabien29200]

Je ne connais pas ImageStream.
C'est une classe à toi ?

Sinon ça répond pas au problème, mais pourquoi ne pas directement appeler les images via leurs URL sans passer par une Servlet ?

[lemfi]

Salut,

ImageStream est effectivement une classe à moi, j'ai oublié d'en détailler le contenu, voilà ce que fait le write :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
 
  public synchronized void write(OutputStream fout, File filename) throws FileNotFoundException, IOException {
        BufferedOutputStream bout = new BufferedOutputStream(fout);
 
        byte[] bytes = new byte[512];
        FileInputStream fileInputStream = new FileInputStream(filename);
        int i;
        while ( (i = fileInputStream.read(bytes)) > 0) {   
            bout.write(bytes,0,i);
        }
        fileInputStream.close();
        bout.close();     
    }

En gros ça écrit dans la sortie
Si je mets ce bout de code directement dans ma servlet (ce qui serait mieux, je ne vois pas pourquoi j'ai créé cette classe elle ne sert à rien !!!) ça ne marche pas mieux.

Pour ce qui est de ta question de mettre directement l'url, je ne le fais pas c'est un souhait, mon site est un site de photos, les photos ne sont pas placées au même endroit que l'appli, dans la servlet je fournis un chemin relatif, c'est l'application qui gère le chemin complet, comme ça l'adresse du stockage des photos sur mon serveur n'est pas affichée ...

Le problème dont je parle ne survient pas quand j'attaque mon site à distance (web) mais uniquement quand je l'attaque depuis chez moi (je l'héberge chez moi).

[OButterlin]

A la louche, je dirais que ton problème vient justement de ta classe ImageStream.
Le ImageStream.getInstance() fait penser à une instance unique (singleton) et de ce fait, comme les servlets sont multi-thread, ça va poser problème.
Il faudrait déjà synchroniser ImageStream dans la servlet pour éviter que 2 thread ne s'écrasent mutuellement les références.
Quelque chose comme ça

Code :

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

1
2
3
4
5
6
7
8
 
...
ImageStream imageStream = ImageStream.getInstance();
synchronized (imageStream)
{
   ... le code 
}
...

A+