Problém s dedením – Java – Fórum – Programujte.com

Ahojte, 

robím program v ktorom analyzujem rámec, ktorý si odchytím pomocou programu Wireshark. Zachytený rámec môže byť typu Ethernet II, IEEE 802.3 - LLC atď..keď je typu Ethernet II tak môže byť napr IP, ARP atď.. a keď je napríklad IP môže obsahovať vnorený protokol UDP alebo TCP alebo iné . Mám vytvorenú triedu Paket v ktorej si nastavím cieľovú a zdrojovú MAC adresu a zistím akého typu daný rámec je. A potom mám ďalšie triedy Ethernet II,  IEEE 802.3 - LLC..ktoré majú svoje ďalšie vlastné pridané parametre, ktore postupne dopĺňajú..takisto aj IP ARP a ostatné čo som spomenul. Ale asi niečo robím zle lebo pri dedení mi to padne. Chcel by som sa opýtať čím to bude? 

//tu si načítam paket a vytvorím nový objekt 
while(pcap.nextEx(header, jbuffer) == Pcap.NEXT_EX_OK){ //citam az kym nie som na konci
			 PcapPacket packet = new PcapPacket(header,jbuffer);		
			  Paket paket = new Paket(frame_number,packet);
	...
}

//konštruktor v triede Paket
public Paket(int frame, PcapPacket packet){
		this.frame = frame;
		this.packet = packet;
		setAdress(packet);		//nastaví MAC adresy
		setSize(packet);		//zistí veľkosť
		setTyp(packet);  		//Ethernet II, IEEE 802.3.. 
	if(getTyp().equals("Ethernet II")){
               Ethernet ethernet1 = new Ethernet(frame_number,packet);
               
           }
           else if(getTyp().equals("IEEE 802.3 - LLC")){
               LLC llc1 = new LLC(frame_number,packet);
               
           }
           else if(getTyp().equals("IEEE 802.3 - SNAP")){
               SNAP snap1 = new SNAP(frame_number, packet);
               
           }
           else if(getTyp().equals("IEEE 802.3 - Raw")){
               RAW raw1 = new RAW(frame_number, packet);
              
           }
	}

a keď je to napr EtherType Ethernet II tak idem do triedy Ethernet  kde je konštruktor :

public Ethernet(int frame_number, PcapPacket packet) {
		super(frame_number, packet);  //vyhodi chybu
		setEtherType(packet);
		setIP(packet);
		
	}

Ak by ste mi vedeli pomôcť bol by som vďačný. Vopred ďakujem. 

Problém je, že ten tvůj packet dělá víc než je třeba. Měl by obsahovat pouze data, né žádnou logiku, na to by si měl vytvořit nějakej Helper, Builder nebo jinej objekt. Pro vytvoření Ethernetu by si měl udělat Factory, která podle typu vrátí instanci.

a to že ti to v Ethernet vyhodí chybu... od čeho dědí Ethernet? Pokud od ničeho, tak ten super tam nemůže takto být.
 

#2 ingiraxo
Proč by třída Paket neměla mít žádné metody? V Javě se přece má programovat objektově.

#3 Kit
tak v javě to jinak ani moc dobře nejde, ale tím jsem reagoval na to vytváření instance Ethernetu uvnitř paketu, což je blbost, by mělo být řešené přes Factory. A potom je ještě otázka, jestli to dávat přímo do paketu, jestli jen přes setter to nenastavit mimo paket.

#4 ingiraxo
V Javě se běžně programuje procedurálně, i když to není žádoucí - viz nadužívání getterů a setterů.

Factory se používá tam, kde potřebuješ vyrobit objekty různých tříd se stejným rozhraním, což asi není tento případ.

#5 Kit
Je pravda, že nevim jestli ty objekty mají společnýho předka (Ethernet, LLC, SNAP, RAW, ..).. nevim jestli to jsou jeho objekty nebo z API, ale vzhledem k tomu, ze uvedl konstruktor na Ethernet, tak hádám, že dědí z nějakýho Ethernetu z API, není problém tam dosypat vlastní interface do všech 4 objektů

#2 ingiraxo Ide o to, že v triede Paket si nastavím MAC adresy, veľkosť a typ. Ak je typ Ethernet tak zas v triede Ethernet(ktorá dedí od triedy Paket) si nastavím aký protokol v ňom je a IP adresy. Pointa je, že keď sa mi zanalyzuje rámec tak chcem aby mal všetky informácie(MAC adresy, veľkosť, typ, IP adresy a ďalšie)..

#7 Kent
vůbec se mi třeba nelíbí, že Ethernet dědí z Packet a ty v konstruktoru Paket vytváříš instanci třídy, která vychází z Packet... nepřipadá ti to takový divný?
Proč vůbec Ethernet dědí z Packet?

A vůbec.. Ethernet a ty další 3 by neměli dědit z Packet, když už to takhle "divně" chceš a potřebuješ mít referenci na Paket, tak na Ethernetu udělej asociaci na Packet, ale i tak mi to nepřijde potřeba

Takhle to už dává smysl proč ti to nejde.. si se zacyklil..

#7 Kent
např. toto je jedna z možností http://paste.ubuntu.com/10569480/

Dost záleží co s tím děláš, jestli všechny ty paketový typy mají stejný vlastnosti jen jiný obsah nebo i odlišný vlastnosti..  veškerá implementace záleží na způsobu použití

Pokud všechny maji stejný vlastnosti, tak v konsturktoru ve třídě Result by měl být volán Factory (PacketFactory)

#9 ingiraxo
Ďakujem pomohlo mi to a prerobím si to :)