That's a java thing:

to compare Strings use something like if (strResult.equals("Four")),
as (strResult == "Four") checks if the operands are the same object
and not if they have the same content.

On 26 Aug., 19:49, erik wagner <erik.j.wag...@gmail.com> wrote:
> Hi all, I have a very simple game that I'm trying to do for learning
> purposes.  One of the options is selecting the number of players (2-4)
> via a spinner.  When a choice is made from the spinner I am moving the
> value to a string and then using that string in a series of if/else if
> statements to (eventually) handle the option.  I am running into the
> problem where the condition that (to me) seems like it should be true
> is evaluating to false.
> If I select "Two", the value moved to the variable is "Two" but the
> check for the variable == "Two" isn't getting hit.
> (hope that was all clear, backstory may not be needed but better safe
> than vague).  Anyway, the spinner's OnSelectedItemListener...
>
>     public class MyPlayerOnItemSelectedListener implements
> OnItemSelectedListener {
>         public void onItemSelected(AdapterView<?> parent, View view, int
> pos, long id) {
>                 String strResult;
>                 strResult = parent.getItemAtPosition(pos).toString();
>                 if (strResult == "Two") {
>                         Toast.makeText(parent.getContext(), "this is the 
> result when
> one is selected", Toast.LENGTH_LONG).show();
>                 } else if (strResult == "Three") {
>                         Toast.makeText(parent.getContext(), "three players /" 
> +
> strResult + "/", Toast.LENGTH_LONG).show();
>                 } else if (strResult == "Four") {
>                         Toast.makeText(parent.getContext(), "four",
> Toast.LENGTH_LONG).show();
>                 } else {
>                         Toast.makeText(parent.getContext(), "something else 
> /" +
> strResult + "/", Toast.LENGTH_LONG).show();
>                 }
>
>         }
>         public void onNothingSelected(AdapterView<?> parent) {
>
>         }
>     }
>
> No matter what I select using this code the Toast shows as "something
> else /" + what the value actually is + "/".  i.e. if I chose 'Two' the
> toast reads "something else /Two/".
> Any help would be greatly appreciated on this.  I'm certain it is
> something very fundamental that I'm just missing.  Thanks in advance
> Erik Wagner

-- 
You received this message because you are subscribed to the Google
Groups "Android Developers" group.
To post to this group, send email to android-developers@googlegroups.com
To unsubscribe from this group, send email to
android-developers+unsubscr...@googlegroups.com
For more options, visit this group at
http://groups.google.com/group/android-developers?hl=en

Reply via email to