Error validating login

Navigation
1

I have the following code: 

package br.edu.utfpr.exer07;

    /*
    7) Implementar uma interface que contenha os métodos à ser usados:
    a. double soma(double valor1, double valor2, double valor3);
    b. String calculaTabuada(double tabuada, double valorInicial, double
    valorFinal);
    c. boolean login(String login, String senha);
    Implementar uma classe utilizando encapsulamento para armazenar cada tipo
    dos valores de retornos desta interface e que esta classe implemente esta
    interface e que contenha mais os métodos para apresentar cada um dos
    valores.
     */

public interface Calculos {

    public double soma(double valor1, double valor2, double valor3);
    public String calculaTabuada(double tabuada, double valorInicial, double valorFinal);
    public boolean login(String login, String senha);       
}

package br.edu.utfpr.exer07;

/*
7) Implementar uma interface que contenha os métodos à ser usados:
a. double soma(double valor1, double valor2, double valor3);
b. String calculaTabuada(double tabuada, double valorInicial, double
valorFinal);
c. boolean login(String login, String senha);
Implementar uma classe utilizando encapsulamento para armazenar cada tipo
dos valores de retornos desta interface e que esta classe implemente esta
interface e que contenha mais os métodos para apresentar cada um dos
valores.
 */

public class ExecutaCalculos implements Calculos {

    private double valor1;
    private double valor2;
    private double valor3;
    private double tabuada;
    private double valorInicial;
    private double valorFinal;
    private String login = "adm";
    private String senha = "123";

    public double getValor1() {
        return valor1;
    }

    public void setValor1(double valor1) {
        this.valor1 = valor1;
    }

    public double getValor2() {
        return valor2;
    }

    public void setValor2(double valor2) {
        this.valor2 = valor2;
    }

    public double getValor3() {
        return valor3;
    }

    public void setValor3(double valor3) {
        this.valor3 = valor3;
    }

    public double getTabuada() {
        return tabuada;
    }

    public void setTabuada(double tabuada) {
        this.tabuada = tabuada;
    }

    public double getValorInicial() {
        return valorInicial;
    }

    public void setValorInicial(double valorInicial) {
        this.valorInicial = valorInicial;
    }

    public double getValorFinal() {
        return valorFinal;
    }

    public void setValorFinal(double valorFinal) {
        this.valorFinal = valorFinal;
    }

    public String getLogin() {
        return login;
    }

    public void setLogin(String login) {
        this.login = login;
    }

    public String getSenha() {
        return senha;
    }

    public void setSenha(String senha) {
        this.senha = senha;
    }

    @Override
    public double soma(double valor1, double valor2, double valor3) {
        return valor1 + valor2 + valor3;
    }

    @Override
    public String calculaTabuada(double tabuada, double valorInicial, double valorFinal) {
        for(double i=  valorInicial; i<=valorFinal; i++) {
            System.out.println(tabuada+" X "+i+" = "+tabuada*i);
        }
        return null;
    }

    @Override
    public boolean login(String login, String senha) {
        boolean estado;
        if(login == this.login && senha == this.senha) {
            System.out.println("Login efeituado com sucesso");
            estado = true;
        } else {
            System.out.println("Login e/ou senha inválidos");
            estado = false;
        }
        return estado;
    }
}

package br.edu.utfpr.exer07;

import javax.swing.JOptionPane;

public class Exer07 {

    public static void main(String[] args) {


        ExecutaCalculos ec = new ExecutaCalculos();

        //ec.setLogin(JOptionPane.showInputDialog("Login: "));
        //ec.setSenha(JOptionPane.showInputDialog("Senha: "));

        if(ec.login((JOptionPane.showInputDialog("Login: ")), (JOptionPane.showInputDialog("Senha: "))) == true) {
            System.out.println("| 1 - Somar            |"
                             + "| 2 - Calcular tabuada |");
            int opcao = Integer.parseInt(JOptionPane.showInputDialog(("1 - Soma\n"
                                                                    + "2 - Calcular tabuada\n"
                                                                    + "Opção: ")));
            switch(opcao) {
            case 1:
                ec.setValor1(Double.parseDouble(JOptionPane.showInputDialog("Informe o primeiro valor:")));
                ec.setValor2(Double.parseDouble(JOptionPane.showInputDialog("Informe o segundo valor:")));
                ec.setValor3(Double.parseDouble(JOptionPane.showInputDialog("Informe o terceiro valor:")));
                System.out.println(ec.soma(ec.getValor1(), ec.getValor2(), ec.getValor2()));
                break;
            case 2:
                ec.setTabuada(Double.parseDouble(JOptionPane.showInputDialog("Informe a tabuada que deseja calcular:")));
                ec.setValorInicial(Double.parseDouble(JOptionPane.showInputDialog("Informe o valor inicial:")));
                ec.setValorFinal(Double.parseDouble(JOptionPane.showInputDialog("Informe o valor final")));
                ec.calculaTabuada(ec.getTabuada(), ec.getValorInicial(), ec.getValorFinal());
            }
        }

    }

}

And when you run this if on the main class, calling the login method of the ExecutarCalculos class, it always returns false . What could be the problem? 

if(ec.login((JOptionPane.showInputDialog("Login: ")), (JOptionPane.showInputDialog("Senha: "))) == true)
    
asked by anonymous 25.05.2015 / 01:57

1 answer

2

It's a common mistake that you have, I believe everyone has already been through this =)

In java when == is used to compare two objects it checks whether both objects point to the same memory space. An example with String would be this: 

final String string1 = new String("String");
final String string2 = "String";
final String string3 = string1;
final String string4 = "String";
if (string1 == string2) {
    System.out.println("string1 e string2 são o mesmo objeto.");
} else {
    System.out.println("string1 e string2 NÃO são o mesmo objeto.");
}
if (string1.equals(string2)) {
    System.out.println("string1 e string2 possuem o mesmo conteúdo, mas não o mesmo.");
} else {
    System.out.println("string1 e string2 NÃO possuem o mesmo conteúdo, mas não o mesmo.");
}
if (string1 == string3) {
    System.out.println("string1 e string3 são o mesmo objeto.");
} else {
    System.out.println("string1 e string3 NÃO são o mesmo objeto.");
}
if (string1 == string4) {
    System.out.println("string1 e string4 são o mesmo objeto.");
} else {
    System.out.println("string1 e string4 NÃO são o mesmo objeto.");
}
if (string2 == string4) {
    System.out.println("string2 e string4 são o mesmo objeto.");
} else {
    // dead code, sempre será igual
    System.out.println("string2 e string4 NÃO são o mesmo objeto.");
}
if (string3.equals(string4)) {
    System.out.println("string3 e string4 possuem o mesmo conteúdo, mas não o mesmo.");
} else {
    System.out.println("string3 e string4 NÃO possuem o mesmo conteúdo, mas não o mesmo.");
}

That would result in this: 

string1 e string2 NÃO são o mesmo objeto.
string1 e string2 possuem o mesmo conteúdo, mas não o mesmo.
string1 e string3 são o mesmo objeto.
string1 e string4 NÃO são o mesmo objeto.
string2 e string4 são o mesmo objeto.
string3 e string4 possuem o mesmo conteúdo, mas não o mesmo.

That is, we know that the content of String is always the same, that is, String . However we have in this case two different objects, in other words, we occupy two different memory spaces with the same content. Any combination with equals will always be equal, but with == no, since between string1 and string2 or string4 will be false.

In these cases the equals of Object verifying the equity of content, when overwritten. By default it only checks to point to the same memory space.

    
25.05.2015 / 02:28
How to insert an array into a variable? Replace URL in an embed tag