Concaténation AssetInputStream+FileInputStream + AssetInputStream

Version imprimable

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

Code:

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.

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:

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:

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

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

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

03/04/2015, 18h50
tchize_

Tu ne fais rien de ton fis, son contenu ne risque pas d'arriver dans la destination...
06/04/2015, 11h04
tlili_info

je n'ai pas compris votre commentaire
tu peux me donner plus de détails :)
06/04/2015, 14h02
nicroman
Dans votre code:
Code:

1 2 3 InputStream fis = new FileInputStream(file); concatenateStreams(fos,prompt3);
Il faudrait plutôt faire:
Code:

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).

Citation:

Envoyé par nicroman

Dans votre code:

Code:

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

Il faudrait plutôt faire:

Code:

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).

Salut,
j'ai essayé de regler mon code :

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
 
 
        InputStream fis = null;
        FileOutputStream fos = null;
        InputStream prompt = null;
        InputStream prompt3 = null;
        try {
 
            prompt = VVMApplication.getContext().getResources().openRawResource(R.raw.operator_vs_prompt1);
 
            prompt3 = VVMApplication.getContext().getResources().openRawResource(R.raw.operator_vs_prompt2);
            fos = new FileOutputStream(PROMPT_TEMP_FILE, true);
            final File file = new File(fileUri.getPath());
            fis = new FileInputStream(file);
            concatenateStreams(fos,prompt,fis,prompt3 );
 
        } catch (Exception e) {
            return Uri.fromFile(PROMPT_TEMP_FILE);
        }
        finally {
            try {
                fos.close();
                fis.close();
                prompt.close();
                prompt3.close(); 
            }
            catch (final Exception e){
                Log.i(TAG, "exception close " + e.getMessage());
 
            }
          return Uri.fromFile(PROMPT_TEMP_FILE);
    }

et la 2eme méhode :

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
 
public static int concatenateStreams(FileOutputStream out, InputStream... ins)
    {
        int total = 0;
        byte[] buffer = new byte[BUFFER_SIZE];
        try {
            for (InputStream in : ins) {
                int avail;
                in.read(buffer);
                while ((avail = in.read(buffer)) > 0) {
                    out.write(buffer, 0, avail);
                    total += avail;
 
                }
            }
        } catch (Exception e) {
            Log.i(TAG, " exception " + e.getMessage());
        }
        return total;
    }

Mais toujours ça fonctionne pas :(

08/04/2015, 18h12
tchize_

"fonctionne pas" n'est pas informaticien. Qu'est-ce qui se passe? Tu as des messages d'erreur? Le fichier de sortie est vide? Le contenu n'est pas correct?
09/04/2015, 17h31
tlili_info

Le fichier de sortie est vide

Pour la deuxieme méthode il vaut mieux faire:
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 public static int concatenateStreams(FileOutputStream out, InputStream... ins) throws IOException { int total = 0; byte[] buffer = new byte[BUFFER_SIZE]; for (InputStream in : ins) { int avail; in.read(buffer); while ((avail = in.read(buffer)) > 0) { out.write(buffer, 0, avail); total += avail; } } return total; }
Et c'est dans la seconde qu'il faudrait logger l'exception...
Et surtout pas avec:

Code:

Log.i(TAG, " exception " + e.getMessage());

qui perd tout un tas d'information mais par:

Code:

Log.e(TAG, "Echec écriture du fichier",e);

qui non seulement affichera l'exception (getMessage() peut être vide), mais aussi toute la stack au moment ou celle-ci a eu lieu.