Discussion :

[tlili_info]

Bonjour,
j'essaye de concaténer 3 fichiers (2 de dossier res/raw et une identifié par une URL) ...
voilà mon code

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
 
final int promptId =
                Class.forName(R.class.getName() + "$raw").getDeclaredField("raw1").getInt(null);
            final AssetInputStream prompt =
                (AssetInputStream) VVMApplication.getContext().getResources().openRawResource(promptId);
 
            final int promptId3 =
                            Class.forName(R.class.getName() + "$raw").getDeclaredField("raw2").getInt(null);
                        final AssetInputStream prompt3 =
                            (AssetInputStream) VVMApplication.getContext().getResources().openRawResource(promptId3);           
            final FileOutputStream fos = new FileOutputStream(PROMPT_TEMP_FILE, false);
            final File file = new File(fileUri.getPath());
            final int length = (int) (file.length() + prompt.available()+prompt3.available());
            final byte[] buffer = new byte[length];
            FileInputStream fis = new FileInputStream(file);
            fis.read(buffer);
            prompt.read(buffer);
            prompt3.skip(6);
            prompt3.read(buffer, (int) file.length()+prompt.available(), prompt3.available());
            fos.write(buffer);
            fos.close();
            fis.close();
            return Uri.fromFile(PROMPT_TEMP_FILE);

Les 3 fichiers sont concaténer mais le problème que la 2eme fichier est coupé , on entend juste la 2eme moitié de fichier....

Je vais les concaténer dans un fichier FileOutputStream sous la forme file de raw1 + File + file de raw2

Solution ?

Merci.

[nicroman]

Pourquoi passer par un buffer intermédiaire dont on ne sait pas la taille à l'avance (surtout si l'InputStream n'est pas entièrement lu).

Pour rappel: available() n'indique pas la taille du stream, mais la quantité de "bytes" disponibles à la lecture à l'instant T....


Après, il faut faire une petite fonction "utilitaire":

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
 
private static final int BUFFER_SIZE = 2048;
public static int  concatenateStreams(OutputStream out, InputStream ... ins)
{
    int total = 0;
    byte[] buffer = new byte[BUFFER_SIZE];
    for (InputStream in : ins) {
       int avail;
       while ((avail = in.read(buffer)) > 0) {
          out.write(buffer,0,avail);
          total += avail;
       }
    }
    return total;
}

Avec un appel du genre:

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
 
{
     // on ouvre les streams
     FileOutputStream out = xxxxx;
     AssetInputStream prompt1 = xxxxx;
     FileInputStream prompt2 = xxxx;
     AssetInputStream prompt3 = xxxxx;
 
     try {
         // on prépare les streams
         prompt3.skip(6);
         // on concatene
         concatenateStreams(out,prompt1,prompt2,prompt3);
     } finally {
         // on ferme les stream (toujours dans un finally !!!)
         out.close();
         prompt1.close();
         ...
     }
}

[tlili_info]

Salut
Merci pour votre reponse
Mais si la liste de InputStream contient plus que 1 InputStream ça fonctionne pas ...
Voilà mon code :

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
 
 public Uri buildPrompt(final Uri fileUri)
    {
 
        try {
            final InputStream prompt =
               VVMApplication.getContext().getResources().openRawResource(R.raw.operator_vs_prompt1);
          final InputStream prompt3 =
               VVMApplication.getContext().getResources().openRawResource(R.raw.operator_vs_prompt2);
            FileOutputStream fos = new FileOutputStream(PROMPT_TEMP_FILE, false);
            final File file = new File(fileUri.getPath());
 
            InputStream fis = new FileInputStream(file);
            concatenateStreams(fos,prompt3);
 
            fos.close();
            fis.close();
            return Uri.fromFile(Mon Fichier);
        } catch (final Exception e) {
            return fileUri;
        }
 
public static int  concatenateStreams( FileOutputStream out, InputStream ... ins)
    {
        int total = 0;
        byte[] buffer = new byte[BUFFER_SIZE];
        try {
        for (InputStream in : ins) {
           int avail;
           while ((avail = in.read(buffer)) > 0) {
              out.write(buffer,0,avail);
              total += avail;
           }
        }
        } catch (Exception e){
        }
        return total;
    }

[tchize_]

Tu ne fais rien de ton fis, son contenu ne risque pas d'arriver dans la destination...

[tlili_info]

je n'ai pas compris votre commentaire
tu peux me donner plus de détails

[nicroman]

Dans votre code:

Code :

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

1
2
3
 
InputStream fis = new FileInputStream(file);
concatenateStreams(fos,prompt3);

Il faudrait plutôt faire:

Code :

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

1
2
3
 
InputStream fis = new FileInputStream(file);
concatenateStreams(fos,prompt,fils,prompt3);

Et la gestion des exceptions est bizarre... votre version de concatenateStreams catch une exception mais n'en fait rien ... pourquoi ?
De même... la gestion des ressources ne marche pas dans buildPrompts:
Y a bien un try() mais dans le cas d'une exception les ressources ne sont pas fermées... (j'avais bien dit dans mon message qu'il faut fermer les ressources dans le "finally" du try).