Grazie per la risposta, alla fine quello corretto è così?

codice:
package frase;
import java.util.Scanner;
public class frase {
 public static void main(String[] args) {
  // TODO Auto-generated method stub
  @SuppressWarnings("resource")
  Scanner in=new Scanner(System.in);
  String frase=new String();
  System.out.println("Inserisci una frase");
  frase=in.nextLine();
  double percspazi,percvocali,perconsonanti,contspazi=0,contvocali=0,contconsonanti=0;
  int p;
  for(int i=0; i<frase.length();i++)
  {   
   p=frase.charAt(i);
   if (p== ' ')
   {
    contspazi++;
   }
   if((p=='a') || (p=='e') || (p=='i') || (p=='o') || (p=='u'))
   {
    contvocali++;
   }
   if((p=='b') || (p=='c') || (p=='d') || (p=='f') || (p=='g') || (p=='h') || (p=='j') || (p=='k') || (p=='l') || (p=='m') || (p=='n') || (p=='p') || (p=='q') || (p=='r') || (p=='s') || (p=='t') || (p=='v') || (p=='w') || (p=='x') || (p=='y') || (p=='z'))
   {
    contconsonanti++;
   }
  }
  percspazi=contspazi/(contspazi+contvocali+contconsonanti)*100;
  percvocali=contvocali/(contspazi+contvocali+contconsonanti)*100;
  perconsonanti=contconsonanti/(contspazi+contvocali+contconsonanti)*100;
  System.out.println("Percentuale spazi->"+percspazi);
  System.out.println("Percentuale vocali->"+percvocali);
  System.out.println("Percentuale consonanti->"+perconsonanti);
 }
}