Discussion :

[anat1212]

Bonjour,

J'aimerais optimiser une de mes application pour le multithread,

Le soucis qui se pose c'est que (je n'ai pas essayé) si plusieurs thread exécutant un délégué afin de faire la mise à jour d'une datagridview

example de délégué :

Code :

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

1
2
3
4
5
6
 
    Public Sub ajout()
        Dim tableau As String()
        tableau = datas.Split(":")
        DataGridView1.Rows.Add(tableau)
    End Sub

Si cette procédure est appelée par plusieurs thread en même temps, je dois utiliser SyncLock non ?

Autrement j'aimerais savoir si c'est possible possible d'ajouter des éléments à une collection sans utiliser SyncLock

Je n'ai jamais utilisé SyncLock donc je ne sais pas si c'est ça que je dois utiliser pour pallier a mon éventuel problème ?

Merci

[Pol63]

oui synclock bloque les différents thread
le premier thread a passé à l'endroit du synclock franchit la barrière
tant qu'il n'est pas sorti (end synclock), tous les autres threads attendent devant la porte (le thread qui a eut le droit de passer peut repasser au meme endroit sans soucis)
une fois qu'il a fini, le thread suivant (le 2eme arrivé) rentre, les autres attendent encore

sur une collection, si un thread fait un for each pendant qu'un autre fait .Add, ca plante, donc le synclock est en théorie nécessaire
mais dans ce cas précis, un readerwriterlock serait plus approprié, voir meme le slimreaderwriterlock (nouveau sur le framework 3 ou 3.5)
ca gagne un peu de perf par rapport au synclock

[anat1212]

oui synclock bloque les différents thread
le premier thread a passé à l'endroit du synclock franchit la barrière
tant qu'il n'est pas sorti (end synclock), tous les autres threads attendent devant la porte (le thread qui a eut le droit de passer peut repasser au meme endroit sans soucis)
une fois qu'il a fini, le thread suivant (le 2eme arrivé) rentre, les autres attendent encore

sur une collection, si un thread fait un for each pendant qu'un autre fait .Add, ca plante, donc le synclock est en théorie nécessaire
mais dans ce cas précis, un readerwriterlock serait plus approprié, voir meme le slimreaderwriterlock (nouveau sur le framework 3 ou 3.5)
ca gagne un peu de perf par rapport au synclock

Merci pour les infos

[anat1212]

Il est dit sur msdn :

N'utilisez pas l'instruction SyncLock pour verrouiller des threads qui manipulent des contrôles ou des formulaires. Comme les méthodes des contrôles et formulaires rappellent parfois une procédure d'appel, vous risquez de créer sans vous en rendre compte un blocage ; dans cette situation, deux threads attendent chacun que l'autre annule le verrou, ce qui entraîne l'arrêt de l'application.

comment je dois faire alors ?

[Pol63]

suffit de pas verrouiller des morceaux de code qui utilisent des controles ... m'enfin en théorie on met des sub à part qu'on appelle via des délégués ...

tu es confronté au problème ?

[anat1212]

Désolé je n'avais encore rien essayé, maintenant j'ai effectivement fait des essais, synclock marche bien comme ceci

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
 
Dim lock As New Object
    Private Sub BackgroundWorker1_DoWork(ByVal sender As System.Object, ByVal e As System.ComponentModel.DoWorkEventArgs) Handles BackgroundWorker1.DoWork
        SyncLock lock
            Dim index As Integer = 0
            While Col(index).EstFait And index < Col.Count - 1 And index < Col.Count - 1
                index += 1
            End While
            Col(index).SetData(f.DataDL(tb1, Col(index).GetNumero, tb2, CheckBox1.Checked))
            Me.Invoke(DelegueAjouterligne)
        End SyncLock
    End Sub

J'ai comme ceci 5 backgroundworker

A noter que au lieu d'utiliser un délégué nous aurions pu utiliser l'évènement RunWorkerCompleted du backgroundworker dans ce cas

j'ai quand même un soucis quand j'essaye de faire ce traitement

Code :

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

1
2
3
4
5
6
7
8
9
 
Delegate Sub SimpleDelegate()
Dim DelegueMajDGV As New SimpleDelegate(AddressOf majDGV)
    Public Sub majDGV()
        DataGridView1.Rows.Clear()
        For Each unel As Ligne In Col
            DataGridView1.Rows.Add(unel.GetData)
        Next
    End Sub

J'essaye de l'appeler avec un délégué mais ca ne passe jamais, même dans un synclock, même en mettant mon code dans l'évènement terminé du BGW avec un synclock.

A noter que pour l'instant j'utilise un compteur et met a jour me.text à la place de la mise à jour de la datagridview, donc il semble que le problème soit le foreach ou l'accès à la list


(HS): j'ai remarqué un bug étrange que j'avais déjà remarqué avec des applications (que je pensais avant être un bug courant)
Admettons que dans mon interface graphique j'ai un bouton et au clique j'appelle ma procédure majDGV, l'interface ne répondra plus sauf si je fais clic droit sur le programme dans la barre des tâches ( a chaque clic le gui se mettra à jour)