Zmienne lokalne, do których odwołuje się wyrażenie lambda, muszą być ostateczne lub efektywne.

Question

Mam program JavaFX 8 (dla JavaFXPorts cross platfrom), który został w znacznej części sformułowany w ramce, aby zrobić to, co chcę, ale powstał jeden krok. Program odczytuje plik tekstowy, zlicza linie, aby ustalić losowy zakres, wybiera losową liczbę z tego zakresu i odczytuje tę linię w celu wyświetlenia.

The error is: local variables referenced from a lambda expression must be final or effectively final 
     button.setOnAction(e -> l.setText(readln2)); 

jestem trochę nowy w Javie, ale wydaje się, czy mogę użyć lambda albo nie mieć kolejnej linii losowe wyświetlanie w Label l, mój button.setOnAction(e -> l.setText(readln2)); linia spodziewa wartości statycznej.

Jakieś pomysły, w jaki sposób mogę poprawić to, co mam, aby po prostu nacisnąć następną wartość wyświetlacza var readln2 za każdym razem, gdy naciskam przycisk na ekranie?

góry dzięki i tu jest mój kod:

String readln2 = null; 
in = new BufferedReader(new FileReader("/temp/mantra.txt")); 
long linecnt = in.lines().count(); 
int linenum = rand1.nextInt((int) (linecnt - Low)) + Low; 
try { 
    //open a bufferedReader to file 
    in = new BufferedReader(new FileReader("/temp/mantra.txt")); 

    while (linenum > 0) { 
     //read the next line until the specific line is found 
     readln2 = in.readLine(); 
     linenum--; 
    } 

    in.close(); 
} catch (IOException e) { 
    System.out.println("There was a problem:" + e); 
} 

Button button = new Button("Click the Button"); 
button.setOnAction(e -> l.setText(readln2)); 
// error: local variables referenced from a lambda expression must be final or effectively final 

Answer 1

można po prostu skopiować wartość readln2 do zmiennej final:

final String labelText = readln2 ; 
    Button button = new Button("Click the Button"); 
    button.setOnAction(e -> l.setText(labelText)); 

Jeśli chcesz, aby pobrać nową linię losowo za każdym razem, można buforować kierunki zainteresowania i wybrać losowo jeden w obsługi zdarzeń:

Button button = new Button("Click the button"); 
Label l = new Label(); 
try { 
    List<String> lines = Files.lines(Paths.get("/temp/mantra.txt")) 
     .skip(low) 
     .limit(high - low) 
     .collect(Collectors.toList()); 
    Random rng = new Random(); 
    button.setOnAction(evt -> l.setText(lines.get(rng.nextInt(lines.size())))); 
} catch (IOException exc) { 
    exc.printStackTrace(); 
} 
// ... 

Albo możesz po prostu ponownie przeczytać plik w module obsługi zdarzeń. Pierwsza technika jest (dużo) szybsza, ale może pochłaniać dużo pamięci; drugi nie zapisuje w pamięci żadnej zawartości pliku, ale czyta plik po każdym naciśnięciu przycisku, co może spowodować brak reakcji interfejsu użytkownika.

Błąd, który zasadniczo wyjaśnił, co było nie tak: jedynymi zmiennymi lokalnymi, do których można uzyskać dostęp z wnętrza wyrażenia lambda są: final (zadeklarowany jako final, co oznacza, że ​​muszą zostać przypisane wartości dokładnie raz) lub "skutecznie ostateczny" (co w zasadzie oznacza, że ​​możesz je uczynić ostatecznymi bez żadnych innych zmian w kodzie).

Twój kod nie kompiluje się, ponieważ readln2 ma przypisaną wartość wiele razy (wewnątrz pętli), więc nie można jej oznaczyć jako final. W związku z tym nie można uzyskać do niego dostępu w wyrażeniu lambda. W powyższym kodzie jedynymi zmiennymi dostępnymi w lambda są l, lines i rng, które wszystkie są "skutecznie końcowe", ponieważ są przypisane wartości dokładnie jeden raz.(Możesz zadeklarować je jako ostateczne, a kod nadal się kompiluje.)

Answer 2

Błąd napotkali Państwo oznacza, że ​​każda zmienna, że ​​masz dostęp do wnętrza ciała wyrażeń lambda musi być ostateczna lub skutecznie finału. Różnicy, zobacz tę odpowiedź tutaj: Difference between final and effectively final

Problem w kodzie jest następująca zmienna

String readln2 = null; 

Zmienna zostanie uznany i przypisany później, kompilator nie może wykryć, jeśli zostanie przydzielony jeden lub wiele razy, więc nie jest to skutecznie ostateczne.

Najprostszym sposobem rozwiązania tego problemu jest użycie obiektu opakowania, w tym przypadku StringProperty zamiast String. Owijka ta zostanie przydzielony tylko raz, a więc jest skutecznie końcowy:

StringProperty readln2 = new SimpleStringProperty(); 
readln2.set(in.readLine()); 
button.setOnAction(e -> l.setText(readln2.get())); 

ja skrócony kod, aby pokazać tylko odpowiednie części ..

Answer 3

Regularnie przekazuję obiekt zewnętrzny do implementacji interfejsu w następujący sposób: 1. Utwórz uchwyt obiektu, 2. Ustaw ten uchwyt obiektu w żądanym stanie, 3. Zmień wewnętrzne zmienne w obiekcie właściciela, 4. Pobierz te zmienne i użyj ich.

Oto jeden przykład z Vaadin:

Object holder : 
    public class ObjectHolder<T> { 
    private T obj; 
    public ObjectHolder(T obj) { 
     this.obj = obj; 
    } 
    public T get() { 
     return obj; 
    } 
    public void set(T obj) { 
     this.obj = obj; 
    } 
} 

Chcę przekazać podpisy przycisk zewnętrznie zdefiniowane tak:

String[] bCaption = new String[]{"Start", "Stop", "Restart", "Status"}; 
String[] commOpt = bCaption; 

Następny mam dla pętli, a chcesz utworzyć przyciski dynamicznie i przekazuj wartości w następujący sposób:

for (Integer i = 0; i < bCaption.length; i++) { 
    ObjectHolder<Integer> indeks = new ObjectHolder<>(i); 
    b[i] = new Button(bCaption[i], 
     (Button.ClickEvent e) -> { 
      remoteCommand.execute(
       cred, 
       adresaServera, 
       comm + " " + commOpt[indeks.get()].toLowerCase() 
      ); 
     } 
     ); 

     b[i].setWidth(70, Unit.PIXELS); 
     commandHL.addComponent(b[i]); 
     commandHL.setComponentAlignment(b[i], Alignment.MIDDLE_CENTER); 
    } 

Mam nadzieję, że to pomoże ..