Konstuktiv kritik af program

ok .. kigger på det i aften :.)

Hvis du vil have et brugernavn/password til up- og download, må du gerne lige smider mig en mail... Så kan jeg lige sende et med det samme...

Jeg har iøvrigt brugt en af dsj's klasser (browserlauncher), hvordan er det med copyright osv? det var en jeg fandt her på exp.

hmmm det siges jo at løsninger - postet på eksperten - må bruges af alle, så jeg vil antage at koden er public domain og at du derfor frit må bruge den...

Hej,
Det er altid spændende at se hvad folk har lavet så jeg tester det oxo :-)
Kan du ikke give mig user/pass på bp@maximuss.dk ??
Hvis du vil have konstruktiv kritik så er det nok nemmest med lidt "live" chat :-)
MSN: bp@maximuss.dk
ICQ: 136540871

2.3

Ophavsretten på materiale udvekslet for point via Eksperten.dk tilfalder Eksperten.dk's brugere. Det vil sige alle brugere frit må benytte de løsninger der udveksles på Eksperten.

3.9

Ved accept af Eksperten.dk's betingelser har du tilkendegivet at du forstår og accepterer ovenstående betingelser.

Hej igen
Jeg har siddet og kigget lidt på dit program. Jeg har lige 3 punkter:
1) Statistik; hvad skal den bruges til ?? Når jeg trykker sker der ikke noget

2) Times New Roman (tror nok at det er den du har brugt) ser ikke så pæn ud - jeg ville nok have valgt Arial eller Verdana. Eller endnu bedre, lad brugeren selv vælge :-)

3) Udskrift; personligt bryder jeg mig ikke om den måd du har lavet udskrift på. Jeg vil hellere have at den bare skrev det ud, uden at oprette en html fil og vise det i en browser hvor man så skriver ud fra.

Håber ikke at du tager det ilde op :-)

Men bortset fra det så ser det temmelig gennemtænkt ud :-)

man kan ikke resize det som man vil
og hvis man dobbelt clicker på window baren (den blå) så bliver programmet minimeret til man kun kan se  filer/hjælp og det er ikke muligt at få det i fuld størelse igen uden at lukke programmet.

Jeg formoder (og lidt test har ikke afkræftet det) at din applikation
kører på J2SE 1.4 og nyere (1.4.0, 1.4.1, 1.4.2 og fremtidige versioner).

Du har en klasse readStatus som nok burde hedde ReadStatus.

Forbedrings forslag: internationalisering så det var nemt at lave en
engelsk version (eller tysk/fransk/spansk hvis man tilfældigvis kan
sådanne eksotiske sprog).

Rasmus klassen burde nok splittes op - den er for stor.

catch(Exception ex)

anses normalt for ikke at være så pænt.

[jo - jmangle virker udmærket men brug af J2SE klasser kan ikke skjules]

Bare lige en kommentar mht. ophavsret.
Fordi en bruger poster noget kode her er det jo ikke nogen garanti for at den ikke er rippet et andet sted fra.
Du kan jo prøve Google og se hvor mange hit du får på BrowserLauncher

Hej igen,
Tak for de mange svar :)

1) Angående statistik funktion, så er det meget underligt! Hvis jeg ikke "mangler" programmet så fungerer det fint, men hvis jeg gør, så dur statistik funktionen ikke. Underligt!

2) God ide med en anden skrifttype :)

3) Jeg har arbejdet med ideen om at lave udskrivning i java, men jeg fik indtrykket af at det var meget omfattende. Det er måske slet ikke så farligt?

4)God pointe der med resize, det havde jeg ikke lige set. Må se om det er til at løse...

5)Jep readStatus skal selvfølgelig være med stor R.

6)Internationalisering lyder godt, hvordan vil man gøre det? Med proberties(filer)?

7)Ja Rasmus klassen skal virkelig splittes op. Men det sker lige så stille efterhånden som jeg får flyttet det programmet over i et IDE. Det gør det også lidt nemmere at rykke rundt på gui komponenter...

8)Hvor kan jeg finde noget om hvilke exceptions forskellige metoder osv. kan kaste?

9)Min ip er: 80.164.110.125, hvis nogen skulle have lyst til at test upload/download. brugernavn/pass: test/test

Tak indtil videre,
mvh
Torben

re 6)

Ja.

http://java.sun.com/docs/books/tutorial/i18n/index.html

re 8)

En god IDE vil fortælle dig løbende hvilke exceptions der
kan blive throwet.

Personligt kan jeg godt lide at udvikle i Eclipse (www.eclipse.org) men der findes masser af IDE, såå... :-)

re 1)
Jeg er ikke helt med hvad du mener ?

re 3)
Jeg har et link her:
http://www.apl.jhu.edu/~hall/java/Swing-Tutorial/Swing-Tutorial-Printing.html

Jeg er har experimenteret lidt med Oracle JDeveloper og Netbeans, så det bliver nok en af de to. Netbeans kan jeg bedst lide at lave gui i, men JDeveloper er two-way i modsætning til Netbeans som er one-way.

Angående 1)
Så har jeg lige prøvet mig frem og fundet ud af at hvis jeg ikke bruger JMangle på min kode, så kan jeg starte statistik funktionen uden problemer. Men hvis jeg JMangler koden, så kan jeg ikke starte statistik funktionen. Det er meget underligt! Så noget tyder på at der er en lille bug i JMangle :(

Angående 3)
Så vil jeg lige prøve at kigge lidt ydeligere på det, men som jeg forstår det, så tegner den jo bare på papiret hvad der er på skærmen (det man kan se af den komponent som man vil printe) og så bliver det jo lidt svært at få text, hvor der er scrollet ned osv. Jeg er dog ikke sikker, så det kan jo komme an på en test :)

--> Arne
Forrygende link til internalisering! Der er da lige til at gå til :o)
Angående exceptions, ja så skal jeg vist bare se at få spændt hjelmen og flyttet det hele over i et IDE...

Du kan evt. prøve med ProGuard i.s.f. JMangle:
  http://sourceforge.net/projects/proguard/

Jeg ved at aslan havde success med det da JMangle drillede ham.

Og ihvertfald bliver ProGuard i modsætning til JMangle
stadig opdateret.

Jeg har ikke prøvet at printe ud endnu så jeg ved faktisk ikke rigtigt hvordan det går for sig og jeg har hellere ikke prøvet det link jeg gav dig :-)

en anden lille detalje ... jeg ved ikke om det er pænt at bruge en deprecated metode? du bør vel bruge getPassword() istedet for getText() på jPasswordField

jeg kan også se at du starter outlook express ved tryk på email .. det er lidt irriterende hvis man bruger en anden
et alternativ kunne være at kalde noget ligende

Runtime.getRuntime().exec("cmd /c start mailto:" + jLabel1.getText()+"?subject=bla");

det skulle gerne launche den default email client .. ihvertfald under win2k/xp
det er så ikke særligt platforms uafhængigt .. men det var det jo ikke i forvejen

endnu en detalje .. du har sat download knappen til disabled som default.
men den bliver enabled når man forsøger at lave en connection .. den burde vel først blive enabled når connection er lykkedes

Hej Loadet, tak for dine kommentar :)
Ja jeg skal helt klart have gjort noget ved det resize værk!

Angående at bruge en deprecated medtode, ja så har du selvfølgelig ret - det vil selvfølgelig være en del smartere at bruge getPassword() istedet for og så typecaste char[] til en string (med String str = new String(charArray);)

Angående Runtime, så er den jo helt forrygende den linie du kommer med! Jeg har ledt og ledt men har ikke kunne finde den kommando, som starter default mail programmet, så det er lækkert!

Angående download knappen, ja så må jeg lige fikse det så den så den kun blive enabled hvis der er forbindelse.

Mange tak for kommentarene!


Hvis folk vil begynde at smide svar ville det være lækkert. Jeg regner bare med at dele pointene ligeligt.

Meget pedantisk betragtet er new String(charArray) ikke en type
cast men bare en constructor. Det rigtige type cast (String)
charArray virker ikke.

svar

Brug System.getProperty("file.separator") som separator. Jeg har nemlig problemer med at køre koden på linux, der jo bruger / i stedet for \.

--> Arne
Nå ja selvfølge, den måde jeg gør det på med new String er jo oprettelse af et objekt og ikke et typecast...

-->billen
God pointe! Returnerer System.getProperty("file.separator") "/" eller "\" alt afhængig af OS på den pågældende PC. Altså det er ikke en void funktion?

System.getProperty returnerer altid en String.

Og System.getProperty("file.separator") returnerer enten \ eller / alt
efter om det er Unix eller Windows.

Alternativt kan man konsekvent bruge /, da ikke-Unix systemer normalt forstår
det alligevel (inklusive Windows).

Svar

ok svar ;)

Jeg må gå ud fra at der ikke er flere, der er interesserede i point - siden der ikke er flere, som har smidt et svar.

Jeg vil sige mange tak for hjælpen og de mange ideer :)

mvh
Torben

det var så lidt

Du er velkommen - sig til hvis du får brug for hjælp igen :-)

hejsa.. har et problem.. håber i kan hjælpe.. men vil gerne lige spørge hvordan jeg kan slette min msn adr på windows msn på en anden computer så de ikke længere olgger på min msn adr når de tænder for deres computer og blokere min msn.. altså jeg ska ha slettet min msn adr fra en anden computer...??? mvh mester81

Du burde faktisk oprette det spørgsmål under en windows kategori og ikke her.
Men der findes et program der hedder noget med "wintweak" eller noget i den retning