Event to return value

[John S]

I have an event handler tied to a text box keypress event. All I am trying
to do is have it read the key and return it in upper case. Because the
event doesn't return anything, I cannot use the keyword "return". I thought
I could do it this way.
Here is the code.

public static void uCaseReturn(obj ect sender,
System.Windows. Forms.KeyPressE ventArgs e){

TextBox tb = (TextBox) sender;

string tb1 = tb.Text.ToUpper ();

tb.Text=tb1;

}

[Steve]

You are correct in that you cannot return a value from this event.

I'm not sure why your event handler is not specific to the text box you instantiated. For instance if your text box is named myTextBox, then the standard naming convention for the key press event handler should look like this:

private void myTextBox_KeyPr ess(object sender, KeyPressEventAr gs e)
{
myTextBox.Text = myTextBox.Text. ToUpper();
}

and your class constructor should have:

myTextBox.KeyPr ess +=new KeyPressEventHa ndler(tbSearch_ KeyPress);
Does this help?
Steve

"John S" wrote:

I have an event handler tied to a text box keypress event. All I am trying
to do is have it read the key and return it in upper case. Because the
event doesn't return anything, I cannot use the keyword "return". I thought
I could do it this way.
Here is the code.

public static void uCaseReturn(obj ect sender,
System.Windows. Forms.KeyPressE ventArgs e){

TextBox tb = (TextBox) sender;

string tb1 = tb.Text.ToUpper ();

tb.Text=tb1;

}

[William Ryan eMVP]

Would setting the .CharacterCasin g property of the textbox to Upper work for
you?

If not, just create an eventhandler for KeyPress and use it like so:

private void textBox1_KeyPre ss(object sender,
System.Windows. Forms.KeyPressE ventArgs e)

{

e.Handled = true;

((TextBox)sende r).Text += e.KeyChar.ToStr ing().ToUpper() ;

}
--

W.G. Ryan, eMVP

Have an opinion on the effectiveness of Microsoft Embedded newsgroups?
Let Microsoft know!
https://www.windowsembeddedeval.com/...ity/newsgroups
"John S" <jo********@cin fin.com> wrote in message
news:%2******** ********@TK2MSF TNGP09.phx.gbl. ..

I have an event handler tied to a text box keypress event. All I am trying to do is have it read the key and return it in upper case. Because the
event doesn't return anything, I cannot use the keyword "return". I thought I could do it this way.
Here is the code.

public static void uCaseReturn(obj ect sender,
System.Windows. Forms.KeyPressE ventArgs e){

TextBox tb = (TextBox) sender;

string tb1 = tb.Text.ToUpper ();

tb.Text=tb1;

}

[Ken Kolda]

See the CharacterCasing property on the TextBox control for an easy way to
force all letters to caps.

Ken
"John S" <jo********@cin fin.com> wrote in message
news:%2******** ********@TK2MSF TNGP09.phx.gbl. ..

I have an event handler tied to a text box keypress event. All I am trying to do is have it read the key and return it in upper case. Because the
event doesn't return anything, I cannot use the keyword "return". I thought I could do it this way.
Here is the code.

public static void uCaseReturn(obj ect sender,
System.Windows. Forms.KeyPressE ventArgs e){

TextBox tb = (TextBox) sender;

string tb1 = tb.Text.ToUpper ();

tb.Text=tb1;

}

[Ken Kolda]

See comments inline...

private void textBox1_KeyPre ss(object sender,
System.Windows. Forms.KeyPressE ventArgs e)

{

e.Handled = true;

((TextBox)sende r).Text += e.KeyChar.ToStr ing().ToUpper() ;

}

The event handler code above isn't sufficient because you don't know that
when the user pressed the key that the cursor is at the end of the textbox.
You'll need to use the SelectionStart and SelectionLength properties to
ensure that your character is placed in the correct location (and whatever
text was selected is removed), e.g.

public static void uCaseReturn(obj ect sender,
System.Windows. Forms.KeyPressE ventArgs e)
{
if (!Char.IsContro l(e.KeyChar))
{
TextBox tb = (TextBox) sender;
int charPosition = tb.SelectionSta rt;

// Build the new text for the box by replacing the selected text
with the character (in uppercase)
tb.Text = tb.Text.Substri ng(0, charPosition) +
Char.ToUpper(e. KeyChar) +
tb.Text.Substri ng(tb.Selection Start + tb.SelectionLen gth);

// Place the cursor in the correct location
tb.SelectionLen gth = 0;
tb.SelectionSta rt = charPosition;

// Mark the keystroke as being handled
e.Handled = true;
}
}

Ken

[mikeb]

John S wrote:

I have an event handler tied to a text box keypress event. All I am trying
to do is have it read the key and return it in upper case. Because the
event doesn't return anything, I cannot use the keyword "return". I thought
I could do it this way.
Here is the code.

public static void uCaseReturn(obj ect sender,
System.Windows. Forms.KeyPressE ventArgs e){

TextBox tb = (TextBox) sender;

string tb1 = tb.Text.ToUpper ();

tb.Text=tb1;

}

When the KeyPressed event is fired, the character for that key has not
yet been added to the Text property.

An easier way to force a TextBox to be uppercase is to set the TextBox's
CharacterCasing property;

tb1.CharacterCa sing = CharacterCasing .Upper;

If you need to do fancier manipulation of the Text property, you should
probably handle it in the TextChanged event.

--
mikeb

[Ken Kolda]

Yes, that should certainly work as long as you are careful to reposition the
caret to where it was at the start of the event. I was just trying to show
all that you'd have to do to use the KeyPress event since there are cases
when TextChanged wouldn't be appropriate (e.g. if you decide to filter out
certain characters). As a few people have mentioned, the easiest way to
solve his immediate problem is with the CharacterCasing property.

Ken
"Steve" <St***@discussi ons.microsoft.c om> wrote in message
news:EF******** *************** ***********@mic rosoft.com...

Wouldn't it be easier to do this on the TextChanged event?
That way the .text includes the new key and you know nothing
is currently highlighted, although you still have to care for
the position of the caret. Just curious.... Steve
"Ken Kolda" wrote:

See comments inline...

private void textBox1_KeyPre ss(object sender,
System.Windows. Forms.KeyPressE ventArgs e)

{

e.Handled = true;

((TextBox)sende r).Text += e.KeyChar.ToStr ing().ToUpper() ;

}

The event handler code above isn't sufficient because you don't know that when the user pressed the key that the cursor is at the end of the textbox. You'll need to use the SelectionStart and SelectionLength properties to
ensure that your character is placed in the correct location (and whatever text was selected is removed), e.g.

public static void uCaseReturn(obj ect sender,
System.Windows. Forms.KeyPressE ventArgs e)
{
if (!Char.IsContro l(e.KeyChar))
{
TextBox tb = (TextBox) sender;
int charPosition = tb.SelectionSta rt;

// Build the new text for the box by replacing the selected text
with the character (in uppercase)
tb.Text = tb.Text.Substri ng(0, charPosition) +
Char.ToUpper(e. KeyChar) +
tb.Text.Substri ng(tb.Selection Start + tb.SelectionLen gth);

// Place the cursor in the correct location
tb.SelectionLen gth = 0;
tb.SelectionSta rt = charPosition;

// Mark the keystroke as being handled
e.Handled = true;
}
}

Ken

[William Ryan eMVP]

If you need to have everything capitalized then setting the CharacterCasing
is the way to go . I agree with you though about the positioning.

--

W.G. Ryan, eMVP

Have an opinion on the effectiveness of Microsoft Embedded newsgroups?
Let Microsoft know!
https://www.windowsembeddedeval.com/...ity/newsgroups
"Ken Kolda" <ke*******@elli emae-nospamplease.co m> wrote in message
news:%2******** ********@TK2MSF TNGP11.phx.gbl. ..

See comments inline...

private void textBox1_KeyPre ss(object sender,
System.Windows. Forms.KeyPressE ventArgs e)

{

e.Handled = true;

((TextBox)sende r).Text += e.KeyChar.ToStr ing().ToUpper() ;

}
The event handler code above isn't sufficient because you don't know that
when the user pressed the key that the cursor is at the end of the

textbox. You'll need to use the SelectionStart and SelectionLength properties to
ensure that your character is placed in the correct location (and whatever
text was selected is removed), e.g.

public static void uCaseReturn(obj ect sender,
System.Windows. Forms.KeyPressE ventArgs e)
{
if (!Char.IsContro l(e.KeyChar))
{
TextBox tb = (TextBox) sender;
int charPosition = tb.SelectionSta rt;

// Build the new text for the box by replacing the selected text
with the character (in uppercase)
tb.Text = tb.Text.Substri ng(0, charPosition) +
Char.ToUpper(e. KeyChar) +
tb.Text.Substri ng(tb.Selection Start + tb.SelectionLen gth);

// Place the cursor in the correct location
tb.SelectionLen gth = 0;
tb.SelectionSta rt = charPosition;

// Mark the keystroke as being handled
e.Handled = true;
}
}

Ken