Il tokenizer della stringa in Java dà un'uscita errata.Hai bisogno di aiuto (probabilmente una soluzione facile)

Question

Il mio codice è fondamentalmente per trovare il numero di parole in una stringa (sì, davvero semplice, ma non funziona). Ho provato più metodi e ho provato alterare il codice un po 'di volte. Qualcuno può spiegare il mio perché questo codice non funziona e se è possibile darmi un codice migliore / di lavoro.

 StringTokenizer stspace = new StringTokenizer(sent, " .?");
    int nTokenSpace = stspace.countTokens();



    String mspace[] = new String[nTokenSpace];


    for(int i=0;i<nTokenSpace;i++)
    {
        mspace[i] = stspace.nextToken();


    }
  for(int i=0;i<nToken;i++)
    {
       String k = mspace[i];
       int freq = 0;
       for(int x=0;x<nTokenSpace;x++)
       {
           if(k==mspace[x])
           {
               freq++;
            }
        }
        System.out.println(k+"\t"+freq);
    }

.

L'output richiesto per "My. Nome. è. Anon. È. Anon."dovrebbe essere

My  1
Name 1
Is 2
Anon 2

But I get 
My  1
Name    1
Is  1
Anon    1
Is  1
Anon    1

.

Ho una sensazione che ci sia un errore ovvio qui ma non lo vedo (hai bisogno di nuovi occhi>.>)

Answer 1

Il tuo errore è qui:

if(k==mspace[x])

.

È necessario utilizzare

if (k.equals(mspace[x]))

.

Per confrontare Strings.

Se non si desidera i duplicati nell'output, è possibile aggiungere token a un Set prima del loop:

Set<String> uniqueTokens = new LinkedHashSet<String>(Arrays.asList(mspace));
for (String k : uniqueTokens) {
    int freq = 0;
    for (int x = 0; x < nTokenSpace; x++) {
        if (k.equals(mspace[x])) {
            freq++;
        }
    }
    System.out.println(k + "\t" + freq);
}

.

Answer 2

Confronto di stringhe

Sostituisci

 if(k==mspace[x])
       {
           freq++;
        }

.

con

 if(k.equals(mspace[x]))
       {
           freq++;
        }

.

Answer 3

Hai questa riga:

if(k==mspace[x])

.

Ciò significa che si confrontano due oggetti e devono essere esattamente lo stesso riferimento in memoria, non identici.Quello che stai cercando è due oggetti identici.Per farlo, devi confrontare gli oggetti con .equals(Object).

if (k.equals(mspace[x]))

.

Answer 4

Solo per aggiungere le altre risposte qui, credo che tu sia anche bisogno di rimuovere duplicati dall'elenco o dai risultati saranno come:

My  1
Name    1
Is  2
Anon    2
Is  2
Anon    2

.

Quindi il tuo blocco IF dovrebbe essere:

for(int i=0;i<nToken;i++)
{

   String k = mspace[i];

   if (k.equals(""))
       continue;

   int freq = 0;
   for(int x=0;x<nTokenSpace;x++)
   {
       if(k.equals(mspace[x]) )
       {
             freq++;
             mspace[x] = "";
        }
    }
    System.out.println(k+"\t"+freq);
}

.

Answer 5

Perché non usi le classi di raccolta.Ecco il codice.

import java.io.*;
import java.util.*;
public class HelloWorld{

     public static void main(String []args)
     {
        String givenstring="My. Name. Is. Anon. Is. Anon.";
         String[] words=givenstring.split(" ");

         ArrayList<String> arr=new ArrayList<String>();
         for(int i=0;i<words.length;i++)
          arr.add(words[i]);

        while(arr.size()!=0)
         {

             String word=arr.get(0);
             int frequency=Collections.frequency(arr,word);
             arr.removeAll(Collections.singleton(word));
             System.out.println(word+frequency);
          }
     }
}

.

Spero che aiuti ..