16-1 Simple way

[tadej5553]

I added a check to see if the destination exists or is a directory

main.m

Code: (Objective-C)

#import <Foundation/Foundation.h>

int main (int argc, const char * argv[]) 
{  
    NSAutoreleasePool * pool = [[NSAutoreleasePool alloc] init]; 
    
    NSFileManager *fm = [NSFileManager defaultManager];  
    NSString *dest;  
    NSProcessInfo *proc = [NSProcessInfo processInfo];  
    NSArray *args = [proc arguments]; 
    NSMutableArray *source = [NSMutableArray array];  
    BOOL isDir;  
    int i;
        
    if (argc < 3)  
    {  
        NSLog(@"Usage: %@ src dest", [proc processName]);  
        return 1;  
    }  
    
    for(i = 1; i < argc-1; ++i)
    {
        [source addObject: [args objectAtIndex: i]];  
    }
        
    for( NSString *temp in source)
    {
        dest = [args lastObject]; 

        if (![fm isReadableFileAtPath: temp])  
        {  
            NSLog(@"Can't read %@", temp);  
            return 2;  
        }  
        
        [fm fileExistsAtPath:dest isDirectory:&isDir];  
        if (!isDir)  // Added a check to see if the destination is a directory or even exists
        {
            NSLog(@"Destination is not a directory or doesn't exist!");
            return 3;
        }
        
        dest = [dest stringByAppendingPathComponent: [temp lastPathComponent]]; 
        
        [fm removeItemAtPath: dest error: NULL];  
        
        if (![fm copyItemAtPath: temp toPath: dest error: NULL])  
        {  
            NSLog(@"Copy failed!");  
            return 4;  
        }  
        
        NSLog(@"Copy of %@ to %@ succeeded!", temp, dest);  
    }
    
    [pool drain];  
    return 0;  
}

A question to skochan: Why didn't you use argc in this chapter and used [args count] instead?
I know you try to avoid low-level C stuff, but I think this was unnecesarry.

[skochan]

Dealing withe argv array as an array of character pointers is a little tricky and not object-oriented.  Since I don't require a deep understanding of C in the text,  I decided to use the NSProcessInfo class instead., as I also think it's clearer and easier to understand.

Cheers,

Steve Kochan

[tadej5553]

Sorry, but my question was about argc