2016-09-22 3 views
0

Я пытаюсь определить при запуске, было ли приобретенное приложение, это мой текущий код;if/else инструкции типа в java

if (!(getSharedPreferences("purchased", 0).getBoolean("purchased", false))) 
admob();   
Log.d("NOT UPGRADED", "SHOW AD AS UPGRADED ALREADY"); 
if (!(getSharedPreferences("purchased", 0).getBoolean("purchased", true))) 
admobskip();  
Log.d("UPGRADED", "DO NOT SHOW AD AS UPGRADED ALREADY"); 

Но это, кажется, проходит через весь код, а не проверять, с прошивкой я использую, если/другое, однако мне кажется, не в состоянии использовать это и хотел Java equivelent для этого фрагмента кода, если это возможно? Я поменял купленные на «true» на вторую половину выписки.

+4

Менее своеобразных отступы, безусловно, поможет нам, и, вероятно, со временем, вы, как хорошо. – Bathsheba

+0

Обратите внимание, что ваши операторы 'Log.d' не являются частью операторов if, поэтому они всегда выполняются. – Eran

+1

Всегда используйте фигурные скобки, и вы выясните, в чем проблема: –

ответ

1

Используйте это вместо

if (getSharedPreferences("purchased", 0).getBoolean("purchased",false) == false) { 
    admob();   
    Log.d("NOT UPGRADED", "SHOW AD AS UPGRADED ALREADY"); 
} else if (getSharedPreferences("purchased", 0).getBoolean("purchased",true) == true) { 
    admobskip();  
    Log.d("UPGRADED", "DO NOT SHOW AD AS UPGRADED ALREADY"); 
} 
+0

'== true' - тавтология. – Bathsheba

+0

@ Батшеба просто для того, чтобы прояснить ему, поскольку он новичок в Java;) – Alamri

+0

Спасибо за это, вы ответили без предубеждений, к сожалению, некоторые из других парней были немного суровыми, учитывая, что я новичок в этом, еще раз спасибо ! :) – Hypergater

3

Если вы хотите только операторы для запуска, если if состояние проходит, а затем использовать Scope блоки { и } вокруг этого кода:

if (condition){ 
    /*run all these statements*/ 
} 

Если вы не используете скобки, то только следующие одного заявление является частью if: это то, что происходит в вашем случае.

(Лично я всегда использовать брекеты, поскольку я считаю, что это более четкий стиль, но не все это делают).

+0

Очень полезно, спасибо вам, также хорошо объяснили! – Hypergater

Смежные вопросы