Skip to main content
removeu 10 caracteres do conteúdo
Fonte Link
Isac
  • 25,5mil
  • 3
  • 31
  • 51

Para fazer com que funciona basta chamar o método scramble, que não está a ser chamado. Poderia faze-lo com:

public static void arrayword () {

   ...
   System.out.println(scramble(new Random(), random));
}

E apesar de já funcionar não está muito bom pois está a ser criado outro objeto Random quando já tinha sido criado um anteriormente.

Podemos rever isso e outros detalhes do código e transforma-lo em:

//objeto que gera os aleatorios agora é sempre o mesmo e apenas criado uma vez
static Random aleatorio = new Random();

public static void arraywordarrayWord () { //agora o nome em camel case

    String[] wordgame = { //inicialização do array mais simples assim
        "TEA",
        "COFFEE",
        "BOAT",
        "SEA",
        "SUN"
    };

    int idx = aleatorio.nextInt(wordgame.length);
    String palavraEscolhida = (wordgame[idx]);

    System.out.println(palavraEscolhida); //escrever a variavel anterior, a escolhida
    System.out.println(scramble(palavraEscolhida));
}

//agora não recebe o aleatorio, usa o mesmo da classe
public static String scramble(String inputString ) 
{
    // Convert your string into a simple char array:
    char[] a = inputString.toCharArray();

    // Scramble the letters using the standard Fisher-Yates shuffle, 
    for( int i=0 ; i<a.length ; i++ )
    {
        int j = aleatorio.nextInt(a.length); 

        //Mais Swaplegível lettersassim linha a linha
        char temp = a[i]; 
        a[i] = a[j];  
        a[j] = temp;
    }       

    return new String( a );
}

Exemplo a funcionar no Ideone

Para fazer com que funciona basta chamar o método scramble, que não está a ser chamado. Poderia faze-lo com:

public static void arrayword () {

   ...
   System.out.println(scramble(new Random(), random));
}

E apesar de já funcionar não está muito bom pois está a ser criado outro objeto Random quando já tinha sido criado um anteriormente.

Podemos rever isso e outros detalhes do código e transforma-lo em:

//objeto que gera os aleatorios agora é sempre o mesmo e apenas criado uma vez
static Random aleatorio = new Random();

public static void arrayword () {

    String[] wordgame = { //inicialização do array mais simples assim
        "TEA",
        "COFFEE",
        "BOAT",
        "SEA",
        "SUN"
    };

    int idx = aleatorio.nextInt(wordgame.length);
    String palavraEscolhida = (wordgame[idx]);

    System.out.println(palavraEscolhida); //escrever a variavel anterior, a escolhida
    System.out.println(scramble(palavraEscolhida));
}

//agora não recebe o aleatorio, usa o mesmo da classe
public static String scramble(String inputString ) 
{
    // Convert your string into a simple char array:
    char[] a = inputString.toCharArray();

    // Scramble the letters using the standard Fisher-Yates shuffle, 
    for( int i=0 ; i<a.length ; i++ )
    {
        int j = aleatorio.nextInt(a.length);
        // Swap letters
        char temp = a[i]; a[i] = a[j];  a[j] = temp;
    }       

    return new String( a );
}

Para fazer com que funciona basta chamar o método scramble, que não está a ser chamado. Poderia faze-lo com:

public static void arrayword () {

   ...
   System.out.println(scramble(new Random(), random));
}

E apesar de já funcionar não está muito bom pois está a ser criado outro objeto Random quando já tinha sido criado um anteriormente.

Podemos rever isso e outros detalhes do código e transforma-lo em:

//objeto que gera os aleatorios agora é sempre o mesmo e apenas criado uma vez
static Random aleatorio = new Random();

public static void arrayWord () { //agora o nome em camel case

    String[] wordgame = { //inicialização do array mais simples assim
        "TEA",
        "COFFEE",
        "BOAT",
        "SEA",
        "SUN"
    };

    int idx = aleatorio.nextInt(wordgame.length);
    String palavraEscolhida = (wordgame[idx]);

    System.out.println(palavraEscolhida); //escrever a variavel anterior, a escolhida
    System.out.println(scramble(palavraEscolhida));
}

//agora não recebe o aleatorio, usa o mesmo da classe
public static String scramble(String inputString ) 
{
    // Convert your string into a simple char array:
    char[] a = inputString.toCharArray();

    // Scramble the letters using the standard Fisher-Yates shuffle, 
    for( int i=0 ; i<a.length ; i++ )
    {
        int j = aleatorio.nextInt(a.length); 

        //Mais legível assim linha a linha
        char temp = a[i]; 
        a[i] = a[j];  
        a[j] = temp;
    }       

    return new String(a);
}

Exemplo a funcionar no Ideone

removeu 10 caracteres do conteúdo
Fonte Link
Isac
  • 25,5mil
  • 3
  • 31
  • 51

Para fazer com que funciona basta chamar o método scramble, que não está a ser chamado. Poderia faze-lo com:

public static void arrayword () {

   ...
   System.out.println(scramble(new Random(), random));
}

E apesar de já funcionar não está muito bom pois está a ser criado outro objeto Random quando já tinha sido criado um anteriormente.

Podemos rever isso e outros detalhes do código e transforma-lo em:

//objeto que gera os aleatorios agora é sempre o mesmo e apenas criado uma vez
static Random aleatorio = new Random();

public static void arrayword () {

    String[] wordgame = new String[]{ //inicialização do array mais simples assim
        "TEA",
        "COFFEE",
        "BOAT",
        "SEA",
        "SUN"
    };

    int idx = aleatorio.nextInt(wordgame.length);
    String palavraEscolhida = (wordgame[idx]);

    System.out.println(palavraEscolhida); //escrever a variavel anterior, a escolhida
    System.out.println(scramble(palavraEscolhida));
}

//agora não recebe o aleatorio, usa o mesmo da classe
public static String scramble(String inputString ) 
{
    // Convert your string into a simple char array:
    charchar[] a[]a = inputString.toCharArray();

    // Scramble the letters using the standard Fisher-Yates shuffle, 
    for( int i=0 ; i<a.length ; i++ )
    {
        int j = aleatorio.nextInt(a.length);
        // Swap letters
        char temp = a[i]; a[i] = a[j];  a[j] = temp;
    }       

    return new String( a );
}

Para fazer com que funciona basta chamar o método scramble, que não está a ser chamado. Poderia faze-lo com:

public static void arrayword () {

   ...
   System.out.println(scramble(new Random(), random));
}

E apesar de já funcionar não está muito bom pois está a ser criado outro objeto Random quando já tinha sido criado um anteriormente.

Podemos rever isso e outros detalhes do código e transforma-lo em:

//objeto que gera os aleatorios agora é sempre o mesmo e apenas criado uma vez
static Random aleatorio = new Random();

public static void arrayword () {

    String[] wordgame = new String[]{ //inicialização do array mais simples assim
        "TEA",
        "COFFEE",
        "BOAT",
        "SEA",
        "SUN"
    };

    int idx = aleatorio.nextInt(wordgame.length);
    String palavraEscolhida = (wordgame[idx]);

    System.out.println(palavraEscolhida); //escrever a variavel anterior, a escolhida
    System.out.println(scramble(palavraEscolhida));
}

//agora não recebe o aleatorio, usa o mesmo da classe
public static String scramble(String inputString ) 
{
    // Convert your string into a simple char array:
    char a[] = inputString.toCharArray();

    // Scramble the letters using the standard Fisher-Yates shuffle, 
    for( int i=0 ; i<a.length ; i++ )
    {
        int j = aleatorio.nextInt(a.length);
        // Swap letters
        char temp = a[i]; a[i] = a[j];  a[j] = temp;
    }       

    return new String( a );
}

Para fazer com que funciona basta chamar o método scramble, que não está a ser chamado. Poderia faze-lo com:

public static void arrayword () {

   ...
   System.out.println(scramble(new Random(), random));
}

E apesar de já funcionar não está muito bom pois está a ser criado outro objeto Random quando já tinha sido criado um anteriormente.

Podemos rever isso e outros detalhes do código e transforma-lo em:

//objeto que gera os aleatorios agora é sempre o mesmo e apenas criado uma vez
static Random aleatorio = new Random();

public static void arrayword () {

    String[] wordgame = { //inicialização do array mais simples assim
        "TEA",
        "COFFEE",
        "BOAT",
        "SEA",
        "SUN"
    };

    int idx = aleatorio.nextInt(wordgame.length);
    String palavraEscolhida = (wordgame[idx]);

    System.out.println(palavraEscolhida); //escrever a variavel anterior, a escolhida
    System.out.println(scramble(palavraEscolhida));
}

//agora não recebe o aleatorio, usa o mesmo da classe
public static String scramble(String inputString ) 
{
    // Convert your string into a simple char array:
    char[] a = inputString.toCharArray();

    // Scramble the letters using the standard Fisher-Yates shuffle, 
    for( int i=0 ; i<a.length ; i++ )
    {
        int j = aleatorio.nextInt(a.length);
        // Swap letters
        char temp = a[i]; a[i] = a[j];  a[j] = temp;
    }       

    return new String( a );
}
Fonte Link
Isac
  • 25,5mil
  • 3
  • 31
  • 51

Para fazer com que funciona basta chamar o método scramble, que não está a ser chamado. Poderia faze-lo com:

public static void arrayword () {

   ...
   System.out.println(scramble(new Random(), random));
}

E apesar de já funcionar não está muito bom pois está a ser criado outro objeto Random quando já tinha sido criado um anteriormente.

Podemos rever isso e outros detalhes do código e transforma-lo em:

//objeto que gera os aleatorios agora é sempre o mesmo e apenas criado uma vez
static Random aleatorio = new Random();

public static void arrayword () {

    String[] wordgame = new String[]{ //inicialização do array mais simples assim
        "TEA",
        "COFFEE",
        "BOAT",
        "SEA",
        "SUN"
    };

    int idx = aleatorio.nextInt(wordgame.length);
    String palavraEscolhida = (wordgame[idx]);

    System.out.println(palavraEscolhida); //escrever a variavel anterior, a escolhida
    System.out.println(scramble(palavraEscolhida));
}

//agora não recebe o aleatorio, usa o mesmo da classe
public static String scramble(String inputString ) 
{
    // Convert your string into a simple char array:
    char a[] = inputString.toCharArray();

    // Scramble the letters using the standard Fisher-Yates shuffle, 
    for( int i=0 ; i<a.length ; i++ )
    {
        int j = aleatorio.nextInt(a.length);
        // Swap letters
        char temp = a[i]; a[i] = a[j];  a[j] = temp;
    }       

    return new String( a );
}