Re: [Tsp-devel] suggestion pour core.common.TspStreamReader etcore.common.TspSes sion en Java

[Eric.NOULARD]

Le principe me paraît bien.

Je n'ai pas le temps de regarder le détails d'implem' tout de suite
notamment pour comprendre pourquoi la méthode configure est
nécessaire.

A+
Eric

-----Original Message-----
From:   address@hidden on behalf of CHOQUET, Mathias
Sent:   Wed 5/11/2005 1:10 PM
To:     'address@hidden'
Cc:    
Subject:        [Tsp-devel] suggestion pour core.common.TspStreamReader etcore.common.TspSes sion en Java
Bonjours,
Je souhaiterais faire une modification à TSPStreamReader et TSPSession pour
les raisons suivantes :

La FIFO présente dans TSPSession bufferise les données qu'elle reçoit de
TSPDataInputStream. Or j'utilise une IHM qui bufferise déjà les données
qu'elle reçoit.
J'aimerai donc pouvoir rendre l'utilisation de la FIFO optionnelle pour
permettre aux utilisateurs de se brancher directement à TSPDataInputStream
en passant par TSPStreamReader.

Pour se faire, voici ma proposition :
Créer une classe abstraite : public abstract class TspStreamReader
implements Runnable

Cette classe abstraite aurait en plus une méthode configure :

public Configure(TspDataInputStream in, TspGroup[] groups) {
        this.in     = in;
        this.groups = groups;
        stop        = false;
    }

Le constructeur et la méthode « public void readGroup(int i) » serait à
implémenter par des classes étendant la classe abstraite TspStreamReader.

L'utilisation de la FIFO se ferait par défaut avec la classe
TSPFifoStreamReader.La méthode readGroup serait identique et le constructeur
serait le suivant :

public TspFifoStreamReader(TspSampleFifo fifo) {
        this.fifo   = fifo;  }

Du coup, l'utilisateur a le choix d'implémenter ou non sa propre classe
TSPStreamReader (par exemple totoStreamReader extends TspStreamReader).

Cette classe serait instancié par l'utilisateur et envoyée à la méthode
TSPSession.RequestSampleInit(totoStreamReader)  qui surchargerait la méthode
actuelle.

Si l'utilisateur ne l'implemente pas, il appelle normallement
TSPSession.RequestSampleInit() qui instancie TspFifoStreamReader et qui
appelle le configure.

Pour TSPStreamReader, les changements seraient :
_Créer une classe abstraite TSPStreamReader en laissant vide readGroup et le
constructeur
_Ajouter la méthode configure (décrite au-dessus).
_Créer une classe TSPFifoStreamReader qui implémente le constructeur et
readGroup.

Pour TSPSession, les changements seraient :
_Changer le nom du thread  « sampleFifoThread » en « sampleThread »
_Ajouter une méthode RequestSampleInit(TspStreamReader)
_Enlever l'attribut de classe "TspSampleFifo sampleFifo"