El codigo que planteas tiene algunos errores que pueden pasar factura según crezca tu modelo de datos.
El mas grande es realizar una consulta de toda la tabla para obtener el listado de usuarios. Lo primero seria que la SQL solo diera con el resultado del usuario por username o ninguno. Para ello se ha de añadir un filtro. Si este filtro estarias recorriendo toda la tabla si el ususario no es válido lo que es una perdida de tiempo y memoria. Ademas de poder ocasionar errores OutOfMemoryException si no tenemos memoria suficiente para obtener toda la tabla en memoria.
El segundo problema es poner las variables "usuario" y "contraseña" como variables de la clase. Si esta clase es un "singleton" podrias tener problemas de concurrencia y un usuario acceda como otro (entre otras cosas). O que se use el nombre guardado anteriormente al realizar login de nuevo.
Ejemplo usando el select con filtro:
String usuario = txt_usuario.getText();
String password = txt_pass.getText();
Connection cn;
PreparedStatement pst;
ResultSet rs;
try {
// Se inicializa a null
String usuarioCorrecto = null;
String passwordCorrecta = null;
String tipo_nivel = null;
String nombre = null;
// Realizar la conexion SQL
cn = DriverManager.getConnection("jdbc:mysql://localhost/servicio_tecnico", "root", "");
// Ejecutar la consulta
pst = cn.prepareStatement("SELECT username,password,tipo_nivel,nombre_usuario FROM usuarios WHERE username = ?");
pst.setString(1, usuario )
rs = pst.executeQuery();
// Solo se obtiene el primer registro (Si existe)
if (rs.next()) {
usuarioCorrecto = rs.getString(1);
passwordCorrecta = rs.getString(2);
tipo_nivel = rs.getString(3);
nombre = rs.getString(4);
}
// Si se ha obtenido un usuario y password y ademas esta es coincidente
if (usuarioCorrecto !=null && password!=null && password.equals(passwordCorrecta)) {
if(tipo_nivel!=null){
if (tipo_nivel.equals("Administrador")) {
ventanaMenu vm = new ventanaMenu();
this.setVisible(false);
vm.setVisible(true);
} else if (tipo_nivel.equals("Capturista")) {
} else if (tipo_nivel.equals("Tecnico")) {
}
}
} else {
JOptionPane.showMessageDialog(null, "Usuario o contraseña incorrectos");
}
} catch (Exception e) {
JOptionPane.showMessageDialog(null, "Error " + e);
} finally {
// Cierre de recursos (si se usa java7 se puede usar un try-with-resources
if(rs!=null){
rs.close();
}
if(pst!=null){
pst.close();
}
if(cn!=null){
cn.close();
}
}
Tambien seria recomendable no usar la clave en plano en la base de datos ya que cualquiera que pueda acceder a ella podra ver la clave del usuario. Seria recomendable cifrar la clave con algún cifrado de Mysql (podriamos resolverlo con una SQL mas completa) o bien algun método ajeno como usar una funcion SHA256 o MD5 en Java
Por ultimo una gestión de la conexión y el acceso a datos mediante una clase especializada puede mejorar aun mas el funcionamiento y mantenimiento futuros. Programar toda la logica en un botón, incluido abrir el acceso a datos es un mal diseño de software que puede acarrear problemas en un futuro.
En tu ejemplo tienes la logica en el metodo:
button_okActionPerformed(java.awt.event.ActionEvent evt)
que es un metodo de una acción de un botón donde se realiza todo el proceso.
Lo ideal es que este proceso solo realizara una llamada a un metodo de login de una clase y que esta contenga toda la logica.
Ejemplo:
private void button_okActionPerformed(java.awt.event.ActionEvent evt) {
String usuario = txt_usuario.getText();
String clave = txt_pass.getText();
Usuario usuario = controlador.login(usuario,clave);
if(usuario == null){
JOptionPane.showMessageDialog(null, "Usuario o contraseña incorrectos")
}
}
Donde "Usuario" seria un objeto java (un POJO) con la información del usuario y "controlador" seria un objeto con el codigo de acceso a datos y métodos como "login" para realizar el acceso
Nota: El codigo esta realizado sin conocer la versión de Java por lo que según la versión algunas cosas podrian ser mejor implementadas