0

Estou criando uma Activity em Android, cujo código está aqui, e estou obtendo um NullPointerException próximo da linha

final TextView txtResult = new TextView(this);

Obtive o seguinte dump do stack:

05-07 09:12:57.180: E/AndroidRuntime(887): FATAL EXCEPTION: main
05-07 09:12:57.180: E/AndroidRuntime(887): Process: br.com.colorcalc, PID: 887
05-07 09:12:57.180: E/AndroidRuntime(887): java.lang.RuntimeException: Unable to instantiate activity ComponentInfo{br.com.colorcalc/br.com.colorcalc.Calculator}: java.lang.NullPointerException
05-07 09:12:57.180: E/AndroidRuntime(887):  at android.app.ActivityThread.performLaunchActivity(ActivityThread.java:2102)
05-07 09:12:57.180: E/AndroidRuntime(887):  at android.app.ActivityThread.handleLaunchActivity(ActivityThread.java:2226)
05-07 09:12:57.180: E/AndroidRuntime(887):  at android.app.ActivityThread.access$700(ActivityThread.java:135)
05-07 09:12:57.180: E/AndroidRuntime(887):  at android.app.ActivityThread$H.handleMessage(ActivityThread.java:1397)
05-07 09:12:57.180: E/AndroidRuntime(887):  at android.os.Handler.dispatchMessage(Handler.java:102)
05-07 09:12:57.180: E/AndroidRuntime(887):  at android.os.Looper.loop(Looper.java:137)
05-07 09:12:57.180: E/AndroidRuntime(887):  at android.app.ActivityThread.main(ActivityThread.java:4998)
05-07 09:12:57.180: E/AndroidRuntime(887):  at java.lang.reflect.Method.invokeNative(Native Method)
05-07 09:12:57.180: E/AndroidRuntime(887):  at java.lang.reflect.Method.invoke(Method.java:515)
05-07 09:12:57.180: E/AndroidRuntime(887):  at com.android.internal.os.ZygoteInit$MethodAndArgsCaller.run(ZygoteInit.java:777)
05-07 09:12:57.180: E/AndroidRuntime(887):  at com.android.internal.os.ZygoteInit.main(ZygoteInit.java:593)
05-07 09:12:57.180: E/AndroidRuntime(887):  at dalvik.system.NativeStart.main(Native Method)
05-07 09:12:57.180: E/AndroidRuntime(887): Caused by: java.lang.NullPointerException
05-07 09:12:57.180: E/AndroidRuntime(887):  at android.content.ContextWrapper.getResources(ContextWrapper.java:89)
05-07 09:12:57.180: E/AndroidRuntime(887):  at android.view.ContextThemeWrapper.getResources(ContextThemeWrapper.java:78)
05-07 09:12:57.180: E/AndroidRuntime(887):  at android.view.View.<init>(View.java:3429)
05-07 09:12:57.180: E/AndroidRuntime(887):  at android.view.View.<init>(View.java:3496)
05-07 09:12:57.180: E/AndroidRuntime(887):  at android.widget.TextView.<init>(TextView.java:622)
05-07 09:12:57.180: E/AndroidRuntime(887):  at android.widget.TextView.<init>(TextView.java:617)
05-07 09:12:57.180: E/AndroidRuntime(887):  at android.widget.TextView.<init>(TextView.java:613)
05-07 09:12:57.180: E/AndroidRuntime(887):  at br.com.colorcalc.Calculator.<init>(Calculator.java:26)
05-07 09:12:57.180: E/AndroidRuntime(887):  at java.lang.Class.newInstanceImpl(Native Method)
05-07 09:12:57.180: E/AndroidRuntime(887):  at java.lang.Class.newInstance(Class.java:1208)
05-07 09:12:57.180: E/AndroidRuntime(887):  at android.app.Instrumentation.newActivity(Instrumentation.java:1061)
05-07 09:12:57.180: E/AndroidRuntime(887):  at android.app.ActivityThread.performLaunchActivity(ActivityThread.java:2093)
05-07 09:12:57.180: E/AndroidRuntime(887):  ... 11 more

Será que tem algo relacionado com meu método onCreate?

        public void onCreate(Bundle savedInstanceState) {
            super.onCreate(savedInstanceState);
            setContentView(R.layout.activity_main);
            txtResult = (TextView) findViewById(R.id.txtResult);
            btnCalc.setOnClickListener (new View.OnClickListener() {
                    public void onClick(View v) {      
                            result = "#" + rst1 + remainderR + rst2 + remainderG + rst3 + remainderB;
                            txtResult.setText(result);
                    }
                });
        }
11
  • 1
    final significa que você está declarando uma constante ao invés de uma variável, logo só poderá atribuir o valor à txtResult em sua criação. Sobre o NPE, você tem TextView chamado txtResult no seu xml?
    – Math
    Commented 7/05/2014 às 13:47
  • Experimenta assim: final TextView txtResult = new TextView(ATuaActivity.this);
    – Jorge B.
    Commented 7/05/2014 às 13:50
  • Tenho sim Math. E inclusive utilizei-o para mostrar o resultado de uma equação. Vou incluir a linha no post principal. Commented 7/05/2014 às 13:51
  • 1
    Se entendi bem seu dump, br.com.colorcalc.Calculator é sua Activity, correto? O <init> significa que o código está sendo executado no construtor. Você está mesmo fazendo tudo isso dentro do construtor da Activity ou eu entendi algo errado? Se estiver criando as views dentro do construtor, o conceito está errado, você deveria criar as views (manualmente, ou através do método setContentView) dentro do onCreate da Activity, ou outro lugar parecido. Commented 7/05/2014 às 14:02
  • 2
    Apenas lembrem-se de formatar a pergunta ao final da discussão para evitar links externos. E tentem fazer uma pergunta sucinta, que seja útil para futuros visitantes e que não fuja da pergunta original. @leo.saldanha, recomendo a leitura de dois links: Como fazer uma boa pergunta? e Como criar um exemplo Mínimo, Completo e Verificável
    – Math
    Commented 7/05/2014 às 14:58

2 Respostas 2

2

Na linha 40 do código postado

btnCalc.setOnClickListener (new View.OnClickListener()

Você está utilizando btnCalc, mas ele não foi atribuído antes. Antes de utilizar btnCalc falta algo como

btnCalc = (Button)findViewById(R.id.btnCalc);

Mais um detalhe, para converter um número inteiro x para sua notação em hexadecimal, é possível simplesmente fazer isso:

Integer.toHexString(x)
4
  • O fato de criar o evento no botão no XML e adicionar esse código à classe atrapalha em algo? Commented 7/05/2014 às 14:55
  • Se você já colocou no XML, então, não precisa colocar o setOnClickListener no cóldigo, aliás, nem deveria ;) Agora, se tirar do XML, aí pode colocar no código, sem problemas. Commented 7/05/2014 às 14:58
  • Funcionou, obrigado! Commented 7/05/2014 às 14:59
  • 1
    Depois tente dar uma otimizada naquelas conversões de hexadecimal, como te falei ;) Commented 7/05/2014 às 15:01
1

Final - Para declaração de constantes.

Agora em questão a boas práticas, essa sua linha do código.
final TextView txtResult = new TextView(this);
No meu caso eu mantenho as boas práticas e faço o seguinte:

- Declaro todos objetos de interface que vou usar, de maneira global (logo abaixo do public class). -
- Dessa maneira: private TextView txtResult; - Dentro do método onCreate() eu referencio que a variável txtResult da seguinte maneira:

txtResult = (TextView) findViewById(R.id.ID_DO_OBJETO);

Assim dessa maneira você pode usar o txtResult em várias partes do código.

Outra boa prática que uso é o seguinte, no XML do layout, no código do botão, eu acrescento :

android:onClick="METODO"

obs: dentro do onClick="" o método deve ser escrito sem o ().
Dessa maneira não é necessário fazer oque você fez, que no caso foi:

btnCalc.setOnClickListener (new View.OnClickListener()  {   
    public void onClick(View v){
        result = "#" + rst1 + remainderR + rst2 + remainderG + rst3 + remainderB;
        txtResult.setText(result);  
    }
});
5
  • Dá um olhada no dump dele... Aparentemente isso tudo está acontecendo dentro do br.com.colorcalc.Calculator.<init>, ou seja, o construtor da classe (o que não seria o melhor lugar para instanciar as views). E sobre o final TextView txtResult ..., não tem problema declarar uma view como final para ser utilizada dentro do método. Só colocaria como um campo da classe, se realmente precisasse utilizar a view em mais de um método da Activity Commented 7/05/2014 às 14:13
  • Mas essa maneira que eu disse como boa prática é o aconselhável né? do que colocar as views no onCreate. Commented 7/05/2014 às 14:18
  • Sobre a formatação do seu código, você pode aprender um pouco lendo esse tópico: Como devemos formatar perguntas e respostas?
    – Math
    Commented 7/05/2014 às 14:19
  • Não entendi bem a pergunta do seu comentário, @EnzoTiezzi... Se quiser, depois vem falar comigo no primeiro horário, que eu estou meio tranquilo! :) Commented 7/05/2014 às 14:23
  • ok @carlosrafaelgn, depois da uma olhada no face, te deixei uma pergunta, Valeu!! Commented 7/05/2014 às 14:24

Você deve fazer log-in para responder a esta pergunta.

Esta não é a resposta que você está procurando? Pesquise outras perguntas com a tag .