Kommentartråd: The Sigma Project - Et Dropbox-inspirert prosjekt

[javanuben]

Dette er kommentartråden til https://www.diskusjon.no/index.php?showtopic=1381852!

 

Alle tilbakemeldinger mottas med stor takknemlighet!

Endret 11. oktober 2011 av javanuben

[torbjørn marø]

Hva med å opprette et prosjekt på github for dette?

 

1. Du får erfaring med verdens beste kildekodekontrollverktøy
   
2. Alle kan følge med på hvordan prosjektet utvikler seg
   
3. Hvis noen får lyst kan de komme med bidrag du kan inkludere om du vil
   

 

Lykke til uansett!

[javanuben]

Takk for forslaget Torbjørn! Jeg har hørt mye om github, men egentlig aldri satt meg inn i det. Kanskje dette er en fin anledning til det. Jeg skal se litt på saken, så kanskje jeg får opprettet et prosjekt på github!

[torbjørn marø]

Tok en rask titt på koden din, og det så ganske ryddig og fint ut. Flere ting jeg kunne tenke meg å kommentere, men det første er at du i FileSender.java har stor grad av kodeduplisering mellom sendFile og sendFileSize. Vurder ulike måter å trekke ut det som er felles til en felles metode, og bruk denne i hver av dem.

 

Det samme gjelder for FileReceiver.java.

 

Hvordan liker du git / github sålang forresten?

Endret 11. oktober 2011 av torbjørn marø

[javanuben]

Takk for tilbakemeldingen! Jeg ser hva du mener.. Ser ut som det gikk litt fort med copy-paste der. Skal prøve å få skrevet om FileSender og FileReceiver i løpet av dagen.

 

Jeg liker konseptet med git / github veldig godt. Dog hadde jeg noen problemer i starten (fant f.eks. ikke 'rm'-kommandoen da jeg googlet "remove file github", men mange andre rare måter som fungerte mer eller mindre bra). Men det går bedre nå, og jeg begynner å trives med det.