XL 2019 Optimisation de code VBA

[mmaiga]

Bonjour,

Je viens vers vous pour pouvoir m'aider concernant l'optimisation de ce code. le code fonctionne mais le temps de l'execution n'est pas à mon goût ( 61 secondes).
Pouvez-vous m'aider à améliorer ce code ?

Je vous joins le fichier Excel. et j'aimerais que les données de la colonne E soient affichées.
Etant donné que le fichier est lourd, j'ai supprimé des données.

Code

Option Explicit

Sub Calculer_test()

' Tranfert pour TEST
Sheets("Feuil1").Range("A1:F13842") = Sheets("Feuil2").Range("A1:F13842").Value
' --------------------------------------------

' Options de calcul

Dim CalcMode As Long
CalcMode = Application.Calculation
Application.ScreenUpdating = False
Application.Calculation = xlCalculationManual
Application.EnableEvents = False
Application.DisplayStatusBar = False

'---------------------------------------------------

Dim T0: T0 = Timer
Dim lastRow As Long, lat1 As Double, lat2 As Double, lon1 As Double, lon2 As Double, R As Double
Dim dLat As Double, dLon As Double, a As Double, c As Double, d As Double, i As Long, j As Long, k As Long, doublon As Boolean
Dim tablo, T, Cte ' Tableaux de transfert, T étant le résultat
R = 6371 ' Rayon de la terre en km
Cte = Application.Pi / 180 ' PI/180, constante calculée une seule fois au début
lastRow = Range("B" & Rows.Count).End(xlUp).Row
[E:E].ClearContents
tablo = Range("A1:E" & lastRow)
ReDim T(1 To UBound(tablo)): T(1) = "Doublons"
For i = 2 To UBound(tablo)
'Application.StatusBar = "Ligne traitée : " & i & " sur " & lastRow ' A valider si on veut le suivi dans le statusbar
lat1 = tablo(i, 2) * Cte 'conversion de degrés à radians
lon1 = tablo(i, 3) * Cte
doublon = False 'initialisation du drapeau de doublon
For j = i + 1 To UBound(tablo)
If i <> j Then 'ne compare pas avec soi-même
lat2 = tablo(j, 2) * Cte
lon2 = tablo(j, 3) * Cte
dLat = lat2 - lat1
dLon = lon2 - lon1
If Abs(lat1 - lat2) / Cte < 0.1 Or Abs(lon1 - lon2) / Cte < 0.1 Then
a = Sin(dLat / 2) * Sin(dLat / 2) + Cos(lat1) * Cos(lat2) * Sin(dLon / 2) * Sin(dLon / 2)
d = 2 * R * Application.Asin(Sqr(a))
If d < 10 And tablo(i, 4) = tablo(j, 4) And tablo(i, 5) = tablo(j, 5) Then 'enregistre si la distance est inférieure à 10 km et s'il y a un doublon dans la colonne D
'T(i) = T(i) & " " & tablo(j, 1)
T(i) = T(i) & " " & tablo(j, 1) & "(" & j & ": " & Format(d, "0.00") & " km)"
doublon = True 'marque le drapeau de doublon
End If
End If
End If
Next j
If Not doublon Then T(i) = "Aucun doublon trouvé" 'si aucun doublon n'est trouvé, enregistre "Aucun doublon trouvé"
Next i
[F1].Resize(UBound(T), 1).Value = Application.Transpose(T)
Application.StatusBar = ""


'---------------------------------

Application.Calculation = CalcMode
Application.ScreenUpdating = True
Application.EnableEvents = True
Application.DisplayStatusBar = True


[K2] = Timer - T0
End Sub





Merci d'avance..

 

[sylvanu]

@vgendron
Déjà qu'il ne lit pas ce qu'on écrit, ce n'est pas pour répondre aux questions. 🤣
Vous avez une liste de points définis par leur latitude et longitude.
Est déclaré "Doublon" un point qui se trouve à moins de 10km d'un, ou d'autres points.
Mais uniquement en descendant comme le dit Fanch.

@mmaiga

mais j'aimerais juste améliorer la performance

Vous vouliez dire " j'aimerais juste que vous amélioriez la performance" ?
En PJ, un peu mieux en calculant une seule fois latitude et longitude en degrés, ce qui évite pas mal de calculs ensuite.
Et en supprimant le "If i <> j Then" comme déjà dit puisque j=i+1.
Les résultat sur mon PC :

 

[cp4]

je n'ai toujours pas compris ce qu'était un doublon....

Moi aussi... On perd notre temps pour rien. On aurait dû aider d'autres demandeurs.

 

[vgendron]

Moi aussi... On perd notre temps pour rien. On aurait dû aider d'autres demandeurs.

Pas grave.. je suis parti à faire mon montage video avec Tortues et baracuda 🙂

 

[mmaiga]

Moi aussi... On perd notre temps pour rien. On aurait dû aider d'autres demandeurs.

@vgendron
Déjà qu'il ne lit pas ce qu'on écrit, ce n'est pas pour répondre aux questions. 🤣
Vous avez une liste de points définis par leur latitude et longitude.
Est déclaré "Doublon" un point qui se trouve à moins de 10km d'un, ou d'autres points.
Mais uniquement en descendant comme le dit Fanch.

@mmaiga

Vous vouliez dire " j'aimerais juste que vous amélioriez la performance" ?
En PJ, un peu mieux en calculant une seule fois latitude et longitude en degrés, ce qui évite pas mal de calculs ensuite.
Et en supprimant le "If i <> j Then" comme déjà dit puisque j=i+1.
Les résultat sur mon PC :
Regarde la pièce jointe 1168264

Merci ça me va. Merci également à vos efforts, désolé pour les personnes qui n'ont pas compris le concept, c'était pas dans mon intention de perdre votre temps.

Je m'excuse...