[Java] Entra in un if dove non dovrebbe entrare -.-'

[StErMiNeiToR]

import javax.swing.*;

import java.awt.BorderLayout;
import java.awt.Frame;
import java.awt.GridLayout;
import java.awt.event.ActionEvent;
import java.awt.event.ActionListener;
import java.io.IOException;


import javax.swing.JButton;
import javax.swing.JFrame;
import javax.swing.JPanel;
import javax.swing.JTextField;

/*
 * 
 *  Program created and coded by `StErMi
 *  www.tdoportal.it
 *  mail: [email protected]
 *  for bugs or info contact me.
 * 
 * 
 */

public class interfaccia extends Frame implements ActionListener
{
	int valuta = 0;
	JTextField importo = new JTextField("");
	JLabel conv = new JLabel("");
	JFrame f = new JFrame("Euro Convertitore");
	JPanel p = new JPanel();
	JPanel p2 = new JPanel();
	JRadioButton euro = new JRadioButton("€");
	JRadioButton lire = new JRadioButton("£");
	JButton converti = new JButton("Converti");
	JButton riavvia = new JButton ("Riavvia");
	ButtonGroup group = new ButtonGroup();
	
	
	public interfaccia()
	{
		group.add(euro);
		group.add(lire);
		
		
		
		// setto i layout
		f.setLayout(new BorderLayout());
		f.setSize(250,250);
		p.setLayout(new GridLayout(4,2,10,10));
		f.getContentPane().add(p, "West");
		
		// aggiungo gli oggetti
		p.add( new JLabel("Importo: "));
		p.add(importo);
		p.add(euro);
		p.add(lire);
		p.add(converti);
		p.add(riavvia);
		p.add( new JLabel("Convertito: "));
		p.add(conv);

		// azioni -> bottone
		converti.addActionListener(this);
		euro.addActionListener(this);
		lire.addActionListener(this);
		riavvia.addActionListener(this);
		
		// proprietà finistra
		f.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
		f.setVisible(true);
	}

	public static void main(String[] args) throws IOException
	{
		interfaccia main = new interfaccia();
		
	}
	public void	actionPerformed (ActionEvent e  )
	{
		double val = 0;
		String v = "";
		String bottone = "";
		bottone = e.getActionCommand();
		
		if ( euro.isSelected() ); {
			valuta = 2;
		}
		
		if ( lire.isSelected()) {
			valuta = 1;
		}
		if ( !( euro.isSelected() ) && !( lire.isSelected() ) ) {
			conv.setText("Scegli una valuta");
			valuta = 0;
		}
		if  ( bottone.equals("Converti") ) {
			if ( ( valuta != 0 ) && ( importo.getText() != "" ) ) {
			v = importo.getText();
			val = Integer.parseInt(v);
			if ( valuta == 1 ) {
				val *= 1936.27;
				conv.setText( "" + val + " £" );
			} else if ( valuta == 2 ) {
				val /= 1936.27;
				conv.setText( "" + val + " €" );
			}
		}
		}
		
		if ( bottone.equals("Riavvia") ) {
			conv.setText("");
			importo.setText("");
		}
	}
}

Come mai entra qua? if ( ( valuta != 0 ) && ( importo.getText() != "" ) )

provate a non immettere nulla nella casella, a selezionare euro o lire e poi a convertire

[end.is.forever]

Prova a sostituire con:
if ( ( valuta != 0 ) && ( !importo.getText().equals( "" ) ) )
oppure più semplicemente:
if ( ( valuta != 0 ) && ( importo.getText().length() > 0 ) )

Non penso che l'implementazione default dell'operatore di uguaglianza faccia uso di equals, per cui lo devi chiamare esplicitamente e non usare != che altrimenti fa un confronto dei riferimenti.

Con la seconda versione poi eviti di istanziare ogni volta una stringa inutile da distruggere subito dopo.

[StErMiNeiToR]

grazie mille.

altre tre cose:

1) come faccio a far si che il layout si ridimensioni in automatico in base alla grandezza dei bottoni? come mai sono subito appiccicati ai bordi?
2) come faccio a immetere un double con solo 2 decimali in una textarea?
3) come faccio a rimettere di default il JRadioButton ? ovvero che non siano pigiati?

grazie mille