Warum gibt dieser Code -1 zurück?

Question

Ich habe etwas Code und Zeile 33 gibt -1 zurück, weil es den Wert nicht finden konnte. Aber warum findet es es nicht ?! Ich denke, es könnte etwas mit Char [] zu tun haben, aber ich bin mir hier nicht wirklich sicher.

Es könnte auch wegen der Linie 31 (int temp) aber ich weiß es wirklich nicht.

import java.util.Arrays; 
import java.util.HashMap; 
import java.util.Scanner; 

class Main { 

public static void main(String args[]) { 

HashMap<Integer, String> map = new HashMap<Integer, String>(); 

System.out.println("Enter your string to cipher: "); 

Scanner s = new Scanner(System.in); 
String str = s.nextLine(); 

char[] letters = {'a', 'b', 'c', 'd', 'e', 'f', 'g', 'h', 'i', 'j', 'k', 'l', 'm', 'n', 'o', 'p', 'q', 'r', 's', 't', 'u', 'v', 'w', 'x', 'y', 'z'}; 
Integer[] numbers = {0, 1, 2, 3, 4, 5, 6, 7, 8, 9, 10, 11, 12, 13, 14, 15, 16, 17, 18, 19, 20, 21, 22, 23, 24, 25}; 

if (str instanceof String) { 
    System.out.println("Input received: OK."); 
} else { 
    System.out.println("Error! You cannot input numbers. Exiting program..."); 
    System.exit(1); 
} 

for (int i = 0; i < str.length(); i++) { 
    char l = str.charAt(i); 

    System.out.println(l); 

    int temp = Arrays.asList(letters).indexOf(l); 

    System.out.println(temp); 


    } 


    } 

} 

Answer 1

Ändern Sie den Typ der Variable ‚Buchstaben‘ zu Character[] und es wird funktionieren.

Bonus Kleiner Optimierungsvorschlag: Verschieben Sie diese Zeile int temp = Arrays.asList(letters).indexOf(l) außerhalb der Schleife, da Sie bei jeder Iteration eine Liste erstellen.

auch entscheiden, ob Sie generische Sammlungen mit Arrays oder Arrays wie char[] usw.