trouble changing colors back from buttons after playing sound...

Question

4.00/5 (1 vote)

See more:

So i have been working on a program where you can call up wav files with buttons named tabs or by putting in the name of the note with a masked textbox. what i want is when i play my inputted note it collors my respective note in a line of arrays red. but when the note stops playin i want to have the original default color.

my code is like this:
playbutton:

C#

private void button1_Click(object sender, EventArgs e)
        {
            string noot;
            int i = 0,j = 0;
            noot = noottextbox.Text;
            Compare(noot, ref i, ref j);
            playtab(i,j);
            _tabs[i, j].BackColor = System.Drawing.Color.FromArgb(((int)(((byte)(255)))), ((int)(((byte)(128)))), ((int)(((byte)(128)))));
        }

compare the input string with my array:

C#

private void Compare(string noot, ref int x, ref int y)
       {
           Boolean gevonden = false;
           x = 0;
           y = 0;
           for (int i = 0; i < 6 && !gevonden; i++)
               for (int j = 0; j < 13 && !gevonden; j++)
                   if (noot == _tabsnaam[i, j])
                   {
                       x = i;
                       y = j;
                       gevonden = true;
                   }
       }

play the note:

C#

private void playtab(int i, int j)
       {
           SoundPlayer tabsound = new SoundPlayer(@"C:\Users\Niko D'haes\Documents\Visual Studio 2005\Projects\guitarmidi\_FILES\beep.wav");
           if (i == 0 && j == 0)
           {
               tabsound.Play();
           }
       }

Now when i try to insert folowing line:
_tabs[i, j].UseVisualStyleBackColor = true;
in my buttonfunction, it just won't light up.
and when i put it behind my play function the button just stays red.
is there any way i can change the color back when it stops playin the sound?

Posted 9-Dec-10 0:48am

niko D'haes

Add a Solution

Comments

E.F. Nijboer 9-Dec-10 6:51am

Just as side note: In your method playtab, why do you create a SoundPlayer object with some chance that it isn't used? You could better move that line into the if statement and create the object only when needed.

niko D'haes 9-Dec-10 7:55am

i'll keep that in mind, thank you

2 solutions

Add a Solution

Add your solution here

Treat my content as plain text, not as HTML

Preview 0

…

Existing Members

Sign in to your account

...or Join us

Download, Vote, Comment, Publish.

Your Email
Password
Forgot your password?

Your Email
This email is in use. Do you need your password?
Optional Password

I have read and agree to the Terms of Service and Privacy Policy
Please subscribe me to the CodeProject newsletters

When answering a question please:

Read the question carefully.
Understand that English isn't everyone's first language so be lenient of bad spelling and grammar.
If a question is poorly phrased then either ask for clarification, ignore it, or edit the question and fix the problem. Insults are not welcome.
Don't tell someone to read the manual. Chances are they have and don't get it. Provide an answer or move on to the next question.

Let's work to help developers, not make them feel stupid.

This content, along with any associated source code and files, is licensed under The Code Project Open License (CPOL)

Espen Harlinn · Answer 1 · 2011-01-07T07:51:00

Solution 1

How about saving and restoring original color?

Regards
Espen Harlinn

Posted 7-Jan-11 7:51am

Espen Harlinn

William Winner · Answer 2 · 2011-01-07T08:16:00

Ok, so this won't answer your question, but why in the world did you write this:

C#

System.Drawing.Color.FromArgb(((int)(((byte)(255)))), ((int)(((byte)(128)))), ((int)(((byte)(128)))))

First of all, why are you using all of those parentheses? You could just have easily written:

C#

System.Drawing.Color.FromArgb((int)(byte)255,(int)(byte)128,(int)(byte)255);

But to take it one step further, why in the world are you converting an integer value to a byte and then back into an int? The normal way to write than line would have just been:

C#

System.Drawing.Color.FromArgb(255,128,128);

As to your question...what Espen said...