• Najnowsze pytania
  • Bez odpowiedzi
  • Zadaj pytanie
  • Kategorie
  • Tagi
  • Zdobyte punkty
  • Ekipa ninja
  • IRC
  • FAQ
  • Regulamin
  • Książki warte uwagi

question-closed Java porównywanie odczytu z textField do łancucha znakowego z pliku tekstowego w interfejsie AWT

Object Storage Arubacloud
0 głosów
518 wizyt
pytanie zadane 28 stycznia 2021 w Java przez sznycel Nowicjusz (220 p.)
zamknięte 29 stycznia 2021 przez sznycel

Cześć!

Ogólnym zadaniem programu jest otwieranie i edycja plików tekstowych z menu czyli dopisywanie i usuwanie linii tekstu oraz dodawanie nowych plików do menu. Usuwanie i sprawdzanie czy plik jest w menu nie działa poprawnie. środowisko to NeatBeans 8.2

chodzi mi o linijki 65 i 121 gdzie porównuje to co wpisał użytkownik z tym co jest w pliku. Problem jest ten sam w 2 przypadkach obydwa podejścia nie działają .equals wyrzuca błąd java.lang.nullpointerexception a == kompletnie pomija i metoda wykonuje się tak jakby bez if i to jak zadeklaruje r tak je zwróci.

Proszę o pomoc bo na prawdę skończyły mi się pomysły.

tutaj wyodrębnione błędy które zauważyłem

              if(s.equals(u)){

                  pw.println("");
                  
               }
                if(s==w){
                  r++;                  
                }

tutaj kod całego programu

import java.io.FileWriter;
import java.io.PrintWriter;
import java.io.FileReader;
import java.io.BufferedReader;
import java.io.IOException; 
import java.io.File;

public class Zaliczenie extends java.applet.Applet {
    String plik="";
    

    void odczyt(String plik){
        String s;
        
        try{
            
            FileReader fr = new FileReader(plik+".txt");
            BufferedReader br = new BufferedReader(fr);
            
            do{
               s = br.readLine();
               if(s != null){
                  textArea1.append(s+"\n");   
               }
            }while(s != null);
         
            fr.close();
            br.close();
        }catch(IOException e){
            System.out.println("Błąd we/wy...");
        }
        
    }
    void dopisz(String s, String p){
        
        try{
            FileWriter fw = new FileWriter(p+".txt",true);
            PrintWriter pw = new PrintWriter(fw);
            
            pw.println(s);
            //textArea1.append(s);

            
            
            pw.close();
            fw.close();
            
        }catch(IOException e){
            System.out.println("błąd wejścia wyjścia");
        } 
           
    }
    
    void usun(String u){
       //String s="";
       try{
            String s="";
            FileReader fr = new FileReader(plik+".txt");
            BufferedReader br = new BufferedReader(fr);
            FileWriter fw = new FileWriter("buf.txt",true);
            PrintWriter pw = new PrintWriter(fw);
            
            do{
               s = br.readLine();
               if(s.equals(u)){
                  pw.println("");
                  
               }
               else{               
               if(s != null){
                  pw.println(s);  
               }
               }
            }while(s != null);
         
            fr.close();
            br.close();
            pw.close();
            fw.close();
            
            File n = new File(plik+".txt");
            n.delete();
            
            FileReader fr1 = new FileReader("buf.txt");
            BufferedReader br1 = new BufferedReader(fr1);
            FileWriter fw1 = new FileWriter(plik+".txt",true);
            PrintWriter pw1 = new PrintWriter(fw1);
            do{
               s = br1.readLine();
                             
               if(s != null){
                  pw1.println(s);  
               }
               
            }while(s != null); 
            
            fr1.close();
            br1.close();
            fw1.close();
            pw1.close();
            
            File b = new File("buf.txt");
            b.delete();
            
           
            
            
            
        }catch(IOException e){
            System.out.println("Błąd we/wy...");
        }
    }
    int spr(String w){
                       
        try{
            int r = 1 ; 
            String s;            
            
            FileReader fr = new FileReader("menu.txt");
            BufferedReader br = new BufferedReader(fr);
            
            do{
               s = br.readLine();
               
               if(s==w){
                  r++;                  
                }
               
            }while(s != null);
            
            fr.close();
            br.close();
            return r;
        }catch(IOException e){
            System.out.println("Błąd we/wy...");
            return 0;
        }
    }
       
       
        
    

        

    public void init() {
       
        try {
            java.awt.EventQueue.invokeAndWait(new Runnable() {
                public void run() {
                    initComponents();
                }
            });
        } catch (Exception ex) {
            ex.printStackTrace();
        }
    }

    /**
     * This method is called from within the init() method to initialize the
     * form. WARNING: Do NOT modify this code. The content of this method is
     * always regenerated by the Form Editor.
     */
    // <editor-fold defaultstate="collapsed" desc="Generated Code">//GEN-BEGIN:initComponents
    private void initComponents() {

        label1 = new java.awt.Label();
        textArea1 = new java.awt.TextArea();
        button1 = new java.awt.Button();
        button2 = new java.awt.Button();
        button3 = new java.awt.Button();
        label2 = new java.awt.Label();
        button4 = new java.awt.Button();
        textField2 = new java.awt.TextField();
        button5 = new java.awt.Button();

        button1.setLabel("Otwórz");
        button1.addActionListener(new java.awt.event.ActionListener() {
            public void actionPerformed(java.awt.event.ActionEvent evt) {
                button1ActionPerformed(evt);
            }
        });

        button2.setLabel("Dopisz");
        button2.addActionListener(new java.awt.event.ActionListener() {
            public void actionPerformed(java.awt.event.ActionEvent evt) {
                button2ActionPerformed(evt);
            }
        });

        button3.setLabel("Usuń");
        button3.addActionListener(new java.awt.event.ActionListener() {
            public void actionPerformed(java.awt.event.ActionEvent evt) {
                button3ActionPerformed(evt);
            }
        });

        button4.setLabel("Menu");
        button4.addActionListener(new java.awt.event.ActionListener() {
            public void actionPerformed(java.awt.event.ActionEvent evt) {
                button4ActionPerformed(evt);
            }
        });

        textField2.setText("Gry");

        button5.setLabel("Utwórz nowy plik");
        button5.addActionListener(new java.awt.event.ActionListener() {
            public void actionPerformed(java.awt.event.ActionEvent evt) {
                button5ActionPerformed(evt);
            }
        });

        javax.swing.GroupLayout layout = new javax.swing.GroupLayout(this);
        this.setLayout(layout);
        layout.setHorizontalGroup(
            layout.createParallelGroup(javax.swing.GroupLayout.Alignment.LEADING)
            .addGroup(layout.createSequentialGroup()
                .addGroup(layout.createParallelGroup(javax.swing.GroupLayout.Alignment.LEADING)
                    .addGroup(layout.createSequentialGroup()
                        .addGap(145, 145, 145)
                        .addGroup(layout.createParallelGroup(javax.swing.GroupLayout.Alignment.LEADING, false)
                            .addComponent(label1, javax.swing.GroupLayout.DEFAULT_SIZE, javax.swing.GroupLayout.DEFAULT_SIZE, Short.MAX_VALUE)
                            .addComponent(button1, javax.swing.GroupLayout.DEFAULT_SIZE, javax.swing.GroupLayout.DEFAULT_SIZE, Short.MAX_VALUE))
                        .addPreferredGap(javax.swing.LayoutStyle.ComponentPlacement.RELATED)
                        .addGroup(layout.createParallelGroup(javax.swing.GroupLayout.Alignment.LEADING, false)
                            .addGroup(layout.createSequentialGroup()
                                .addComponent(button2, javax.swing.GroupLayout.DEFAULT_SIZE, javax.swing.GroupLayout.DEFAULT_SIZE, Short.MAX_VALUE)
                                .addPreferredGap(javax.swing.LayoutStyle.ComponentPlacement.RELATED)
                                .addComponent(button3, javax.swing.GroupLayout.PREFERRED_SIZE, javax.swing.GroupLayout.DEFAULT_SIZE, javax.swing.GroupLayout.PREFERRED_SIZE))
                            .addComponent(label2, javax.swing.GroupLayout.DEFAULT_SIZE, javax.swing.GroupLayout.DEFAULT_SIZE, Short.MAX_VALUE)))
                    .addGroup(layout.createSequentialGroup()
                        .addGap(146, 146, 146)
                        .addGroup(layout.createParallelGroup(javax.swing.GroupLayout.Alignment.LEADING, false)
                            .addComponent(textArea1, javax.swing.GroupLayout.DEFAULT_SIZE, 392, Short.MAX_VALUE)
                            .addComponent(textField2, javax.swing.GroupLayout.DEFAULT_SIZE, javax.swing.GroupLayout.DEFAULT_SIZE, Short.MAX_VALUE)
                            .addGroup(layout.createSequentialGroup()
                                .addComponent(button4, javax.swing.GroupLayout.PREFERRED_SIZE, javax.swing.GroupLayout.DEFAULT_SIZE, javax.swing.GroupLayout.PREFERRED_SIZE)
                                .addPreferredGap(javax.swing.LayoutStyle.ComponentPlacement.RELATED)
                                .addComponent(button5, javax.swing.GroupLayout.PREFERRED_SIZE, javax.swing.GroupLayout.DEFAULT_SIZE, javax.swing.GroupLayout.PREFERRED_SIZE)))))
                .addContainerGap(205, Short.MAX_VALUE))
        );
        layout.setVerticalGroup(
            layout.createParallelGroup(javax.swing.GroupLayout.Alignment.LEADING)
            .addGroup(layout.createSequentialGroup()
                .addGap(81, 81, 81)
                .addGroup(layout.createParallelGroup(javax.swing.GroupLayout.Alignment.TRAILING)
                    .addComponent(label1, javax.swing.GroupLayout.PREFERRED_SIZE, javax.swing.GroupLayout.DEFAULT_SIZE, javax.swing.GroupLayout.PREFERRED_SIZE)
                    .addComponent(label2, javax.swing.GroupLayout.PREFERRED_SIZE, javax.swing.GroupLayout.DEFAULT_SIZE, javax.swing.GroupLayout.PREFERRED_SIZE))
                .addPreferredGap(javax.swing.LayoutStyle.ComponentPlacement.RELATED)
                .addGroup(layout.createParallelGroup(javax.swing.GroupLayout.Alignment.LEADING)
                    .addGroup(layout.createSequentialGroup()
                        .addGroup(layout.createParallelGroup(javax.swing.GroupLayout.Alignment.LEADING)
                            .addComponent(button2, javax.swing.GroupLayout.PREFERRED_SIZE, javax.swing.GroupLayout.DEFAULT_SIZE, javax.swing.GroupLayout.PREFERRED_SIZE)
                            .addComponent(button1, javax.swing.GroupLayout.PREFERRED_SIZE, javax.swing.GroupLayout.DEFAULT_SIZE, javax.swing.GroupLayout.PREFERRED_SIZE))
                        .addPreferredGap(javax.swing.LayoutStyle.ComponentPlacement.RELATED)
                        .addComponent(textField2, javax.swing.GroupLayout.PREFERRED_SIZE, javax.swing.GroupLayout.DEFAULT_SIZE, javax.swing.GroupLayout.PREFERRED_SIZE)
                        .addPreferredGap(javax.swing.LayoutStyle.ComponentPlacement.RELATED)
                        .addGroup(layout.createParallelGroup(javax.swing.GroupLayout.Alignment.TRAILING)
                            .addComponent(button4, javax.swing.GroupLayout.PREFERRED_SIZE, javax.swing.GroupLayout.DEFAULT_SIZE, javax.swing.GroupLayout.PREFERRED_SIZE)
                            .addComponent(button5, javax.swing.GroupLayout.PREFERRED_SIZE, javax.swing.GroupLayout.DEFAULT_SIZE, javax.swing.GroupLayout.PREFERRED_SIZE))
                        .addGap(29, 29, 29)
                        .addComponent(textArea1, javax.swing.GroupLayout.PREFERRED_SIZE, 189, javax.swing.GroupLayout.PREFERRED_SIZE))
                    .addComponent(button3, javax.swing.GroupLayout.PREFERRED_SIZE, javax.swing.GroupLayout.DEFAULT_SIZE, javax.swing.GroupLayout.PREFERRED_SIZE))
                .addContainerGap(64, Short.MAX_VALUE))
        );
    }// </editor-fold>//GEN-END:initComponents

    private void button1ActionPerformed(java.awt.event.ActionEvent evt) {//GEN-FIRST:event_button1ActionPerformed
        textArea1.setText("");
        plik=textField2.getText();
        if(plik.equals("")){
        textArea1.setText("Podaj nazwę pliku");     
        }
        else if (spr(plik)==0){                      
        textArea1.setText("Pliku nie ma w bazie");        
        }
        else{
        odczyt(plik);
        label1.setText("Jesteś w:");
        label2.setText(plik);
        textField2.setText(""); 
        }
       
    }//GEN-LAST:event_button1ActionPerformed

    private void button2ActionPerformed(java.awt.event.ActionEvent evt) {//GEN-FIRST:event_button2ActionPerformed
        textArea1.setText("");
        if(textField2.getText().equals("")){
            textArea1.setText("podaj tekst");
        }
        else if(plik.equals("")){
            textArea1.setText("nie jesteś w pliku");
        }
            else{
        label1.setText("Jesteś w:");
        label2.setText(plik);
        dopisz(textField2.getText(), plik); 
        textField2.setText("");
        textArea1.setText("");
        odczyt(plik);
                    }
                   
    }//GEN-LAST:event_button2ActionPerformed

    private void button3ActionPerformed(java.awt.event.ActionEvent evt) {//GEN-FIRST:event_button3ActionPerformed
        textArea1.setText("");
        usun(textField2.getText());
        odczyt(plik);
    }//GEN-LAST:event_button3ActionPerformed

    private void button4ActionPerformed(java.awt.event.ActionEvent evt) {//GEN-FIRST:event_button4ActionPerformed
        textArea1.setText("");
        label1.setText("Jesteś w:");
        label2.setText("menu czytasz "+plik);
        odczyt("menu");
    }//GEN-LAST:event_button4ActionPerformed

    private void button5ActionPerformed(java.awt.event.ActionEvent evt) {//GEN-FIRST:event_button5ActionPerformed
        textArea1.setText("");
        dopisz(textField2.getText(), "menu");
        odczyt("menu");
        label1.setText("Jesteś w:");
        label2.setText("menu czytasz "+plik);
        dopisz("", textField2.getText());
        textField2.setText("");
    }//GEN-LAST:event_button5ActionPerformed


    // Variables declaration - do not modify//GEN-BEGIN:variables
    private java.awt.Button button1;
    private java.awt.Button button2;
    private java.awt.Button button3;
    private java.awt.Button button4;
    private java.awt.Button button5;
    private java.awt.Label label1;
    private java.awt.Label label2;
    private java.awt.TextArea textArea1;
    private java.awt.TextField textField2;
    // End of variables declaration//GEN-END:variables
}

 

komentarz zamknięcia: rozwiązałem problem samodzielnie
komentarz 29 stycznia 2021 przez sznycel Nowicjusz (220 p.)

Dziękuje za wszystkie odpowiedzi wystarczyło dopisać warunek że s != null teraz wygląda to tak i wszystko działa.

if(s != null && s.equals(w)){

 

1 odpowiedź

+1 głos
odpowiedź 28 stycznia 2021 przez Wiciorny Ekspert (270,110 p.)
wybrane 29 stycznia 2021 przez sznycel
 
Najlepsza
if(s==w){
  r++;                  
}

to jest porównanie referencji, to nigdy nie będzie prawdą [prawie nigdy, chyba że domyslnie ustawisz referencje W na S]  jeśli obiekty generalnie są stringami. 
Stringi porównujesz equals jak poprzednio 

if(s.equals(w)){
  r++;                  
}

 

1
komentarz 28 stycznia 2021 przez tkz Nałogowiec (42,000 p.)
Czytamy chyba inne odpowiedzi. Widzę wyjaśnienie dlaczego == w javie nie zadziała dla obiektów. Widzę również potwierdzenie, że .equals do tego służy. Prowadzisz narrację podobną do osoby, która jakiś czas temu "płakała", że nikt tutaj nie pomaga, żadni "eksperci" i wiele innych truizmów.
komentarz 28 stycznia 2021 przez Wiciorny Ekspert (270,110 p.)

@majstersztyk64, A O CZYM świadczy twoim zdaniem 
 

null.equals("coś");

a no może na tym, że obiekt na którym wywołujemy metode jest nullem? Więc może by tak zrozumieć, że metoda odwołuje się do obiektu, a obiekt jest referencyjny, ta referencja jest nulem... idąc dalej czemu tak się dzieje, a może mimo że "do" while... czyli 
gwarancje mamy że wykona się wczytanie, miało miejsce z przypisaniem jednak coś poszło nie tak ? 
 

      s = br.readLine();   // zwraca nam null czyli wypada analizować dlaczego 

nie pobieramy wartości, jak Ci się kolego nie podoba, to zmień forum.  
To ma być "POMOC", ta pomoc ma go czegoś nauczyć, żeby na przyszłość zrozumiał dlaczego ma stosować equals, a nie "mi tu zwraca NULL więc nie bede stosował equals"

ma to też go czegoś nauczyć dlaczego warto czasami nawet sprawdzać zanim wywołamy inna metode, na innym obiekcie np samochód.run(); dlaczego jak poleci null to coś z obiektem, a tak naprawdę referencją na obiekt jest nie tak :) 

A JEST TO NA TYLE ISTOTNE, ŻE jeśli wywołamy metode na obiekcie który istnieje 
 

"string".equals(null);

to zauważ, że zwróci nam FALSE a nie nullpointer :) ... 

komentarz 28 stycznia 2021 przez majstersztyk64 Nowicjusz (100 p.)
Widzę, że dalej ciągniesz temat, nic do niego nie wnosząc. To, że napiszesz coś grubszą albo czerwoną czcionką, nie zmieni tego, że jedyne co tu zrobiłeś to przekleiłeś kawałki kodu autora.
komentarz 28 stycznia 2021 przez Wiciorny Ekspert (270,110 p.)
ciesze się że mogłem pomóc, polecam sie na przyszłość
komentarz 28 stycznia 2021 przez ScriptyChris Mędrzec (190,190 p.)

Ty te 100 tysięcy punktów tak nabiłeś, po prostu pisząc głupoty i nawet nie zaglądając do załączników, które ktoś wysyła? Jeśli tak wygląda pomoc "eksperta", to trochę kiepsko to świadczy o jakości całego forum.

@majstersztyk64, zachowaj proszę kulturę wypowiedzi i okaż szacunek do rozmówcy. Uwagi merytoryczne można napisać bez obrażania kogokolwiek.

Podobne pytania

0 głosów
6 odpowiedzi 939 wizyt
pytanie zadane 16 września 2016 w Java przez Kacperos_89 Początkujący (400 p.)
0 głosów
1 odpowiedź 722 wizyt
0 głosów
0 odpowiedzi 235 wizyt

92,570 zapytań

141,422 odpowiedzi

319,643 komentarzy

61,958 pasjonatów

Motyw:

Akcja Pajacyk

Pajacyk od wielu lat dożywia dzieci. Pomóż klikając w zielony brzuszek na stronie. Dziękujemy! ♡

Oto polecana książka warta uwagi.
Pełną listę książek znajdziesz tutaj.

Akademia Sekuraka

Kolejna edycja największej imprezy hakerskiej w Polsce, czyli Mega Sekurak Hacking Party odbędzie się już 20 maja 2024r. Z tej okazji mamy dla Was kod: pasjamshp - jeżeli wpiszecie go w koszyku, to wówczas otrzymacie 40% zniżki na bilet w wersji standard!

Więcej informacji na temat imprezy znajdziecie tutaj. Dziękujemy ekipie Sekuraka za taką fajną zniżkę dla wszystkich Pasjonatów!

Akademia Sekuraka

Niedawno wystartował dodruk tej świetnej, rozchwytywanej książki (około 940 stron). Mamy dla Was kod: pasja (wpiszcie go w koszyku), dzięki któremu otrzymujemy 10% zniżki - dziękujemy zaprzyjaźnionej ekipie Sekuraka za taki bonus dla Pasjonatów! Książka to pierwszy tom z serii o ITsec, który łagodnie wprowadzi w świat bezpieczeństwa IT każdą osobę - warto, polecamy!

...