ohjelmoinnin opettaja oli heittänyt että instanceof:n käyttöä tulisi rajoittaa minimiin ja se vihjaa huonosti suunnitellusta ohjelmasta, koska luokat ja niiden toiminnallisuus tulisi tehdä niin että tyyppitarkastuksia ei tarvita missään vaiheessa (tai jotain). mitä mieltä ootte tästä, allekirjoitatteko?
instanceof:n käyttö huonoa suunnittelua?
opettajan suusta
4206
0Äänestä
Muut lukivat myös
Odottaa vastausta
Näihin keskusteluihin ei ole vielä vastattu. Ole ensimmäinen.
Vastaukset 4
Vanhimmat
En ole kuullut tuollaista väitettä. Toisaalta en ole tarvinnut instanceof:ia käytännössä koskaan. Lähinnä instanceof:ia voisi tarvita tilanteessa missä jokin olio välitetään Object-luokan avulla, jolloin instanceof:lla olisi hyvä tarkistaa, että saatu olio on oikeaa tyyppiä.
esim.
public Object palauta() {
return new JokuLuokka();
}
Object olio=palauta();
if (olio instanceof JokuLuokka) ((JokuLuokka)olio).teeJotain();
Käyttämällä rajapintoja, abstrakteja luokkia ja periytymistä oikein, ei tarvetta instanceof:in käytölle pitäisi syntyä.
Kyllä itse allekirjoitan täysin kyseisen väitteen. Kun luokka tarkastusta tarvitaan, niin se viittaa usein siihen, että ohjelmoija voi antaa jollekin metodille vääränlaisia olioita, ilman että kääntäjä siitä herjaa. Lisäksi kyseisen käskyn käyttö viittaa siihen, että ohjelmassa on pakotettuja tyyppimuunnoksia, jotka johtavat suunnittelun pettäessä helposti ajonaikaisiin virheisiin.
Kuten edellinen kirjoittaja sanoi, niin mm. rajapinnoilla voi yleensä kiertää tyyppi tarkastuksen tarpeen.
mutta toisaalta instanceof käytetään juuri parametrien tarkistukseen ja haarautumiseen siten että eri instannsi saa eri käsittely, tietysti tuo pitäisi ehkä tehdä mielluummin metodien overloadingina.
Sitä tarkistamista juuri tarkoitin, sillä ohjelma pitäisi suunnittele siten, että parametrien tarkistukset tekee kääntäjä eikä "ohjelma" ajon aikaisti. Tällöin virheet ohjelmassa tulevat esiin jo ohjelmaa käännettäessä.
Yleensähän se intanceof tarkistus tehdään juuri ennen pakotettua tyyppimuunnosta (ks. esimerkki 1). Kumpikin, niin pakotettu tyyppimuunnos että instanceof vihjaavat yleensä huonoon suunnitteluun.
Eli aivan kuten totesit, niin parempi tapa on käyttää overloadingia, (ks. esimerkki 2 joka on 1:n korjattu versio)
--- ESIMERKKI 1 ALKAA ---
public static void teeJotain(Object olio) {
if (olio instanceof Luokka1) {
((Luokka1) olio).luokan1Metodi();
}
if (olio instanceof Luokka2) {
((Luokka2) olio).luokan2Metodi();
}
}
--- ESIMERKKI 1 LOPPUU ---
--- ESIMERKKI 2 ALKAA ---
public static void teeJotain(Luokka1 olio) {
olio.luokan1Metodi()
}
public static void teeJotain(Luokka2 olio) {
olio.luokan2Metodi()
}
--- ESIMERKKI 2 LOPPUU ---