Perfomance verbessern
#1
Liebe Excel Pro's

Das folgende Makro funktioniert einwandfrei. Einzige Schwachstelle bis jetzt ist die Geschwindigkeit der Ausführung. Gibt es eventuell vielleicht ein Möglichkeit die Geschwindigkeit der For-Each Schlaufe zu verbessern? (In Anbetracht, dass am Ende nicht nur ein Sheet verglichen werden soll.) Benutze Excel 2010, 32bit.

Das Makro vergleicht jede Zelle eines Ranges mit einem anderen Range und kopiert die Formatierungen der gefundenen Zelle zurück zur vergleichenden Zelle. 

Code:
Sub farbenuebertragen()
   Dim rngZelle, rngfound As Range
   Application.ScreenUpdating = False
   For Each rngZelle In Range("S19:BC85")
       If Not IsEmpty(rngZelle) Then
           Set rngfound = Worksheets("Config").Range("A2:A100").Find(What:=rngZelle.Value, LookIn:=xlValues, LookAt:=xlWhole)
           If Not rngfound Is Nothing Then
               rngfound.Copy
               rngZelle.PasteSpecial Paste:=xlPasteFormats
           End If
       End If
   Next
   Application.ScreenUpdating = True
End Sub

Lieber Gruss

romanimtv
Top
#2
Hallo!
Über wie viele Formate in Worksheets("Config").Range("A2:A100") sprechen wir?
Ist bedingte Formatierung keine Option?

Gruß Ralf
Gib einem Mann einen Fisch und du ernährst ihn für einen Tag. 
Lehre einen Mann zu fischen und du ernährst ihn für sein Leben. (Konfuzius)
Top
#3
(02.09.2015, 16:04)RPP63 schrieb: Hallo!
Über wie viele Formate in Worksheets("Config").Range("A2:A100") sprechen wir?
Ist bedingte Formatierung keine Option?

Gruß Ralf

HAllo Raf

Vielen Dank für Deine Antwort. Die Lösung über bedingte Formate hatte ich bereits. Es sind zuviele. Jede Zelle von A2:A100 hat eine andere Farbe.

Lieber Gruss


romanimtv
Top
#4
99 Farben?  Huh
Wer soll die denn auseinander halten?
Aber gut, dies ist ja nicht mein Problem.
Man könnte einen Algorithmus entwickeln, der sich anhand des Zellwerts einen RGB-Wert bastelt.
Dies dürfte erheblich schneller laufen als die Kopieraktionen iVm Find.

Gruß Ralf
Gib einem Mann einen Fisch und du ernährst ihn für einen Tag. 
Lehre einen Mann zu fischen und du ernährst ihn für sein Leben. (Konfuzius)
Top
#5
Ach so:
Kopiere Dein Blatt "Config" mal in eine neue Datei und lade sie hier hoch.

Gruß Ralf
Gib einem Mann einen Fisch und du ernährst ihn für einen Tag. 
Lehre einen Mann zu fischen und du ernährst ihn für sein Leben. (Konfuzius)
Top
#6
Danke für eure Antworten.  Ich habe es nun mit Workbook_SheetChange gelöst. Statt alle Farben nur beim Speichern anzupassen, sondern bei jeder Zelländerung im intersect-Bereich. Damit konnte ich der langsamen For Each - Schlaufe entgehen und habe nun meine eigene «bedingte» Formatierung. Smile 

Vielen Dank euch allen. Immer wieder spannend hier. 

Lieber Gruss

romanimtv
Top


Gehe zu:


Benutzer, die gerade dieses Thema anschauen: 1 Gast/Gäste