[Java] NullPointerException przy wczytywaniu pliku tekstowego do listy.

Question

Witam ponownie, dziś mam problem z wczytywaniem pliku. Kod: 

import java.io.BufferedReader;
import java.io.FileReader;
import java.io.IOException;
import java.util.ArrayList;

public class Zad1 {

	public static void main(String[] args) throws IOException{

		 BufferedReader x = new BufferedReader(new FileReader("panTadeusz.txt"));

	        String schowek = "";

	        ArrayList<String> list = new ArrayList<>();

	        do{
	        	schowek = x.readLine().toLowerCase();
	        	list.add(schowek);
	        	}while(schowek!=null);
	        x.close();

	        ArrayList<String> alfabet = new ArrayList<>();
	        ArrayList<Integer> iloscWystapien = new ArrayList<>();
	        ArrayList<String> wykres = new ArrayList<>();

	        schowek = "aąbcćdeęfghijklłmnńoóprsśtuwyzźż";

	        //sporządzanie listy znaków
	        for(int i=0; i<schowek.length(); i++){
	            alfabet.add(Character.toString(schowek.charAt(i)));
	            iloscWystapien.add(0);
	        }

	        for(int i=0; i<list.size();i++){
	            schowek = list.get(i);
	            for(int j=0; j<schowek.length(); j++){
	                String literka = Character.toString(schowek.charAt(j));
	                for(int k=0; k<alfabet.size(); k++){
	                    if(literka.equals(alfabet.get(k))){
	                        iloscWystapien.set(k, iloscWystapien.get(k)+1);
	                    }
	                }
	            }
	        }

	        System.out.println(iloscWystapien);
	
	}
}

Plik tekstowy to treść I księgi "Pana Tadeusza", uprzednio pozbawiony pustych linijek (między wierszami). 

 

Info z konsoli:

Exception in thread "main" java.lang.NullPointerException
    at Zad1.main(Zad1.java:18)- linijka:                 list.add(schowek);
 

Answer 1

Dlatego że na początku pobierasz potem dodajesz a na koncu sprawdzasz czy to nie był null. Jak Ci wystąpi null to i tak go dodajesz.

Comments

A no faktycznie, wielkie dzięki. 

---

Dalej nie działa. 

 

 schowek = x.readLine().toLowerCase();
	        do{
	        	list.add(schowek);
	        	schowek = x.readLine().toLowerCase();
	        }while(schowek!=null);
	        x.close();

Teraz taki kod dałem. Pierw wczytuję 1 linijkę a w pętli dodaję starą, wczytuję nową i patrzę, czy nie jest nullem. Jak jest, wychodzę z pętli, jak nie to jeszcze raz. 

---

A teraz sam znalazłem błąd. Próbuję na nową linijkę nałożyć metodę toLowerCase() nie sprawdzając wcześniej, czy nie jest nullem. ;)

---

mozesz zrobić coś w stylu.

for(String temp = x.readLine(); x!=null; temp=x.readLine()){
   list.add(temp.toLowerCase();
}

 

---

A tak swoją drogą jak iterujesz po całej liscie to polecam używać pętli for each.

---

Już to załatwilem trochę prościej (według mnie). :)

 

schowek = x.readLine();
	        do{
	        	schowek = schowek.toLowerCase();
	        	list.add(schowek);
	        	schowek = x.readLine();
	        }while(schowek!=null);
	        x.close();

Zamiany z dużych na małe dokonuje po uprzednim zatwierdzeniu, że moja linijka nie jest nullem. :) Ale dzięki za kod, zawsze to jakaś nowa wiedza. 

---

A spróbuję, nie ukrywam, ze dopiero sie uczę programować. :)

---

:) Powodzenia w dalszej nauce

---

Nie mam zbyt dużego doświadczenia w programowaniu, ale zamiast do while, użyj po prostu while, bo do while zawsze wykona się przynajmniej raz, jeżeli ktoś poda pusty plik to już przy pierwszej próbie aplikacja się wywali :-)

---

Po prostu zrobię warunek sprawdzajacy czy to null, jak zrobię petlę while i wrzuce pierw ten bufferedreader to znowu będzie sytuacja, ze do listy chcę wcisnąć null'a, nim program sprawdzi czy wlaśnie nullem nie jest.

---

schowek = x.readLine();
		
                if(schowek!=null){
		do {
			schowek = schowek.toLowerCase();
			list.add(schowek);
			schowek = x.readLine();
		} while (schowek != null);
		
                x.close();
                }
                else{
			System.out.println("Nie można wczytywać pustego pliku!");
			System.exit(0);
		}

Teraz będzie OK

---

Raczej myślałem o takim przerobieniu Twojej pętli:

while (schowek != null) {
         schowek = schowek.toLowerCase();
         list.add(schowek);
         schowek = x.readLine();
}
x.close();

 

o ile nic nie pomyliłem ;-)

---

Jeżeli chcesz zastąpić mój cały kod, brakuje x.nextLine();, bez tego schowek jest nullem, pomija tą pętlę i powoduje problemy dalej.

---

package projekt;

import java.io.BufferedReader;
import java.io.FileReader;
import java.io.IOException;
import java.util.ArrayList;

public class Zad1 {

    public static void main(String[] args) throws IOException {

        BufferedReader x = new BufferedReader(new FileReader("C:\\Users\\Piotrek\\Desktop\\panTadeusz.txt"));

        String schowek = "";

        ArrayList<String> list = new ArrayList<>();
        
        schowek = x.readLine();
        
        while (schowek != null) {
            schowek.toLowerCase();
            list.add(schowek);
            schowek = x.readLine();
        };
        x.close();

        ArrayList<String> alfabet = new ArrayList<>();
        ArrayList<Integer> iloscWystapien = new ArrayList<>();
        ArrayList<String> wykres = new ArrayList<>();

        schowek = "aąbcćdeęfghijklłmnńoóprsśtuwyzźż";

        // sporządzanie listy znaków
        for (int i = 0; i < schowek.length(); i++) {
            alfabet.add(Character.toString(schowek.charAt(i)));
            iloscWystapien.add(0);
        }

        for (int i = 0; i < list.size(); i++) {
            schowek = list.get(i);
            for (int j = 0; j < schowek.length(); j++) {
                String literka = Character.toString(schowek.charAt(j));
                for (int k = 0; k < alfabet.size(); k++) {
                    if (literka.equals(alfabet.get(k))) {
                        iloscWystapien.set(k, iloscWystapien.get(k) + 1);
                    }
                }
            }
        }

        System.out.println(iloscWystapien);

    }
}

 

spróbuj teraz :-)

---

Zadziałać zadziała, ale nie widze większej róznicy między do while, a while ;) ogólnie w moich kodach dużo do while używam, jakoś wygodniej mi. ;)

---

Różnica jest taka, że gdy robisz

do {
   jakieś_instrukcje
} while (jakiś_warunek)

to jakieś_instrukcje zostaną wykonane co najmniej raz, nawet jeśli jakiś_warunek od początku nie jest spełniony, w przeciwieństwie do pętli

while (jakiś_warunek) {
   jakieś_instrukcje
}

w tej pętli, jeśli jakiś_warunek nie jest spełniony (jego wartość jest false) to jakieś_instrukcje nie zostaną wykonane ani razu :-)

wszystko zależy od tego, co potrzebujesz :-)

---

Też prawda. Moim zdaniem do while bardziej przejrzyście wyglada. :P No dobra, najważniejsze że działa. 

---

Tutaj w sumie nie chodzi o to czy bardziej przejrzyście wygląda, tylko o samą funkcjonalność, czasami będziesz potrzebował coś wykonać raz, nawet jeśli warunek nie będzie sprawdzony :-) np. będziesz chciał wylosować liczbę naturalną większą niż 1, ale tylko jedną, ale jednak losować chcesz  z całego zakresu integer, wtedy robisz pętlę

Random generator = new Random();
int x;
do {
  x =  generator.nextInt();
} while (x < 1)

to może nie jest najlepszy przykład, ale może chociaż trochę Ci przedstawi zastosowanie :-)

---

boolean sprawdzaj = false;

Random generator = new Random();
int x = 1;
do {
  x =  generator.nextInt();

if(x==1)
sprawdzaj = true;

} while (sprawdzaj == false)

Czyli mimo, że x = 1, to zalosuje tak? 

---

No tak, jak teraz wrzuciłeś w kodzie, to tak :-)

Mi jednak bardziej chodziło o to, że jeżeli nie zainicjalizowałbyś x nadając mu wartość 1, to losowanie w tej pętli na pewno raz by się odbyło :-) a potem to już w zależności od tego co się wylosuje.

---

No czyli trzeba uważać, dzięki za rady. ;)

---

Nie ma za co :-)