Shake phone and play sound

[aceiswild]

Clearly I'm missing out on somthing and i know its some stupid mistake in my code but my eyes are not catching it. I have a list of arraySounds that play fine with a button. But now i want it to play a random sound when you shake the phone.

Heres my code:

// SHAKE START --------------------------------------

- (void)motionBegan:(UIEventSubtype)motion withEvent:(UIEvent *)event
{
    if(event.type == UIEventSubtypeMotionShake)
    {
        NSLog(@"Iphone has been shook");
        {
{
int index = arc4random % [soundArray count];
AVAudioPlayer* theSound = [soundArray objectAtIndex: index];
[theSound play];
}
        }
    }
}

- (BOOL)canBecomeFirstResponder
{ 
    return YES; 
}


- (void)viewDidAppear:(BOOL)animated {
    [super viewDidAppear:animated];
    [self becomeFirstResponder];
}

// SHAKE END ---------------------------------

Thanks for any suggestions!

[Domele]

Your use of { } do not make sense at all. Why do I have a feeling you are copying and pasting code without really knowing how it works...

This is how you use braces correctly:

if (this)
{
     //run this code
    //and that
}

if (this)
{

    if (that)
    {
       //run some code
       //run some more code
    }

}

Excuse my terrible formatting, for some reason, I can't use tab here.

Here is your code partly fixed:

- (void)motionBegan:(UIEventSubtype)motion withEvent:(UIEvent *)event
{
    if(event.type == UIEventSubtypeMotionShake)
    {
        NSLog(@"Iphone has been shook"); {
        int index = arc4random % [soundArray count];
        AVAudioPlayer* theSound = [soundArray objectAtIndex: index];
        [theSound play];
    }
}

There is one "{" that does not belong. Find it and remove it.

[aceiswild]

Originally Posted by Domele

Your use of { } do not make sense at all. Why do I have a feeling you are copying and pasting code without really knowing how it works...

This is how you use braces correctly:

if (this)
{
     //run this code
    //and that
}

if (this)
{

    if (that)
    {
       //run some code
       //run some more code
    }

}

Excuse my terrible formatting, for some reason, I can't use tab here.

Here is your code partly fixed:

- (void)motionBegan:(UIEventSubtype)motion withEvent:(UIEvent *)event
{
    if(event.type == UIEventSubtypeMotionShake)
    {
        NSLog(@"Iphone has been shook"); {
        int index = arc4random % [soundArray count];
        AVAudioPlayer* theSound = [soundArray objectAtIndex: index];
        [theSound play];
    }
}

There is one "{" that does not belong. Find it and remove it.

Thanks. Havnt done the changes yet. Not home. Actually I never copied and pasted that code. The reason why I had extra brackets it I forgot to take them out from my other sound code before I decided to randomize it. But I had no errors when I removes the sound play code and ran it. I'm still getting used to objective c coming from pure web design. I have to say you a mean but awesome teacher! Lol you should make a tutorial/business website.