Discussion :

[pinocchio]

Bonjour,
Je récupère un code qui met du temps à s'exécuter et je cherche à le rendre plus rapide. Mon problème est que je connais pas le VBA.
En lisant, la FAQ, il est montré comment parcourrir une plage de manière plus rapide en évitant l'énumération .
Mon souci est que dans le code, dans le parcours des cellules, il est utilisé et la valeur et la cellule. Y'a-t-il un moyen de travailler sur la valeur et la cellule en utilisant la méthode pré décrite.
Voici un petit bout de code

Code :

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

1
2
3
4
 
                For Each point_cell In cdp.Cells
                    If point_cell.Value <> " " Then
                        If IsInRange(Range(Plage.item(1)), point_cell) Then

Comme écrit dans le code précédent, il est utilisé point_cell et point_cell.Value
Cordialement
Pinocchio

[Jean-Philippe André]

bonjour,
tu peux éviter le for each déjà, mais peux-tu juste nous donner la fin du code (en virant l'inutile ou le confidentiel) pour qu'on sache ce que tu testes par la suite dans ton code

[pinocchio]

Bonjour,
Pour commencer, excusez-moi du retard mais n'ayant plus de cnx internet chez moi et ne travaillant pas vendredi, je n'ai pas pu vous répondre plus tôt.

tu peux éviter le for each déjà

Comment svp, vu que c'est une boucle de l'ensemble des cellules?

tu peux expliquer ce que tu essaies de faire ?
Je ne comprends pas ton code et n'ai pas d'aide sur InIsRange
CDP est le nom "informatique" d'une feuille ? d'une variable Range ?
Ta logique n'échappe : tu dis que tu ne connais pas le VBA, mais tu demandes d'adapter un bout de code pour qu'il aille plus vite.
Ne serait-il pas plus logique d'exposer le problème précisant éventuellement comment, toi, tu penses le résoudre ?

Pour répondre aux différentes questions.
1) Pardon, je remets plus de code car en effet pas forcément explicite :

Code :

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

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
        For Each comtextplage In Comment_Plage 'Comment_Plage est une collection de plage de cellule contenant des valeurs
 
            Set cdp = Range(Commentaire_Data_Plage.item(complage_index))
 
                c_index = 0
                Dim comment As LROComment   'LROComment est un type de commentaire en Essbase
                Dim ciblecol As Collection
 
                For Each point_cell In cdp.Cells 'Je parcours la liste des cellules de la plage en cours du For each
                    c_index = c_index + 1
                    If point_cell.Value <> " " Then 'Je vérifie si la valeur n'est pas un espace (valeur fixé comme telle)
 
                        Set ciblecol = New Collection
                        Set comment = New LROComment
                        Set ctp = Range(comtextplage) 'Je récupère la plage
                        If IsInRange(Range(Plage.item(1)), point_cell) Then 'IsInRange permet de vérifier si point_cell fait partie de la plage Plage.item(1)
                            Call GetDataPoint(Plage.item(1), point_cell, ciblecol) 'Récupère la valeur nécessaire pour essbase
                            comment_text = ctp.Cells.Value
                            Call comment.SetupComment()
                            If comment.comment_text <> "" Then
                                Commentaire.Add item:=comment
                            End If
                       End If
                    End If
                Next point_cell
                complage_index = complage_index + 1
          Next comtextplage

Les explications sont donc dans les commentaires ajoutés. Je parcours une plage de cellule pour ajouter des commentaires.
2)CDP est une plage de données.
3) Oui, désolé je ne connais pas le langage VBA, je suis développeur c# et Administrateur ORACLE. Je suis sur une application ou actuellement, je n'ai pas vraiment de travail, je cherche donc à améliorer les endroite où il y'a des lenteurs. En regardant la FAQ VBA, j'ai vu ce point et j'essai donc de m'en sortir avec mes moyens et avec vos aides.

Merci
Pinocchio

[Gorfael]

Salut à tous.
pinocchio, je crois qu'on ne s'est pas compris : je ne voulais pas une partie plus importante du code. Je voulais tout le code. Et pas ton code commenté, mais le vrai but de la macro.
On est pas à essayer de trouver où t'as fait une erreur. On essaie d'améliorer sensiblement le traitement, c'est pas pareil

Un exemple à la con :
tu testes chaque cellule de la plage cdp en admettant qu'il y en a 30000 et qu'il y en ai une unique qui soit dans Plage.item(1) (si j'ai bien compris IsInRange : tu as donc 30000 Tests dont 29999 faux et 1 Vrai

Dans ce cas, moi, j'utiliserais plutôt :
For Each comtextplage In intersect(Comment_Plage,Range(Plage.item(1)))
qui ne contiendrais qu'une seule cellule (avec un test de validité de l'intersection en cas d'inexistence).

Tu te dis programmeur : Moi, de mon temps, la première chose qu'on apprenai, c'est qu'une déclaration de variable est en tête de programme. Et dans tous les cas, elle n'est jamais dans une boucle !

Tu utilises une routine appellée par Call, avec un test derrière. Vraisemblablement, comment.SetupComment crée une string que tu mets en commentaire si elle n'est pas vide.
Pourquoi avoir créer un sub au lieu d'une function ? Je pense que ce serait moins gourmand en temps, mais sans garantie.

Quand une macro est lente, il y a toujours deux hypothèses :
- soit le traitement peux être amélioré
- soit le nombre de traitements peut être dimi,ué

Tu as une boucle imbriquée dans une autre boucle. Regardes si tu ne peux pas faire quelque chose de ce coté là
un appel de sous-programme est toujours bouffeur de temps. J'ai vu des programmes avec 10 fois la même séquence ; C'était plus rapide que 10 appels à sous-programme.

A+

[Gorfael]

Salut
tu peux expliquer ce que tu essaies de faire ?
Je ne comprends pas ton code et n'ai pas d'aide sur InIsRange
CDP est le nom "informatique" d'une feuille ? d'une variable Range ?

Ta logique n'échappe : tu dis que tu ne connais pas le VBA, mais tu demandes d'adapter un bout de code pour qu'il aille plus vite.

Ne serait-il pas plus logique d'exposer le problème précisant éventuellement comment, toi, tu penses le résoudre ?

A+

[patbou]

bonjour,

en plus de ta fonction inconnue "isinrange"
qqchose m intrigue

For Each point_cell In cdp.Cells

ici on boucle sur toutes les cellules de la feuille cdp
je pense qu il faut boucler sur les cellules non vide uniquement, le gain de temps sera nettement amelioré.

si effectivement tte la feuille est remplie , ce qui me semble tres improbable
alors la il faudra passer par un tableau ou un variant.

[Gorfael]

Salut
Pas sur les non-vides, puiqu'il est obligé de tester la cellule et ça ne change rien, mais sur UsedRange.cells, les cellules de la plage d'utilisation.
Mais j'attends de connaître le but pour proposer
A+