No, non va bene. Non stai programmando ad oggetti.
Ti ripeto nuovamente: tu sai che lavori con i file, in generale sai che le operazioni che puoi fare sono
1. load(InputStream stream)
2. save(OutputStream stream)
3. visualize(int numRows)
4. skip (int numRows)

queste ti definisco una interfaccia. Sai che vai ad usare un oggetto che è in grado di implementare tale comportamento.
Supponi di chiamare questo file IOManagement e questa è una interfaccia.
Sicuramente avrai almeno una IOManagementImpl che lavora pensando ad un file sotto tutto.


dentro lo switch ti limita a fare
codice:
switch (intValue){
	case 1: loadFile(); 
			break; 
	case 2: saveFile(); 
			break; 
			// continua tu
}
ora in questa fase TU SAI che lavori su file, quindi

codice:
IOManagement ioman = new IOManagementImpl(); 
 
FileInputStream stream = new FileInputStream(filename); 
load(stream);
e via dicendo.
Se tu domani volessi fare il caricamento da che so web page, i cambiamenti sarebbero minimi.
Ovviamente io ti ho descritto a grandi linee le implementazioni, a te i dettagli implementativi.

Ah, ogni implementazione una sua classe, il main serve SOLO a far partire il programma (al max a gestire questo menu)