Co tu należy poprawić?

Question

package sad;

import java.awt.List;
import java.io.BufferedWriter;
import java.io.FileNotFoundException;
import java.io.FileReader;
import java.io.FileWriter;
import java.io.IOException;
import java.math.BigInteger;
import java.util.ArrayList;
import java.util.Random;
import java.util.Scanner;
/// program ma dzielic na slowa, co tu nalezy poprawic?
public class asd {
	static int i=1;
	static int nextI=0;
	static int a=0;
	static String[] tablica= new String[10];
    static String[] podzielNaSlowa(String tekst){
		do{
			if(Character.isWhitespace(tekst.charAt(i))){
				tablica[a]=tekst.substring(nextI, i);
				nextI=i;
				a++;
			}
			i++;
		}while(i<tekst.length());
    	
    	return tablica;
    }
	
public static void main(String[] args){
	System.out.print(podzielNaSlowa("To ma zostac podzielone"));
 }
}

 

Answer 1

Metoda w twoim kodzie: podzielNaSlowa("tekst do podzielenia") zwracała tablicę.

W związku z tym, przy wywołaniu metody System.out.print(tablica) zostawała wywoływana metoda .toString() tej tablicy - przez to dostawałeś brzydki szlaczek jakichś liter.

Aby wyświetlić słowa w tablicy musisz po niej iterować jakąś pętlą. Ja w swoim przykładzie uzyłem pętli for.

Dodatkowo Twój warunek w ifie był niedopracowany. Nie wrzucał do tablicy ostatniego słowa tekstu, ponieważ po ostatnim słowie nie ma spacji.

W ładny sposób można podzielić tekst przy pomocy metody Stringa: .split("znaki, które mają być usuniętę z tekstu i jednocześnie pokazują, w którym miejscu tekst należy podzielić").

/*
 * Program dzieli tekst na części
 */
public class asd {
	// Nie inicjujemy tablicy (= new String[10]), ponieważ metoda .split() zwraca już utworzoną tablicę
    static String[] tablica;
    
    // Metoda dzielącą tekst na części
    static String[] podzielNaSlowa(String tekst){
    	// Dzielimy tekst w miejscu gdzie jest spacja metodą .split.
    	// Metoda ta zwraca tablicę Stringów,
    	// którą przypisujemy do zmiennej tablica
    	tablica = tekst.split(" ");
        
    	// Zwracamy tablicę z podzielonymi słowami
        return tablica;
    }
     
public static void main(String[] args){
	// Metoda zwraca tablicę, którą zapisujemy do zmiennej
	String[] tablicaSłów = podzielNaSlowa("To ma zostac podzielone");
	
	// Iterujemy po tablicy pętlą for i wyświetlamy pokolei każde słowo
	for (int i = 0; i < tablicaSłów.length; i++) {
		System.out.println(tablicaSłów[i]);
	}
 }
}

Comments

Ewentualnie można użyć klasy StringTokenizer bo splitem podzieli uwzględniając jeden separator a np wyrazy możesz mieć po przecinkach itd.

http://docs.oracle.com/javase/7/docs/api/java/util/StringTokenizer.html

---

Dobrze, że o tym mówisz

Można też użyć wyrażenia regularnego w metodzie .split():

tekst.split("[, ]+");

Czyli w nawiasach kwadratowych wpisujemy znaki (w tym przykładzie przecinek i spację), które mają odzielać tekst, a znaczek plus mówi, że mogą one wystąpić raz lub więcej razy w tekscie

---

kurde bardzo często są gotowe metody do rozwiązania zadania, a ja się ******** w robienie czegoś od nowa, muszę zaznajomić się z dokumentacją. Dzięki za pomoc ;)))