Amélioration d'un algorithme

Version imprimable

Salut,

J'ai un petit problème d'algorithme qui me plait pas trop, même si ça marche :

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
 
        'On sait que le debut du fichier audio commence par "RIFF"
        For i = 0 To byteCount - 1
 
            'On recherche l'en-tête du fichier WAV (RIFF), si trouvé, on décode le fichier wav
            If requestBin(i) = retour_ligne(0) Then
                If requestBin(i + 1) = retour_ligne(1) Then
                    If requestBin(i + 2) = retour_ligne(2) Then
                        If requestBin(i + 3) = retour_ligne(3) Then
                            FoundWav = True
                            Exit For
                        End If
                    End If
                End If
            End If
 
        Next

retour_ligne est définit comme ça :

Code:

Dim retour_ligne() As Byte = {Asc("R"), Asc("I"), Asc("F"), Asc("F")}

Il s'agit en fait de traiter l'en-tête d'un fichier wave.

Quelle solution plus "sympa" pourrait-on écrire ? Ici, ça va parce qu'il n'y a que 4 octets, mais si y'en avait plus...

Et le requestbin comme ça :

Code:

1
2
        Dim byteCount As Integer
        Dim requestBin As Byte()

Je récupère un fichier wav par le flux http en fait.

01/09/2008, 17h02
SaumonAgile

Ce n'est peut-être pas le plus élégant mais ça a le mérite d'être simple. Il vaut parfois mieux un code un peu moins élégant mais qui est plus facile à maintenir. ;)
01/09/2008, 20h48
nhervagault

Salut,

Pour la deuxieme question retour_ligne
un Encoding.GetBytes peut etre utiliser

Pour la premiere question

Array.BinarySearch peut t'aider.

Bon coding

Sinon plutôt que lire octet par octet, tu fais une méthode qui te renvoie un tableau de 4 octets (dans le cas présent) et tu compares ce tableau avec celui de référence. S'ils sont identiques c'est gagné.

Pour la comparaison, il suffit de parcourir les éléments
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 14 private bool CompareArray(byte[] b1, byte[] b2) { if (b1.Length != b2.Length) { return false; } for (int i = 0; i < b1.Length; i++) { if (b1[i] != b2[i]) { return false; } } return true; }
Désolé c'est du C#, mais ça reste compréhensible ^^