Klasa Desktop wyjaśnienie kodu

Question

Witam! Na swoje potrzeby wyszukałem w necie jak zrobić metodę, która otwiera mi plik z poziomu programu (ale się podnieciłem jak zadziałało ^^) ale zupełnie nie rozumiem znaczenia kawałka kodu i dlaczego tak akurat.. Chodzi mi o ten kod (a raczej o to co jest między komentarzami):

package biblioteka;

import java.awt.Desktop;
import java.io.File;
import java.io.IOException;

public class OtwieraniePlikuTesktowego
{

	public static void main(String[] args)
	{
		File file = new File("Informacje.pdf");

		if (!Desktop.isDesktopSupported())
		{
			System.out.println("Klasa Desktop nie jest obsługiwana na bieżącej platformie! \n");
		}

		Desktop desktop = Desktop.getDesktop();
		try
		{
			//////////////////////////////////////////////////////////////////
			if (file.exists())
				desktop.open(file);
			file = new File("Informacje/java.pdf"); // ??
			if (file.exists())    //2x to samo?
				desktop.open(file); //i tu też..
			///////////////////////////////////////////////////////////////////
		} catch (IOException e)
		{
			System.err.println(e);
		}

	}

}

 

Answer 1

Ten kod jest ewidentnie kiepski. Po pierwsze gdy już sprawdzi ze desktop not supported to zrobi printa i lexi dalej get desktop :D. A to co zaznaczyles powinno odbyc sie raz.

Comments

hmmm nie znam się nigdy tego nie widziałem a (niestety ;((( ) skopiowałem to z tej strony:

https://www.journaldev.com/864/java-open-file

---

Jak więc to zrobić dobrze?(prócz tego co powiedziałeś ) trzeba się uczyć dobrych praktyk ;D

---

Na pierwszy rzut oka się zdziwiłem, bo kojarzę journaldev i raczej był solidny content. Problem w tym, że Ty ten przykład przerobiłeś. On otwierał plik dwa razy, bo otwierał inne pliki. Chciał pokazać, że jak otworzysz plik .txt to otworzy Ci się w Twoim domyślnym programie od .txt (prawdopodobnie notatniku pod windowsem) a jak plik .pdf to otworzy sie innym programem. Ty w sumie też otwierasz dwa pliki, ale nie widzę sensu :P skoro pytasz o co chodzi to znaczy, że nie otwierasz celowo dwóch plików :P.

I on zrobił return gdy dekstop not supported.

Powinieneś zrobić to najnormalniej w świecie:
 

    public static void main(String[] args) {
        if (!Desktop.isDesktopSupported()) {
            System.out.println("Klasa Desktop nie jest obsługiwana na bieżącej platformie! \n");
            return;
        }
        File file = new File("Informacje.pdf");
        Desktop desktop = Desktop.getDesktop();
        try {
            if (file.exists())
                desktop.open(file);
        } catch (IOException e) {
            System.err.println(e);
        }
    }

 

---

Ale kompromitacja :D Boże.. Nie było tematu, ty tego nie czytałeś, ja tego nie napisałem a teraz idę sie zapaść pod ziemie :D