bloc/ios-exercises

ios-exercises

• BlocExercises.xcodeproj contains the Xcode project with iOS exercises.
• Specific assignments are in Exercise Descriptions.

ios-exercises's Issues

Answer alredy written in MarysAppleHandler.m for last two tests

https://github.com/Bloc/ios-exercises/blob/master/BlocExercises/BlocExercises/MarysAppleHandlerTests.m

- (void)testThatPriceOfDiscountedVodkaIs18Dollars {
    self.appleHandler.getsDiscount = YES;
    NSUInteger cost = [self.appleHandler dollarCostForAppleFlavoredVodka];
    XCTAssertEqual(cost, 18, @"Discounted vodka should be $18.");
}

- (void)testThatPriceOfUndiscountedVodkaIs24Dollars {
    NSUInteger cost = [self.appleHandler dollarCostForAppleFlavoredVodka];
    XCTAssertEqual(cost, 24, @"Undiscounted vodka should be $24.");
}

https://github.com/Bloc/ios-exercises/blob/master/BlocExercises/BlocExercises/MarysAppleHandler.m

- (NSUInteger) dollarCostForAppleFlavoredVodka {
    /* WORK HERE */

    NSUInteger cost = 24;

    if (self.getsDiscount) {
        cost *= .75;
    }

    return cost;
}

If/Else checkpoints/494 - Wrong Highlighting

I noticed on the If/Else checkpoint that the text ask you to replace the Yes with a No but the highlighting is both red

Should the No line be highlighted green to indicate that the Yes was changed to No?

StringCheese.m has not needed if/else statement

In the cheeseNameWithoutCheeseSuffix method, an if statement was added to see if it contained cheese.

I use this as a good place to explore the documentation with my students, and there's a method in NSString that's designed to handle this exact problem, without an if statement. To me, the best answer would be as follows:

NSString *nameNoCheese = [cheeseName stringByReplacingOccurrencesOfString:@" cheese" withString:@"" options:NSCaseInsensitiveSearch range:NSMakeRange(0, cheeseName.length)];

Can we remove the if statement so both answers could be acceptable?

Thanks!

Comment in StringCheese.m needs to be updated.

There is a comment (line 30 and 47) in StringCheese.m stating you will learn more about if/else in the next checkpoint, but the next checkpoint is on Equality. This might need to be updated to state you will learn about "if/else" in a "future" checkpoint.

    /*
     (You will learn more about if/else statements in the next checkpoint.)
     */