Android vibrations settings - red line under else?

Question

I have put this section of code in my Setting Activity, so if the vibration box in the xml file which is working is checked, vibrators will be turned on and if not they will be cancelled. However, there seems to be a problem with the else that won't let me run the app. Any help would be appreciated, thanks.

  if (preference instanceof vibrateapp_checkbox=="true");
    Vibrator.vibrate(new long[] { 0, 200, 0 }, 0);
    Else if (preference instanceof vibrateapp_checkbox=="false");
    Vibrator.cancel();

Hi i changed it to else but there is still a red line that says "delete this token"?? — user2229066, Apr 02 '13 at 11:27

Ankit · Accepted Answer · 2013-04-02T11:34:37.447

2

its not Else its else. java is case sensitive. moreover, the else is without if because you ended if by (;)

if (preference instanceof vibrateapp_checkbox=="true")
    Vibrator.vibrate(new long[] { 0, 200, 0 }, 0);
else if (preference instanceof vibrateapp_checkbox=="false")
    Vibrator.cancel();

moreover, this seems incorrect (preference instanceof vibrateapp_checkbox=="false") since after instanceof, type is expected not value

you can correct it as

if (vibrateapp_checkbox.isChecked())
        Vibrator.vibrate(new long[] { 0, 200, 0 }, 0);
    else
        Vibrator.cancel();

edited Apr 02 '13 at 11:34

answered Apr 02 '13 at 11:26

Ankit

6,554
6
49
71

Hi i changed it to else but there is still a red line that says "delete this token"?? – user2229066 Apr 02 '13 at 11:27
what would you use instead of instanceof – user2229066 Apr 02 '13 at 11:32
what is vibrateapp_checkbox is it a checkbox? – Ankit Apr 02 '13 at 11:34
– user2229066 Apr 02 '13 at 11:36
cheers but what is the problem here:if (vibrateapp_checkbox.isChecked()) vibrator = (Vibrator)getSystemService(Context.VIBRATOR_SERVICE); vibrator.vibrate(50); else vibrator.cancel(); – user2229066 Apr 02 '13 at 17:07

score 0 · Answer 2 · answered Apr 02 '13 at 11:28

Java is Case sensitive . Use else not Else

  if (preference instanceof vibrateapp_checkbox=="true")
    {
        Vibrator.vibrate(new long[] { 0, 200, 0 }, 0);
    }
        else if (preference instanceof vibrateapp_checkbox=="false")
    {
        Vibrator.cancel();
    }

Android vibrations settings - red line under else?

2 Answers2