Dotaz: Osetreni vyjimky JAVA

[Pavel]

Ahoj, jak v programu, kde ctu textovy soubor osetrim vyjimku, ze soubor nelze cist?? nejak nevim, kdy tato vyjimka muze nastat? a ani nevim, ktera vyjimka to je??

(Uz mam osetrene vyjimky IOException a FileNotFoundException....)

Predem diky za rady

---

---

Odpovědi

[Šangala]

Možná by to chtělo vědět jak soubor čtete - od toho se odvíjí vyjímky co mohou nastat.
Ale jinak každá fce má většinou dokumentaci, a tam lze zjistit kdy jaká vyjímka nastává :)
Nelze číst - může být i z důvodu nedostatku volné paměti

[Pavel]

ahoj, dik za reakci

davam sem kod moji metody i se zadanim:

http://pastebin.com/m4db49001

pokud soubour neni nalezen, je to vpohode(vypise Soubor nenalezen) pokud nastane naka jina chyba, tak je to taky vpohode

jen nevim,jaka je vyjimka ze nelze cist....??

diky

[razor]

Myslíš ten IllegalAccessError? To bych vůbec nechytal. http://java.sun.com/j2se/1.4.2/docs/api/java/lang/IllegalAccessError.html

[Pavel]

ano, cetl a pokud to dobre chapu, tak je vyvolana, pokud zkousi cist neco, k cemu nema pristup??

takhle zni zadani: Pokud soubor nebude možné císt, bude vypsána hláška "Soubor nelze cist." a program bude ukoncen.

co si pod tim teda mam predstavit?? kdy nastane?

[Šangala]

Jak píše razor IllegalAccessError bych dal pryč a dal tam IOException(do vnějšího ošetření) a co se týče ošetření to, dle mého, splní zadání

ad. stimhle mam problem, nevim ktera vyjimka to je?

Bych to v tomto případě viděl jen na vyjímku IOException

[impact]

V Java API dokumentacii pre BufferedReader methodu read sa pise o IOException:

IOException - If an I/O error occurs

Urcite by si mal osetrit tuto vynimku. A na koniec mozes osetrit aj vseobecnu Exception, od ktorej su odvodene vsetky ostatne vynimky.

[impact]

Ne, můžeš musíš :)
Zadání přepsáno do řeči vyjímek (v tomto připadě) odpovídá:

• FileNotFoundException −> Pokud soubor neexistuje
• IOException −> Pokud soubor nebude možné číst
• Exception −> Nastane-li jakákoli jiná chyba

A stačí jedno ošetření (1× try) na celý kód, protože vše, v tomto případě, je práce ze souborem. :)

[kulik]

Pro zjisteni zda lze ze souboru cist bych si zavolal na instanci File canRead(), stejne tak pro zjisteni zda soubor existuje exists(). Vyjimkami bych se zabyval az pak.

[Pavel]

tak se to poupravil, ale porad to nejde.....

pridavam kod + vysledky validace :

http://pastebin.com/m66459cda

[kulik]

Viz dokumentace!!! Konstruktor FileReader vyhodi FileNotFoundException pokud "file does not exist, is a directory rather than a regular file, or for some other reason cannot be opened for reading". Udelej to jak jsem ti radil vyse.

[Pavel]

promin, ale ja nejak nechapu, jak mam tyto metody volat??

Ty metody se podle ukazky volaji nad souborem, ale ja tady zadny novy nevytvarim a neni mi jasne, jak metodu zavolat nad tim souborem co uz je vytvorenej....??

tohle je mi jasne: File f1 = new File("f.txt"); System.out.println(f1.exists() ? "exists" : "does not exist"); System.out.println(f1.canRead() ? "is readable" : "is not readable");

ale jak to pouzit vmem programu nevim

(? neco jako nazevSouboru.canRead();

[kulik]

V tvem programu jako v jakemkoliv jinem. File predstavuje abstrakci nad soubory, kdyz zavolam new File(), zadny novy soubor mi nikde nezvnikne. Pri programovani je opravdu je potreba cist dokumentaci a nevkladat jasne definovanym terminum jine intuitivni vyznamy. Ale abych byl konstruktivni:

File file = new File(nazevSouboru);
if (! file.exists()) {
  ...
}
if (! file.isReadable()) {
  ...
}
try {
  BufferedReader vstup = new BufferedReader(new FileReader(file));
  ...

Jinak pro postupne nacitani nepouzivej String ale StringBuilder nebo StringBuffer, bude to rychlejsi (i kdyz tady o to asi tolik nejde).

[Pavel]

aha, no ja sem v programovani zacatecnik, takze mi porad neco nejde, porad neco nechapu, porad se na neco ptam, hledam....

myslis, ze takhle uz by to mohlo chodit?? http://pastebin.com/m5af6154b (je to zdrojovej kod celeho DU....)

a taky bych rad slysel nakou kritiku na muj program - jak by se to dalo efektivneji resit, co je spatne, .......

diky moc

[moira]

Chodit by to snad mohlo (i když jsem to popravdě moc nezkoumal) ale máš tam jeden velký problém, který se nazývá duplicita kódu.

if (pismeno == 'a') {
                cetnostA++;
                if (cetnostA < 10) {
                    pole[0][j] = cetnostA;
                } else {
                    int pom = cetnostA / 10;
                    int pom2 = cetnostA % 10;
                    pole[0][j] = pom;
                    pole[1][j] = pom2;
                }
            }

...tohle je nejžhavější kandidát na samostatnou funkci. Jinak nerozumím proč ukládáš zvlášť jednotky a zvlášť desítky, ale jsem psal, moc jsem to nestudoval.

[moira]

Tak na druhé přečtení vidím, že program jasně nebude splňovat podmínky zadání, pokud četnost nějakého znaku bude větší než 99. Moje doporučení je ukládat četnosti jen do jednorozměrného pole, rozklad na řády si pak uděláš až při výpisu pomocí dělení a zbytků po dělení 10 (něco takového už máš). Jelikož to už budeš mít v poli, tak zaházej proměnné cetnostX. Pole indexuj číslem, které získáš přetypováním char na byte a odečtením 97 (viz. ASCII tabulka). Program se ti zkrátí na pětinu a nezešedivíš, pokud tam budeš chtít udělat nějaké změny/opravy.

[kulik]

Jen strucne:

Ke String/StringBuilder uz jsem se vyjadroval vyse.

Definovat promennou pro citac kazdeho pismene je desne, stejne tak vlastni vetev if, tak by zadny program opravdu vypadat nemel. Pro toto konkretni zadani lze vyuzit ulozeni citacu v poli podle ascii kodu znaku, ktere, nebo obecne pro libovolne znaky bych udelal Map<Character, Integer> a cital to tam (ale bylo by to mnohem mene efektivni z hlediska rychlosti), kazdopadne by byla jedna promenna pro citac a jeden citajici blok, napr:

  int counter[26];
  for (int i = 0; i < len; i++) {
    int asciiVal = (int) vstup.charAt(i);
    if (asciiVal >=97 && asciiVal <= 122) {
      int idx = asciiVal - 97;
      counter[idx] += 1;
      toPrint[i] = counter[idx];      
    }  
  }

Tisk cetnosti na radky pod sebou bych resil az nakonec, tve reseni s dvojrozmernym polem jednak prenasi prezentacni etail do logiky se kterou vubec nesouvisi, jednak nemuzes vedet zda ta cetnost nebude mit vice cifer nez dve. Tzn. zaznamenej si ty cetnosti jako cisla a az dojde na jejich tisk, tak to teprve res (muzes si behem nacitani napriklad udrzovat maximum ze vsech cetnosti, abys vedel kolik radku je treba na konci vytisknout).

Ulohu lze resit bez nacitani souboru do pameti (to bych preferoval), ale malicko se tim zeslozituje to vytisknuti cetnosti pod sebou, protoze neni predem znama delka radku - to ale jde lehce resit treba samozvetsujicim se polem pro citac (tzn. kdyz uz mi pole nestaci, alokuju si nove treba dvakrat vetsi, stare si do nej presunu a jedu dal). Proste kazdy znak co nactes vypises na vystup (+ transformace newline) a nacitas do citace, k nacteni do stringu neni zadny duvod.