Herausforderung mit Java string.indexOf

[barnti]

Hallo zusammen,

ich versuche mit an einer Java.Aufgabe.

Code:

public static int INDEX(String string, String element) {
       int temp = -1;
       if (string == null || element == null)
          return -1;
       else
          temp = string.indexOf(element);
          System.out.println("String_1: " + string);
          System.out.println("Element: " + element);
          return temp;

Der output:
String1_: String_1, String_2
Element: String_1

und der Vergleich endet mit -1.

Kann mich jemand aufklären, was ich hier übersehe?

[Der schöne Günther]

Nichts, du hast dich irgendwo vertippt (wie in deinem geposteten Ausschnitt mehrere male) - z.B. in deinem String "Element" " String_1" stehen statt "String_1" oder so etwas.

Code:

MeinBeispiel.INDEX("String_1, String_2", "String_1")

liefert für mich ganz normal

Delphi-Quellcode:

0

.

[barnti]

Hallo Günther,

danke für Dein Feedback. Die Tipper habe ich korrigiert, sorry dafür. Nein, an den Tippern liegt es nicht. Vielleicht hat es etwas mit der Codierung der Strings zu tun? In den Ausgaben sehen die Strings gleich aus. Ich bekomme aber trotzdem ein -1 zurück.

[dGeek]

Hi!

Kleiner Tippfehler: System.out.println("String_1: + "string);

Folgendermaßen funktioniert es:

Code:

public static int INDEX(String string, String element) {
  int temp = -1;
      
  if (string == null || element == null)
    return -1;
  else {
    temp = string.indexOf(element);
    System.out.println("String_1: " + string);
    System.out.println("Element: " + element);
  }
      
  return temp;
}
 
public static void main (String[] args) {
  System.out.println("temp: " + INDEX("String_1, String_2", "String_1"));
}

Ergibt:

Zitat:

String_1: String_1, String_2
Element: String_1
temp: 0

[barnti]

Ok,

danke auch den Tipper habe ich beseitigt. Ändert leider nichts an dem Ergebnis: temp = -1

[Der schöne Günther]

Wir sind trotzdem zwei gegen einen 8-). Groß/Kleinschreibung der reingesteckten Strings vielleicht?

Kannst du einmal ein minimales, lauffähiges Programm einstellen?

[Dejan Vu]

Na, woran wird es dann wohl liegen? Also wenn ein Programm eigentlich läuft, nur bei Dir nicht? Vielleicht doch an den Eingabedaten? Versuch es mal mit 'A' und 'A B' und dann, indem Du den Inhalt des einen Strings mit Copy&Paste in den anderen kopierst...

[barnti]

Hallo Dekan Vu,

danke für Deine Anregung. Ja, davon gehe ich auch aus. Der Code zeigt ein Beispiel. Für einige der Daten aus meinem Spreadsheet funktioniert es für andere nicht. Daher die Anmerkung/Frage, ob es vielleicht etwas mit der Codierung des Strings (Element) zu tun hat.

[barnti]

Hallo,

ich habe meinen Code geändert und statt indexOf verwende ich contains. Damit funktioniert es. Verstanden habe ich es nicht...

Danke!

[dGeek]

Zitat:

Zitat von barnti (Beitrag 1275031)

Hallo,

ich habe meinen Code geändert und statt indexOf verwende ich contains. Damit funktioniert es. Verstanden habe ich es nicht...

Danke!

So anzufangen ist nicht gut ;) Fehler muss man suchen und den Grund für die nicht-Funktionalität verstehen.

Hier ein funktionierender Code:

Code:

/* package whatever; // don't place package name! */

import java.util.*;
import java.lang.*;
import java.io.*;

/* Name of the class has to be "Main" only if the class is public. */
class Ideone
{
   public static int INDEX(String string, String element) {
        int temp = -1;
      
        if (string == null || element == null)
          return -1;
        else {
          temp = string.indexOf(element);
          System.out.println("String_1: " + string);
          System.out.println("Element: " + element);
        }
      
        return temp;
   }
   
   public static void main (String[] args) throws java.lang.Exception {
      String sInput = "";
      BufferedReader aInputBuffer = new BufferedReader(new InputStreamReader(System.in));
      
      System.out.print("Bitte String eingeben: ");
      sInput = aInputBuffer.readLine();
 
      System.out.println("Ergebnisse");
      System.out.println("temp: " + INDEX("String_1, String_2", sInput));
   }
}