Hei (igjen). Sitter å skriver et lite program i Java, da jeg er ferdig med de øvelsene det er lagt opp til via skolen, men jeg likevel ønsker å øve samt prøve ut nye funksjoner. I mangel av bedre ideer, lager jeg et lite "hjelpeprogram" til SC2, med timers for å huske Injects/Creep-spread, samt kunne vise diverse info om de forskjellige enhetene. Men, jeg har litt problemer med å få alt til å virke på tvers av klassene. Problemene jeg har for øyeblikket er som følger:

• Jeg greier ikke å lese ut info om enhetene fra arrayet, på tross av at innlesingen fra tekstfilen skal fungere (det virker slik ut fra foreløpig testing)
• For øyeblikket bruker jeg anonyme actionlistenere, da jeg ikke fikk det til å fungere på andre måter jeg har lest om. Regner med det må finnes en bedre (og kortere kodemessig) måte å gjøre dette på? Ved event.getSource() el. noe lignende?
• Jeg kunne også tenkt meg at om en av timerne er aktive, og det byttes fane, så skal den stoppes. Vil anta at det er mulig (actionlistener?) at dette kan gjøres ved å sjekke om Focus mistes eller noe sånt?

Ellers tar jeg i mot alle tips som kan gjøre koden mer leselig, eller andre ting som kan gjøres på en bedre måte! Legger kun ved javafilen, da resten var for stort til å lastes opp her. Så det vil komme feilmelding på noen få bilder, lyder og en tekstfil som mangler...

Først av alt må jeg få si at jeg synes det var litt uoversiktlig kode. Forstår at du har samlet alle klassene i en "haug" for oss nå, men klassene dine er fortsatt vanskelige å lese. Foreslår at du leser deg litt opp på noen patterns, som for eksempel de nevnt i GRASP. For eksempel kan det være en god idé å separere logikk fra grafikk. Da blir det både mer oversiktlig, samtidig som koden din vil fungere uavhengig av hvordan du designer selve programmet grafisk.

Uansett, skal forsøke å svare på spørsmålene dine.
1. Klarer ikke helt å se hvor du prøver å lese fra arrayene dine, men du har deklarert variablene flere ganger, både utenfor main og i main (linje 11-13 og 102-104).

2. Anonyme lyttere er helt greit, blir ofte veldig rot. Husk at ActionListener er et interface som kan implementeres i alle klasser. Klasser kan ha konstruktører med parametre. Disse parametrene blir da referanser til objektet du oppgir under konstruksjon. La oss ta et konkret eksempel; btnTumorCounter-objektet ditt.

Her har du valgt anonym lytter, noe som i og for seg er helt greit. Videre kalles metoden "start()" på creepTimer-objektet. Dersom du skulle lagt en klasse av dette, kunne den sett slik ut:
Veldig enkel klasse, men fordelen er nå at du kan bruke objekter av samme klasse flere steder, både til å starte creepTimeren og injectTimeren, da begge er av typen Timer. Kanskje enklere å forstå ved hjelp av kode.
Fordelen med å gjøre det på denne måten, er at om du skal legge til ekstra sjekk, funksjonalitet etc før du starter timeren, gjøres dette kun nå i klassen TimerStartListener, og ikke flere steder i koden. Gjør det enklere å oppdatere kode, og mindre copy/paste.

3. Det skal være mulig å legge til lyttere når du skifter fane. I følge dokumentasjonen til JTabbedPane er det en metode som heter "addChangeListener". Da kan du bruke samme fremgangsmåte som i punkt 2, men klassen din implementerer ikke "ActionListener", men "ChangeListener".

Lykke til!

Er det noen mulighet for at du kan pakke sammen hele prosjektet slik at vi andre som eventuelt er interesserte i å kunne dra det ned og kunne gi noen pekere (0x3A28213A, 0X6339392C, 0x7363682E ) kan bidra? Kan anbefale for eksempel github.com. Da kan andre komme med iterasjoner som kan være til hjelp.

For min egen del gjør det at terskelen for å hjelpe noe lavere da jeg kan bare forke og kjøre på uten å måtte tenke på å splitte kode, hente ressurser som måtte være brukt osv. Det er veldig greit å hoppe inn i kode i debug-modus å steppe seg igjennom koden for å få en oversikt over hva som skjer, i stedet for å måtte lese hele koden og så begynne på arbeidet å analysere hva det er som egentlig foregår i koden.