He simplificado el sistema y reordenado un poco tu código. Probado en windows funciona bien.
Esta es la clase con el main
public class TestCrearDirectorio {
/**
* @param args the command line arguments
*/
public static void main( String[] args ) {
new Xxx();
}
} //class
Clase que hace el trabajo:
//imports
import java.io.File;
import javax.swing.*;
public class Xxx extends JFrame {
private File ruta = new File( "test/" );
private File carpeta;
private File archivo;
private String opp;
/**
* Construct
*
*/
public Xxx() {
super( "Prueba crear directorio" ); //set tittle
setDefaultCloseOperation( EXIT_ON_CLOSE );
setSize( 300, 200 );
//La siguiente llamada reemplaza a la que se produciría sobre
//el método lanzado por el evento, por ejemplo al pulsar un
//botón, para el ejemplo pongo solo la parte que interesa
execute();
}
private void execute() {
opp = JOptionPane.showInputDialog( "Introduce el nombre de la bd" );
archivo = new File( ruta, "BASES.db" );
carpeta = new File( ruta, opp );
//Traza
System.out.println( "archivo -> " + archivo );
System.out.println( "carpeta -> " + carpeta );
crearCarpeta();
crearFichero();
}
private void crearCarpeta() {
//La carpeta NO existe -> intenta crearla
if ( ! carpeta.exists( )) {
try {
carpeta.mkdir();
} catch (Exception e) {
JOptionPane.showMessageDialog(
null,
"Error creando carpeta."
);
System.err.println( e.toString( ));
}
//La carpeta YA existe
} else {
JOptionPane.showMessageDialog(
null,
"La carpeta ya existe."
);
}
}
private void crearFichero() {
//El archivo NO existe -> intenta crearlo
if ( ! archivo.exists( )) {
try {
archivo.createNewFile();
} catch( Exception e ) {
JOptionPane.showMessageDialog(
null,
"Error creando archivo."
);
System.err.println( e.toString( ));
}
} else {
JOptionPane.showMessageDialog(
null,
"El archivo ya existe."
);
}
}
} //class
Lo que vi raro en tu código es el orden de anidado try-catch con el de los condicionales y que preguntas dos veces la misma condición en cada bloque. Primero, cuando tratas de crear la carpeta, todo esta dentro del try-catch, incluso la pregunta de si ya existe la carpeta. Para crear el fichero el try-cath incluye solo la creación del fichero. Con respecto a esto, como no pones nada en el catch, no estas recibiendo ninguna información sobre lo que falla dentro de estas estructuras, solo evitando que se lancen excepciones, así tu código falla en silencio, y has acabado aquí preguntando.
Luego los condicionales, tipo:
a es 1
si a es diferente de 1 hacer X
si a es igual a 1 hacer Z
podrías expresarlos así:
int a = 1;
if ( a != 1 ) {
//hacer X
}
if ( a == 1 ) {
//hacer Z
}
pero si usas el else, como solo hay dos posibles respuestas en este caso, o a valo 1 o no vale 1, sería:
int a = 1;
if ( a != 1 ) {
//hacer X
} else {
//hacer Z
}
Si no entra en el bloque del if (a no es 1) entra en el else (a es 1). Volver a preguntar esta de más y a más código más posibilidad de fallos, más consumo de recursos, código menos limpio, más díficil de mantener... vamos que es malo, como meterse de cabeza en una tinaja si estas sólo en casa.