NSURLDownload abbrechen

[julian_mac]

Hallo zusammen, habe aus dieser Quelle einen Browser programmiert. Alles funktioniert excellent, jedoch wünsche ich mir eine Funktion, mit der ich aktive Downloads per Knopfdruck abbrechen kann. Stundenlange Probiererei und Recherche im Internet haben nichts geholfen. Hat jemand eine Idee?

Danke für Eure Hilfe!

 

[MacApple]

NSURLDownload hat eine Methode cancel. Einfacher geht es kaum, einen Download abzubrechen.

 

[DubiDuh]

Stundenlange Probiererei und Recherche im Internet haben nichts geholfen. Hat jemand eine Idee?

Ja, APIs lesen lernen - Denn die API zum Interface NSURLDownload ist links in der Leiste und es wird "cancel" als Methode angezeigt, sogar mit der Überschrift "Cancel Download".

 

[julian_mac]

Hallo MacApple und DubiDuh!
Danke für Eure schnelle Antworten!

Ich bin noch ein Frischling und stoppe gerade hier, könnt Ihr mir nochmals helfen?

- (void)cancel{
setDeletesFileUponFailure: NO
}

Aber wie kann ich diese Aktion bei Knopfdruck ausführen?
Hätte mir das eher so vorgestellt:

- (IBAction)StoppeDownloadid)sender {

//hier die Stopfunktion
setDeletesFileUponFailure: NO
}


DANKE!

 

[MacApple]

Hätte mir das eher so vorgestellt:

- (IBAction)StoppeDownload:(id)sender {

//hier die Stopfunktion
setDeletesFileUponFailure: NO
}

Das ist ja auch schon der richtige Ansatz. Du musst halt in dieser Action Methode cancel an deine NSURLDownload Instanz schicken.

 

[Schniko]

Vielleicht nochmal deutlicher:

//Irgendwo in deiner App sollte die Instanz mal erstellt worden sein. In etwa so:
NSURLDownload *deinName = [[NSURLDownload alloc] init];

...

//Zum Abbrechen dann entsprechend meiner Vorredner
[deinName cancel];

Du schickst die Cancel Methode direkt an deinen spezifischen Download.

 

[julian_mac]

Diese Instanz lässt sich finden:

NSURLDownload *down = [[NSURLDownload alloc] initWithRequest:request delegate:cellViewController];

[down setDestination:filePath allowOverwrite:NO];

Die Downloads werden jetzt zwar gestoppt beim Drücken des Buttons, aber die App gleich mit.
Xcode liefert eine Fehlermeldung, wie auf dem Screenshot zu sehen.

 

[Schniko]

Momentan legst du eine neue Instanz an, die "down" heißt, und willst diese dann direkt canceln. Das geht so nicht. Du darfst die nicht neu deklarieren, sondern die Instanz nehmen, die vorher den Download gestartet hat.

 

[julian_mac]

Aber wenn ich die Instanz nicht noch einmal festlege, dann sagt mir xcode, dass down nicht bekannt sei.

Hier meine Downlaod Delegate

//Download Delegate

- (void)webView:(WebView *)sender decidePolicyForMIMEType:(NSString *)type request:(NSURLRequest *)request frame:(WebFrame *)frame

decisionListener:(id < WebPolicyDecisionListener >)listener

{


    if ([type isNotEqualTo:mad:"text/html"]) {

        [listener use];

    

        NSURL *url = [request URL];

    

        NSArray *parts = [[[url absoluteString] stringByReplacingPercentEscapesUsingEncoding:NSUTF8StringEncoding] componentsSeparatedByString:mad:"/"];

        NSString *filename = [parts objectAtIndex:[parts count]-1];

    

        if([filename rangeOfString:mad:"?"].location != NSNotFound)

        {

            NSArray *names = [filename componentsSeparatedByString:mad:"?"];

            filename = [names objectAtIndex:0];

        }

    

        NSString *homePath = [self homeDirectory];

        NSString *filePath = [NSString stringWithFormat:mad:"%@/Downloads/%@", homePath, filename];

    

        DownloadData *downloadData = [[DownloadData alloc] init];

        downloadData.fileName = filename;

        downloadData.fileSize = 0;

    

        [listDownloadDatas addObject:downloadData];

    

        upDownCellViewController * cellViewController = [[upDownCellViewControlleralloc] init];

        [cellViewController initWithData:downloadData];

    

        [cellViewControllers addObject:cellViewController];

    

        NSURLDownload *down = [[NSURLDownload alloc] initWithRequest:request delegate:cellViewController];

        [down setDestination:filePath allowOverwrite:NO];

    

        [self.downloadTableViewreloadData];

        [AppDelegatesetArrayForKey:listDownloadDatasforKey:mad:"DownloadData"];

        NSLog(@"Download did start");

    

        return;

    }


    [listener use];

    NSLog(@"%@", type);

}

 

[nomos]

Und jetzt verpack deine Antwort noch in Code-tags, damit wir keine Smileys sehen.

 

[julian_mac]

Und jetzt verpack deine Antwort noch in Code-tags, damit wir keine Smileys sehen.

So besser?

 

[Schniko]

Ja, du legst down innerhalb dieser Methode als lokales Objekt an. Wenn du später da noch ran willst, kommst du so nicht mehr an den Pointer. Am einfachsten ist es, wenn du ganz zu Beginn dieser Klasse down als property anlegst, also so, dass du es als _down benutzen kannst. Dann kannst du später auch darauf zugreifen.

 

[julian_mac]

Ah das Prinzip habe ich verstanden, aber wie mache ich das konkret?

Also erstmal hier mein Button:

//DownloadsAbbrechen
- (IBAction)stopDownload:(id)sender {

    [_down cancel];

    setDeletesFileUponFailure: NO;

}

Und dann die Property:

@property (weak) ... *down;

Als was für einen Typ lege ich diese Property an?

 

[julian_mac]

Niemand da, der mir helfen kann?

 

[julian_mac]

Nach langem recherchieren bin ich immer noch nicht voran gekommen.
Kann mir jemand verraten, wie man die property hier anlegt?

 

[martinv2]

Warum sollte sich der Typ durch die Deklaration als Property ändern?
Benutze NSURLDownload*

 

[julian_mac]

Warum sollte sich der Typ durch die Deklaration als Property ändern?
Benutze NSURLDownload*

Danke für deine Antwort martinv2,
nachdem ich die property angelegt habe und mein Code folgendermaßen ausschaut, wird immer noch nicht der gewünschte Effekt erreicht den Download abzubrechen. Wenn ich nun den Button gedrückt halte, der den Download abbrechen sollte, dann stoppt der Download. Beim loslassen des Buttons wird er wieder fortgesetzt. Was nun?

- (IBAction)CancelActiveDownload:(id)sender {
    [self DownloadAbbrechen];

}
- (void)DownloadAbbrechen{
    NSLog(@"Downlaod was successfully stopped");
    [_down cancel];
    setDeletesFileUponFailure: NO;
}

 

[Schniko]

Wo legst du denn _down an?
Und was soll die Zeile

 setDeletesFileUponFailure: NO;

bedeuten? Die ergibt schon vom Syntax her keinen Sinn.

 

[julian_mac]

Das

setDeletesFileUponFailure: NO;

sorgt dafür, dass die Dateien nach dem Abbrechen des Downloads nicht gelöscht werden.

AppDelegate.h

@interface AppDelegate : NSViewController <NSApplicationDelegate, NSPopoverDelegate, NSTableViewDataSource, NSTableViewDelegate, NSApplicationDelegate, NSUserNotificationCenterDelegate, NSURLDownloadDelegate>
{
    NSMutableArray *cellViewControllers;
    NSMutableArray *listDownloadDatas;
    IBOutlet NSProgressIndicator *progressIndicator;
    NSURLConnection *_urlConnection;
    NSURLRequest *_request;
    NSURLDownload*_down;
   
   
}

AppDelegate.m

//Download Delegate

- (void)webView:(WebView *)sender decidePolicyForMIMEType:(NSString *)type request:(NSURLRequest *)request frame:(WebFrame *)frame

decisionListener:(id < WebPolicyDecisionListener >)listener

{


    if ([type isNotEqualTo:mad:"text/html"]) {

        [listener use];

  

        NSURL *url = [request URL];

  

        NSArray *parts = [[[url absoluteString] stringByReplacingPercentEscapesUsingEncoding:NSUTF8StringEncoding] componentsSeparatedByString:mad:"/"];

        NSString *filename = [parts objectAtIndex:[parts count]-1];

  

        if([filename rangeOfString:mad:"?"].location != NSNotFound)

        {

            NSArray *names = [filename componentsSeparatedByString:mad:"?"];

            filename = [names objectAtIndex:0];

        }

  

        NSString *homePath = [self homeDirectory];

        NSString *filePath = [NSString stringWithFormat:mad:"%@/Downloads/%@", homePath, filename];

  

        DownloadData *downloadData = [[DownloadData alloc] init];

        downloadData.fileName = filename;

        downloadData.fileSize = 0;

  

        [listDownloadDatas addObject:downloadData];

  

        upDownCellViewController * cellViewController = [[upDownCellViewControlleralloc] init];

        [cellViewController initWithData:downloadData];

  

        [cellViewControllers addObject:cellViewController];

  

        NSURLDownload *down = [[NSURLDownload alloc] initWithRequest:request delegate:cellViewController];

        [down setDestination:filePath allowOverwrite:NO];

  

        [self.downloadTableViewreloadData];

        [AppDelegatesetArrayForKey:listDownloadDatasforKey:mad:"DownloadData"];

        NSLog(@"Download did start");

  

        return;

    }


    [listener use];

    NSLog(@"%@", type);

}

DownloadController.m

- (IBAction)CancelActiveDownload:(id)sender {
    [self DownloadAbbrechen];

}
- (void)DownloadAbbrechen{
    NSLog(@"Downlaod was successfully stopped");
    [_down cancel];
    setDeletesFileUponFailure: NO;
}

 

[Schniko]

Du legst hier

NSURLDownload *down = [[NSURLDownload alloc] initWithRequest:request delegate:cellViewController];

        [down setDestination:filePath allowOverwrite:NO];

down an, nicht aber _down. Folglich existiert down nur innerhalb dieser Methode und du kannst später nicht darauf zugreifen. Mach also in den beiden oberen Zeilen ein _down daraus und es wird Wunder wirken.

Zu "setDeletesFileUponFailure": Kompiliert er denn mit dieser Zeile überhaupt? Denn so, wie es da steht, darf es eigentlich nicht gehen.