Fortschrittsbalken tut nicht

  • Fortschrittsbalken tut nicht

    Hallo,

    ich brauche mal wieder Eure Hilfe.
    Ich möchte während einer While-Schleife, in welcher eine Berechnung stattfindet, einen Fortschrittsbalken mittels NSProgressIndicator anzeigen.

    Folgender Aufbau:

    in der AppDelegate.h

    Quellcode

    1. NSProgressIndicator *fortschritt;
    3. @property (nonatomic, retain) IBOutlet NSProgressIndicator *fortschritt;


    in der AppDelegate.m

    Quellcode

    1. ...
    3. anzahl_tage = 0;
    4. // In piiinterval stehen die Anzahl der Tage drin. so oft wird die While-Schlife durchlaufen
    5. double piinterval = [ende timeIntervalSinceDate:beginn] / 86400;
    6. [fortschritt setMaxValue:piinterval];
    7. [fortschritt setMinValue:0.0];
    8. [fortschritt setHidden:FALSE];
    9. [fortschritt startAnimation: self];
    10. [fortschritt display];
    12. do {
    13. //irgendeine Berechnung
    14. anzahl_tage++;
    16. [fortschritt setDoubleValue:(double) anzahl_tage];
    17. [fortschritt display];
    18. while(abbruchbedingung);
    19. }
    20. [fortschritt stopAnimation: self];
    21. [fortschritt setHidden:TRUE];
    Alles anzeigen


    Der Fortschrittsbalken wird bei den ersten gesamten Durchläufen der Schelife gar nicht angezeigt. Bei jeden weiteren Durchläufen erst nach dem die Berechnungen abgeschlossen sind und dann auch nicht kontinuierlich, sondern jedesmal anders. Mal beginnt er in der Mitte, mal springt er sofort aufs Ende.

    Was mache ich falsch?

    Danke und Grüße
    Bernd
    Ich bin gegen Signaturen!!!

  • Die Fortschrittsanzeige kann natürlich nicht neu gezeichnet werden, solange Du den Hauptthread, der für das Zeichnen der Views zuständig ist, mit Deiner while-Schleife blockierst. Der Hauptthread sollte nienienie über längere Zeit (ein paar Millisekunden) blockiert werden.

    Das kannst Du so lösen wie thallius schrieb. Es gibt aber noch zig andere Möglchkeiten, z.B. über eine NSOperation. Such im Forum mal nach NSProgressIndicator, das Thema ist hier schon oft diskutiert worden.
    Multigrad - 360°-Produktfotografie für den Mac

  • Danke erstmal für Eure Antworten.

    Ich führe meine Berechnungen nach Buttonpress in - (IBAction)rechnen:(id)sender aus und gebe das Ergebnis in einer TableView aus. Jedesmal, wenn der Button gedrückt wird, startet eine neue Berechnung.

    Wie muss ich das jetzt umbauen? Ich schnalls nicht :(
    Ich bin gegen Signaturen!!!

  • nur weil ich gerade Langeweile habe

    Quellcode

    1. -(IBAction)rechnen:(id)sender
    2. {
    3. [self performSelectorInBackGround:@selector(calcInBG:) withObject:nil]; // Wenn Du Werte aus dieser Methode in der CalcInBG brauchst kannst du diese in ein NSDictionary packen und dieses als Object mitgeben
    4. };
    6. -(void)calcInBG:(NSDictionary*)meineWerte
    7. {
    8. While()
    9. {
    10. rechne…
    11. [self performselectorInMainThread:@selector(updateProgress:) withObject:[NSNumber numberWithInt:ProgressZustand]];
    12. }
    13. }
    15. -(void)updateProgress:(NSNumber*)Zustand
    16. {
    17. stlle progress auf Zustand
    18. }
    Alles anzeigen


    Rest musst du selber schaffen

    Gruß

    Claus
    2 Stunden Try & Error erspart 10 Minuten Handbuchlesen.

    Pre-Kaffee-Posts sind mit Vorsicht zu geniessen :)

  • little_pixel schrieb:

    Meinem Wissenstand nach ist dieser thread safe.

    Woher hast Du die Info?

    Nach Threading Programming Guide sind alle NSResponder grundsätzlich thread unsafe, es sei denn, in der spezifischen Klassendokumentation steht etwas anderes. Und das tut es nicht. Ausgehend von der Information kann man erstmal nicht davon ausgehen, dass NSProgressIndicator thread safe ist. Oder hast Du eine andere Quelle?

    Multithreading ist übrigens eine ganz tolle Möglichkeit, sich innerhalb kürzester Zeit große Probleme und ekelhaftes, nicht reproduzierbares Fehlverhalten einzufangen, wenn man nicht genau weiß, was man tut. Besser auf eine höhere Abstraktion gehen oder ganz drauf verzichten. Es gibt kaum Situationen, in denen man rohes Multithreading nicht vermeiden kann - und das will man, wo immer möglich.
    Multigrad - 360°-Produktfotografie für den Mac

  • Tobse001 schrieb:

    Mit dispatch_async / Blocks müsste man das nicht in mehrere Methoden auseinander brechen.


    Sicher, aber ich habe bewußt keine Blocks genommen. Ich finde Blocks sind nämlich alles andere als Anfängerfreundlich. Ich habe mich jedenfalls selten so schwer getan etwas zu kapieren wie diese merkwürdigen Konstrukte. Mal ganz davon abgesehen, dass ich mir die Syntax bis heute nicht merken kann und jedesmal wieder nachsehen muss wie man das mit den Parametern schreibt wenn ich einen Block erzeuge.

    Gruß

    Claus
    2 Stunden Try & Error erspart 10 Minuten Handbuchlesen.

    Pre-Kaffee-Posts sind mit Vorsicht zu geniessen :)

  • Wo liegt jetzt das Problem ?

    Dein Programm läuft grundsätzlich erst einmal in einem Thread: Dem MainThread. Dieser teilt sich seine Zeit mit der RunLoop. Bedeutet: Wenn Dein Programm dran ist, dann darf die Runloop nichts machen. Die Runloop ist aber für alles zuständig was der User so macht. Also Eingaben und Ausgaben. Eben auch alles was mit dem UI zusammen hängt. Läuft nun also Dein Programm in einer Schleife, dann muss die Runloop warten bis Dein Programm damit fertig ist und kann dann erst wieder was machen. Sprich solange deine Schleife läuft wird sich an der UI nichts ändern.
    lso der Ablauf ist ganz einfach. Du sagst z.B. dem Progress Indikator im Mainthread "Ey sachmal der Runloop bescheid du hast neue Werte zum Anzeigen". Das macht der auch. Jetzt muss aber erst die Runloop drankommen und die bekommt dann die Message von dem ProgressIOndikator und malt diesen schließlich neu.

    Damit das nicht so ist startest du jetzt einfach deine Berechnung in einem Hintergrund Thread. also einen zweiten Thread. Dein MainThread startet also nur den BackgroundThread und gibt dann wieder ab an die runloop. Der zweite Thread läuft dann halt mal gemütlich los und berechnet was du berechnen willst. Dieser Background Thread hat aber nun dummerweise keine Verbindung zur Runloop. Ergo muss ich, um im UI was zu ändern, dem MainThread Bescheid sagen: "Ey, änder mal den Progress Indikator und geh dann wieder in die Runloop damit diese die Änderung anzeigt".

    Das Ganze ist jetzt stark vereinfacht und sicher nicht Aminfest, aber eventuell ganz ok um den Einstieg zu bekommen.

    Gruß

    Claus
    2 Stunden Try & Error erspart 10 Minuten Handbuchlesen.

    Pre-Kaffee-Posts sind mit Vorsicht zu geniessen :)

  • Wenn Du das Source Code Gerüst von Claus verwendest, dann darfst Du in calcInBG: fortschritt nicht verwenden.

    Alles was in Deinem Code mit fortschritt zu tun hat gehört in die Methode rechnen: also etwa so:

    Quellcode

    1. - (IBAction)rechnen:(id)sender
    2. {
    3. double piinterval = [ende timeIntervalSinceDate:beginn] / 86400;
    4. [fortschritt setMaxValue:piinterval];
    5. [fortschritt setMinValue:0.0];
    6. [fortschritt setHidden:FALSE];
    7. [fortschritt startAnimation: self];
    9. [self performSelectorInBackGround:@selector(calcInBG:) withObject:nil]; // Wenn Du Werte aus dieser Methode in der CalcInBG brauchst kannst du diese in ein NSDictionary packen und dieses als Object mitgeben
    10. };
    12. - (void)calcInBG:(NSDictionary*)meineWerte
    13. {
    14. anzahl_tage = 0;
    15. // In piiinterval stehen die Anzahl der Tage drin. so oft wird die While-Schlife durchlaufen
    16. double piinterval = [ende timeIntervalSinceDate:beginn] / 86400;
    18. do {
    19. //irgendeine Berechnung
    20. anzahl_tage++;
    22. [self performSelectorOnMainThread:@selector(updateProgress:) withObject:[NSNumber numberWithInt:anzahl_tage] waitUntilDone:NO];
    23. while(abbruchbedingung);
    24. }
    25. [self performSelectorOnMainThread:@selector(calcCompleted) withObject:nil waitUntilDone:NO];
    26. }
    28. - (void)calcCompleted
    29. {
    30. [fortschritt stopAnimation: self];
    31. [fortschritt setHidden:TRUE];
    32. }
    Alles anzeigen

  • Hallo MCDan, danke erstmal. Aber irgendwie funktioniert das immer noch nicht :( Sollte jetzt alles an der Progressbar hängen?

    Ich poste mal die gesamte .m

    Quellcode

    1. #import "AppDelegate.h"
    3. @implementation AppDelegate
    4. @synthesize window = _window;
    5. @synthesize betrag_eingabe;
    6. @synthesize betrag_ausgabe;
    7. @synthesize basiszins_eingabe;
    8. @synthesize dynzins_eingabe;
    9. @synthesize festzins_eingabe;
    10. @synthesize gesamt_ausgabe;
    11. @synthesize geschaeftsart;
    12. @synthesize beginn_eingabe;
    13. @synthesize ende_eingabe;
    14. @synthesize tage_ausgabe;
    15. @synthesize beginn_ausgabe;
    16. @synthesize ende_ausgabe;
    17. @synthesize zinsen_ausgabe;
    18. @synthesize protag_ausgabe;
    19. @synthesize zinssatz_ausgabe;
    20. @synthesize zinsArray;
    21. @synthesize fortschritt;
    23. - (id)init
    24. {
    25. self = [super init];
    26. if (self) {
    27. zinsArray=[[NSMutableArray alloc]init];
    28. newDic=[[NSMutableDictionary alloc] init];
    29. }
    30. return self;
    31. }
    33. - (IBAction)setMinDate:(id)sender
    34. {
    35. [ende_eingabe setMinDate: [beginn_eingabe dateValue]];
    36. }
    38. - (IBAction)setMaxDate:(id)sender
    39. {
    40. [beginn_eingabe setMaxDate: [ende_eingabe dateValue]];
    41. }
    43. - (void)applicationDidFinishLaunching:(NSNotification *)aNotification
    44. {
    45. [ende_eingabe setDateValue: [NSDate date]];
    46. double startbetrag = 1000;
    47. [betrag_eingabe setFloatValue:startbetrag];
    48. [basiszins_eingabe setFloatValue:basiszins];
    49. double dynzins = 2;
    50. [dynzins_eingabe setFloatValue:dynzins];
    51. double festzins = 10;
    52. [festzins_eingabe setFloatValue:festzins];
    56. }
    58. - (IBAction)showSheet:(id)sender {
    60. [NSApp beginSheet:theSheet
    61. modalForWindow:(NSWindow *)_window
    62. modalDelegate:self
    63. didEndSelector:nil
    64. contextInfo:nil];
    65. }
    67. -(IBAction)closeSheet:(id)sender {
    69. [NSApp endSheet:theSheet];
    70. [theSheet orderOut:sender];
    72. }
    74. -(void)updateProgress:(NSNumber*)Zustand
    75. {
    76. [fortschritt setDoubleValue:(double) anzahl_tage];
    79. }
    83. - (void)rechnen:(id)sender
    84. {
    87. // alles leeren
    88. [zinsAC removeObjects:[zinsAC arrangedObjects]];
    90. NSCalendar *gregorian = [[NSCalendar alloc] initWithCalendarIdentifier:NSGregorianCalendar];
    91. NSDateComponents *dateComponents = [[NSDateComponents alloc] init];
    93. beginn = [beginn_eingabe dateValue];
    94. NSDate *beginnfest = [beginn_eingabe dateValue];
    95. ende = [ende_eingabe dateValue];
    97. NSDateFormatter *formatter = [[NSDateFormatter alloc] init];
    98. [formatter setDateFormat:@"yyyy"];
    99. NSString *beginn_jahr = [formatter stringFromDate: beginn];
    100. [formatter setDateFormat:@"MM"];
    101. NSString *beginn_monat = [formatter stringFromDate: beginn];
    102. [formatter setDateFormat:@"dd"];
    103. NSString *beginn_tag = [formatter stringFromDate: beginn];
    104. [formatter setDateFormat:@"yyyy"];
    105. NSString *ende_jahr = [formatter stringFromDate: ende];
    106. [formatter setDateFormat:@"MM"];
    107. NSString *ende_monat = [formatter stringFromDate: ende];
    108. [formatter setDateFormat:@"dd"];
    109. NSString *ende_tag = [formatter stringFromDate: ende];
    111. NSInteger mode = [[geschaeftsart selectedCell] tag];
    112. switch (mode) {
    113. case 0: zinsauswahl = 5; break;
    114. case 1: zinsauswahl = 8; break;
    115. case 2: zinsauswahl = [dynzins_eingabe floatValue]; break;
    116. case 3: zinsauswahl = [festzins_eingabe floatValue]; break; }
    118. zinsArray=[[NSMutableArray alloc]init];
    119. [fortschritt display];
    121. tageimhalbjahrgesamt = 0;
    122. tagecountergesamt = 0;
    123. tagcounterhalbjahr = 0;
    124. anzahl_tage = 0;
    125. jahrflag = 0;
    126. halbjahrflag = 0;
    127. saldo = 0;
    129. do {
    130. NSTimeInterval secondsElapsed = [ende timeIntervalSinceDate:beginn];
    132. anzahl_tage++;
    133. tagecountergesamt = (secondsElapsed / 86400);
    135. unsigned unitFlags = NSMonthCalendarUnit| NSYearCalendarUnit;
    136. NSDateComponents *comp = [gregorian components:unitFlags fromDate:beginn];
    138. switch ([comp year]) {
    139. case 2002: if ((([comp month] - 1) / 6) + 1 == 1) basiszins = 2.57; else basiszins = 2.47; break;
    140. case 2003: if ((([comp month] - 1) / 6) + 1 == 1) basiszins = 1.97; else basiszins = 1.22; break;
    141. case 2004: if ((([comp month] - 1) / 6) + 1 == 1) basiszins = 1.14; else basiszins = 1.13; break;
    142. case 2005: if ((([comp month] - 1) / 6) + 1 == 1) basiszins = 1.21; else basiszins = 1.17; break;
    143. case 2006: if ((([comp month] - 1) / 6) + 1 == 1) basiszins = 1.37; else basiszins = 1.95; break;
    144. case 2007: if ((([comp month] - 1) / 6) + 1 == 1) basiszins = 2.70; else basiszins = 3.19; break;
    145. case 2008: if ((([comp month] - 1) / 6) + 1 == 1) basiszins = 3.32; else basiszins = 3.19; break;
    146. case 2009: if ((([comp month] - 1) / 6) + 1 == 1) basiszins = 1.62; else basiszins = 0.12; break;
    147. case 2010: if ((([comp month] - 1) / 6) + 1 == 1) basiszins = 0.12; else basiszins = 0.12; break;
    148. case 2011: if ((([comp month] - 1) / 6) + 1 == 1) basiszins = 0.12; else basiszins = 0.37; break;
    149. case 2012: if ((([comp month] - 1) / 6) + 1 == 1) basiszins = 0.12; else basiszins = 0.12; break;
    150. default: basiszins = 0.12; }
    152. jahrflag = [comp year];
    153. halbjahrflag = (([comp month] - 1) / 6) + 1;
    154. tageimhalbjahrgesamt = ++tagcounterhalbjahr;
    155. tageimhalbjahrgesamtstringitem = [NSString stringWithFormat: @"%i", tagcounterhalbjahr];
    156. [formatter setDateFormat:@"dd.MM.yyyy"];
    157. zeitraum = [NSString stringWithFormat:@"%@ - %@", [formatter stringFromDate: beginnfest], [formatter stringFromDate: beginn]];
    159. if (mode == 3) {
    160. zinssatz = zinsauswahl; }
    161. else {
    162. zinssatz = basiszins + zinsauswahl; }
    164. if ( ( ([comp year] % 4 == 0) && ([comp year] % 100 != 0) ) || ([comp year] %400 == 0) ) {
    165. tageszinsen = ((zinssatz / 100) / 366) * [betrag_eingabe floatValue]; }
    166. else {
    167. tageszinsen = ((zinssatz / 100) / 365) * [betrag_eingabe floatValue]; }
    169. zinsen = tageszinsen*tageimhalbjahrgesamt;
    170. zinsenimzeitraumstringitem = [NSString stringWithFormat: @"%.2f Euro", zinsen];
    171. zinssatzstringitem = [NSString stringWithFormat: @"%.2f %%", zinssatz];
    172. tageszinsenstringitem = [NSString stringWithFormat: @"%.4f Euro", tageszinsen];
    173. saldo += tageszinsen;
    174. saldostringitem = [NSString stringWithFormat: @"%.2f Euro", saldo];
    175. [formatter setDateFormat:@"ddMM"];
    176. NSString *beginnabfrage = [formatter stringFromDate: beginn];
    178. [dateComponents setDay:+1];
    179. beginn = [gregorian dateByAddingComponents:dateComponents toDate: beginn options:0];
    181. if (![[betrag_eingabe stringValue] isEqualToString: @""]) {
    182. if ( [beginnabfrage isEqualToString: @"3006"] || [beginnabfrage isEqualToString: @"3112"]) {
    183. // Array schreiben
    184. newDic=[NSMutableDictionary dictionaryWithObjectsAndKeys:
    185. zeitraum, @"zeitraum",
    186. tageimhalbjahrgesamtstringitem, @"tage",
    187. zinssatzstringitem, @"zinssatz",
    188. tageszinsenstringitem, @"protag",
    189. zinsenimzeitraumstringitem, @"imzeitraum",
    190. saldostringitem, @"saldo",
    191. nil];
    192. [zinsAC addObject:newDic];
    193. tageimhalbjahrgesamt = 0;
    194. tagcounterhalbjahr = 0;
    195. beginnfest = beginn;
    196. }
    197. }
    199. [self performSelectorOnMainThread:@selector(updateProgress:) withObject:[NSNumber numberWithInt:anzahl_tage] waitUntilDone:NO];
    201. } while (tagecountergesamt >0);
    203. if (![[betrag_eingabe stringValue] isEqualToString: @""]) {
    204. // Array schreiben
    205. newDic=[NSMutableDictionary dictionaryWithObjectsAndKeys:
    206. zeitraum, @"zeitraum",
    207. tageimhalbjahrgesamtstringitem, @"tage",
    208. zinssatzstringitem, @"zinssatz",
    209. tageszinsenstringitem, @"protag",
    210. zinsenimzeitraumstringitem, @"imzeitraum",
    211. saldostringitem, @"saldo",
    212. nil];
    213. [zinsAC addObject:newDic];
    214. gesamtstand = saldo + [betrag_eingabe floatValue];
    215. }
    217. // Ausgabe
    218. NSString *betragString = [NSString stringWithFormat:@"%.2f Euro",[betrag_eingabe floatValue]];
    219. [betrag_ausgabe setStringValue:betragString];
    221. if (![[betrag_eingabe stringValue] isEqualToString: @""]) {
    222. gesamtstring = [NSString stringWithFormat:@"%.2f Euro",gesamtstand];
    223. } else {
    224. gesamtstring = @"Sie müssen einen Rechnungsbetrag eingeben!";
    225. }
    226. [gesamt_ausgabe setStringValue:gesamtstring];
    227. NSString *zinssatzString = [NSString stringWithFormat:@"%.2f %%",zinssatz];
    228. [zinssatz_ausgabe setStringValue:zinssatzString];
    230. NSString *tageString = [NSString stringWithFormat:@"%i Tage",anzahl_tage];
    231. [tage_ausgabe setStringValue:tageString];
    233. NSString *tageszinsenString = [NSString stringWithFormat:@"%.2f Euro",tageszinsen];
    234. [protag_ausgabe setStringValue:tageszinsenString];
    236. NSString *beginnString = [NSString stringWithFormat:@"%@.%@.%@",beginn_tag, beginn_monat, beginn_jahr];
    237. [beginn_ausgabe setStringValue:beginnString];
    239. NSString *endeString = [NSString stringWithFormat:@"%@.%@.%@",ende_tag, ende_monat, ende_jahr];
    240. [ende_ausgabe setStringValue:endeString];
    242. [self performSelectorOnMainThread:@selector(calcCompleted) withObject:nil waitUntilDone:NO];
    243. }
    245. - (IBAction)mainthread:(id)sender
    246. {
    247. piinterval = [ende timeIntervalSinceDate:beginn] / 86400;
    248. [fortschritt setMaxValue:piinterval];
    249. [fortschritt setMinValue:0.0];
    250. [fortschritt setHidden:FALSE];
    251. [fortschritt startAnimation: self];
    252. [self performSelectorInBackground:@selector(rechnen:) withObject:nil];
    253. }
    255. - (void)calcCompleted
    256. {
    257. [fortschritt stopAnimation: self];
    258. [fortschritt setHidden:TRUE];
    259. }
    261. @end
    Alles anzeigen
    Ich bin gegen Signaturen!!!

  • MCDan schrieb:

    Was funktioniert denn nicht?

    In updateProgress: solltest Du den ProgressIndicator natürlich mit dem übergeben Wert aktualisieren und nicht mit anzahl_tage. ;)


    Das wars ;) Jetzt funzt es! Danke!

    Ich krieg aber immer noch dieses CoreAnimation: warning, deleted thread with uncommitted CATransaction; set CA_DEBUG_TRANSACTIONS=1 in environment to log backtraces. auf der Konsole.

    Das CA_DEBUG_TRANSACTIONS=1 spuckt mir folgendes aus:

    CoreAnimation: warning, deleted thread with uncommitted CATransaction; created by:
    0 QuartzCore 0x00007fff8610bb95 _ZN2CA11Transaction4pushEv + 219
    1 QuartzCore 0x00007fff8610b76d _ZN2CA11Transaction15ensure_implicitEv + 273
    2 QuartzCore 0x00007fff861128ab _ZN2CA5Layer13thread_flags_EPNS_11TransactionE + 37
    3 QuartzCore 0x00007fff861127e3 _ZN2CA5Layer4markEPNS_11TransactionEjj + 79
    4 QuartzCore 0x00007fff8611274a _ZN2CA5Layer25set_needs_display_in_rectERK6CGRect + 392
    5 AppKit 0x00007fff8b89047d -[_NSViewBackingLayer setNeedsDisplayInRect:] + 206
    6 QuartzCore 0x00007fff8611259f -[CALayer setNeedsDisplay] + 62
    7 AppKit 0x00007fff8b8903a9 -[_NSViewBackingLayer setNeedsDisplay] + 73
    8 AppKit 0x00007fff8b891bc8 -[NSView(NSInternal) _setLayerNeedsDisplayInViewRect:] + 566
    9 AppKit 0x00007fff8b6cdaad -[NSView setNeedsDisplayInRect:] + 809
    10 AppKit 0x00007fff8b8d51d5 -[NSTableBinder _updateContent] + 392
    11 AppKit 0x00007fff8b8d4f67 -[NSTableBinder _observeValueForKeyPath:ofObject:context:] + 84
    12 AppKit 0x00007fff8b8d4d78 -[NSTableBinder observeValueForKeyPath:ofObject:change:context:] + 63
    13 Foundation 0x00007fff8659f860 NSKeyValueNotifyObserver + 390
    14 Foundation 0x00007fff86599d23 -[NSObject(NSKeyValueObservingPrivate) _notifyObserversForKeyPath:change:] + 967
    15 AppKit 0x00007fff8b830ed3 -[NSController _notifyObserversForKeyPath:change:] + 209
    Ich bin gegen Signaturen!!!