problem with java rmi

[_rapt0r_]

I have a text box when I press a button it sends me a letter in character to a remote method in another class

private void jButton3ActionPerformed(java.awt.event.ActionEvent evt) {                                         
        
            // TODO add your handling code here:
                String leter = aposta.getText();
                char x = leter.charAt(0);
        try {
            ridle(x);
        } catch (NotBoundException ex) {
            ex.printStackTrace();
        }

public void ridle(char leter) throws NotBoundException {
  
     Forca fr = null;
          System.setProperty("java.security.policy", "C:/Users/koo/Documents/NetBeansProjects/JavaApplication3/src/javaapplication3/security.txt/");
		

		String servicename = "hangman" ;

		//Inicializar security manager
        if (System.getSecurityManager() == null) {
            System.setSecurityManager(new SecurityManager());
        }
        try {
                        Registry registry = LocateRegistry.getRegistry(Ip);
           
                fr = (Forca) registry.lookup(servicename);
            
            
                        
                        xx = fr.jogar(leter);
                     
        } catch (RemoteException ex) {
            Logger.getLogger(GUI.class.getName()).log(Level.SEVERE, null, ex);
        }
        
 }

the remote code

 public String jogar(char x) throws RemoteException {

        System.out.println ("numero de"+ palavra.length());
        
        char formaPalavra[] = new char[palavra.length()];
        String apresentaPalavra="\0";

        for(int y = 0; y < palavra.length(); y++){
            System.out.println ("numero de"+ palavra.length());
            formaPalavra[y] = '_';
            apresentaPalavra += formaPalavra[y] + " ";
            }

        
        for(byte i = 0; i < palavra.length();i++){
            System.out.println ("numero de"+ palavra.length());
                if(String.valueOf(palavra.charAt(i)).equalsIgnoreCase(Character.toString(x))){
                     formaPalavra[i] = palavra.charAt(0); //converte de String para char
         apresentaPalavra += formaPalavra[i] + " ";
        }else{
        apresentaPalavra += formaPalavra[i] + " ";
             }
           //apresentaPalavra2 = apresentaPalavra;
  
        
        
    }
        return apresentaPalavra;
    }

the problem is that returns me twice --- and of the letter that I put

[ozzyman]

This site is so dam buggy I posted an answer twice and both times it lost my answer. I cant be bothered to write it all over again but you need to add in a time restriction validation so that you cant 'double-post' within that time period.

[_rapt0r_]

i don´t understant the restriction validation :s

[ozzyman]

private long lastUpdate=0;
private final long quietTime = 5*1000; //5secs

public void jButton3ActionPerformed(java.awt.event.ActionEvent e) {
if (!isReady()) return;
lastUpdate = Calendar.getInstance().getTimeInMillis();
//code
...
}

private boolean isReady() {
return (Calendar.getInstance().getTimeInMillis()-lastUpdate >= quietTime);
}

[_rapt0r_]

doesn't work :s

[ozzyman]

Hmm... so jogar(char x) is being run twice on one click?

[_rapt0r_]

I think the problem is there, but would only go once the method and return the result once. and always run this method only when I find matching in jbutton ..

[ozzyman]

I think the problem is there, but would only go once the method and return the result once. and always run this method only when I find matching in jbutton ..

i'm sorry i couldnt make much sense of that.

but 'always run this method when...' what?

Is there some more code which causes this method to run automatically?

[_rapt0r_]

the method don´t run automatically, he runs when I run this button..

private void jButton3ActionPerformed(java.awt.event.ActionEvent evt) {

// TODO add your handling code here:
String leter = aposta.getText();
char x = leter.charAt(0);
try {
ridle(x);
} catch (NotBoundException ex) {
ex.printStackTrace();
}

I no longer have any code running jogar(char x) method, only the code in my first post

[ozzyman]

Are you sure it is returning twice? Or is it your for loop adding the char to output twice?

[_rapt0r_]

i don´t no, I've seen for the loop several times and can not see what the error: S

See this line?? System.out.println ("numero de"+ palavra.length()); return the number of letters

if the word is "play" and a put a "p" see the output


numero de4
numero de4
numero de4
numero de4
numero de4
numero de4
numero de4
numero de4
numero de4

[ozzyman]

Just to be sure print a String outside the loop to see how many times the method is called.

[_rapt0r_]

}
System.out.println ("the number of request"+ palavra.length());
return apresentaPalavra;
}

output
the number of request3

you're right, but do not see the problem in the code: S

[ozzyman]

Ok so lets look at your loop...

You have a search feature like this...
if (char y).equals(char x)

But if you find a match, you did not break the loop,
That allows there to be more than 1 result returned.

if you only want 1 result, you can take the first result only,
By breaking the loop like this:

for (...) {
  if (found) {
    //add to output
    ...
    break;
  else {
    //do something else
  }
}

[_rapt0r_]

already improved, but now I put a word with 3 and 4 returns


for(byte i = 0; i < palavra.length();i++){
System.out.println ("numero de"+ pala.length());
if(String.valueOf(pala.charAt(i)).equalsIgnoreCase (Character.toString(x))){
formaPalavra[i] = pala.charAt(0); //converte de String para char
apresentaPalavra += formaPalavra[i] + " ";
break;
}else{
apresentaPalavra += formaPalavra[i] + " ";
}
break;

[ozzyman]

Huh? So if you dont want to print anything else why are you adding stuff to output in your first loop?

Did you write any of this code yourself?

[_rapt0r_]

yes i write this code.. i found the problem
for(int y = 0; y < pala.length(); y++){
System.out.println ("numero de"+ pala.length());
formaPalavra[y] = '_';
apresentaPalavra += formaPalavra[y] + " ";
break;
}

for(byte i = 0; i < palavra.length();i++){
System.out.println ("numero de"+ pala.length());
if(String.valueOf(pala.charAt(i)).equalsIgnoreCase (Character.toString(x))){
formaPalavra[i] = pala.charAt(0); //converte de String para char
apresentaPalavra += formaPalavra[i] + " ";
break;
}else{
apresentaPalavra += formaPalavra[i] + " ";
}
break;

so I needed a break more..

tks for the help