Archief - [PROG][lazarus] testen

Het archief is een bevroren moment uit een vorige versie van dit forum, met andere regels en andere bazen. Deze posts weerspiegelen op geen enkele manier onze huidige ideeën, waarden of wereldbeelden en zijn op sommige plaatsen gecensureerd wegens ontoelaatbaar. Veel zijn in een andere tijdsgeest gemaakt, al dan niet ironisch - zoals in het ironische subforum Off-Topic - en zouden op dit moment niet meer gepost (mogen) worden. Toch bieden we dit archief nog graag aan als informatiedatabank en naslagwerk. Lees er hier meer over of start een gesprek met anderen.

Welke score geef je men programma? (denk er aan; kben beginner!)


  • Totaal aantal stemmers
    6
  • Opiniepeiling gesloten.

Jan_DM

Legacy Member
Hoi!

Kben voor school bezig aan een project, en het begint op z'n einde te komen.
Nu wil ik het volgende vragen: als iedereen die tijd heeft even 10 minuten met het programma bezig zou zijn (testen) en alle foutjes hier post, dan kan ik het programma verder optimaliseren.

Het gaat dus over een movie organizer.
Er zijn al enkele printbuttons maar deze zijn codeloos (codesuggesties voor lazarus zijn welkom!)

http://student.vub.ac.be/~jdemesse/movieorganizer.exe
EDIT: Link aangepast, bedankt Robain

Weinig moeite voor jullie, maar een groot voordeel voor mij.
Alvast bedankt!

The Crazy Frog

Legacy Member
1) Fullscreen, waarvoor is dat nodig :wtf:
2) Als je een movie toevoegt, kan je geen hoofdletters gebruiken.
3) 13 megabyte? Hoe heb je dat geflikt? :s
4) Persoonlijk zou ik het systeemlettertype gebruiken voor de GUI.
5) Steek er es een XP manifest in.

Voor de rest, redelijk goed voor een beginner, mits wat werk kan dit een toffe, nuttige applicatie worden.

killgore

Legacy Member
The Crazy Frog zei:
1) Fullscreen, waarvoor is dat nodig :wtf:
2) Als je een movie toevoegt, kan je geen hoofdletters gebruiken.
3) 13 megabyte? Hoe heb je dat geflikt? :s
4) Persoonlijk zou ik het systeemlettertype gebruiken voor de GUI.
5) Steek er es een XP manifest in.

Voor de rest, redelijk goed voor een beginner, mits wat werk kan dit een toffe, nuttige applicatie worden.
img-resources in het programma zelf mssch? (nog niet bekeken, kan er dus naast zitten ;))

edit: k, ligt dus daar nie aan

ma wth gast:
1) ik kan muis niet gebruiken voor te scrollen (edit: weird, nu wel, excuses).
2) slechte tab support (slechte veldvolgorde voornamelijk)
3) FOEILELIJKE UI (ik ben zelf geen groot licht op dat vlak, maar dit gaat erover) + uw ui is nog eens slecht uitgewerkt ook (voor alles apart venstertje = bad, steek alles in 1 overzichtelijk main window).
4) idd: fullscreen, wtf?
5) user selecten ma geen nieuwe aan te maken :s?
6) films dupliceren zich op vreemde wijze

edit: al 1 fout gekregen: "read past end"

Jan_DM

Legacy Member
killgore zei:
2) slechte tab support (slechte veldvolgorde voornamelijk)
3) FOEILELIJKE UI (ik ben zelf geen groot licht op dat vlak, maar dit gaat erover) + uw ui is nog eens slecht uitgewerkt ook (voor alles apart venstertje = bad, steek alles in 1 overzichtelijk main window).
4) idd: fullscreen, wtf?
5) user selecten ma geen nieuwe aan te maken :s?
6) films dupliceren zich op vreemde wijze

edit: al 1 fout gekregen: "read past end"


dank voor de feedback,

tabs worden aangewerkt.

UI is voldoende voor men taak ;) maar die full screen zal ik nog wel proberen afzetten. Twas idd beter geweest alles zoals de showall-form te doen.. maar deze heb ik als allerlaatste toegevoegd toen ik tijd had om met wa componenten te googelen :-)

da user ding was gewoon omda men databasewerkwijze te ingewikkeld zou worden (telkens 'c:\movie organizer\'+username+'\file.dat' is net iets te veel moeite :-)

fout 6 + "read past end" is vermoedelijk bij showall... dit is nu opgelost, binnenkort zal ik eens updaten.

Jan_DM

Legacy Member
The Crazy Frog zei:
2) Als je een movie toevoegt, kan je geen hoofdletters gebruiken.
5) Steek er es een XP manifest in.

tevens bedankt voor de feedback :)

2 is omdat de search anders te ingewikkeld is voor de user (hoofdlettergevoelig), maw als ie een film zonder hoofdletters zou invoegen en dan zoekt met hoofdletters krijgt hij geen resultaat... als er een manier is om de hoofdlettergevoeligheid bij read(bestand,array) weg te werken, ze zijn welkom

5 is me niet al te duidelijk..


enja, de images zijn in het programma ingeladen... die 13 mb zullen daar wel mee te maken hebben :)

L°rre

Legacy Member
Ik weet niet of het aan mij ligt, maar wanneer ik search doe dan loopt het programma om mysterieuze wijze vast.

The Crazy Frog

Legacy Member
Jan_DM zei:
tevens bedankt voor de feedback :)

2 is omdat de search anders te ingewikkeld is voor de user (hoofdlettergevoelig), maw als ie een film zonder hoofdletters zou invoegen en dan zoekt met hoofdletters krijgt hij geen resultaat... als er een manier is om de hoofdlettergevoeligheid bij read(bestand,array) weg te werken, ze zijn welkom

5 is me niet al te duidelijk..


enja, de images zijn in het programma ingeladen... die 13 mb zullen daar wel mee te maken hebben :)
2) Je kan toch met hoofdletters opslaan in de database en dan tolower() gebruiken bij de searches, dan is uw search niet hoofdlettergevoelig
5) Een XP manifest is een xml resource die dervoor zorgt dat uw applicatie een XP look krijgt in plaats van een win98 look.

Jan_DM

Legacy Member
beoordelingen + foutmeldingen zijn nog steeds welkom

aan alle mensen die al gereageerd hebben, proficiat!
aan alle mensen die nog niet gereageerd hebben, maar dit wel zullen doen, euh, ook proficiat!

L°rre

Legacy Member
Ik heb mijn fout gevonden.

Wanneer ik search doe naar een film die ik nog niet heb toegevoegd, dan maakt hij het venster inactief. Omdat het programma ook fullscreen is, kan je het dus niet meer selecteren.

Esc drukken maakt het scherm weer actief.

Jan_DM

Legacy Member
idd een probleempje... eens kijken wat ik kan doen

normaal geeft ie
showmessage('movie not found');

da lukte dan ook lange tijd, ma blijkbaar heeft een wijzigingetje in de code daar een stok in het wiel gestoken.

Jan_DM

Legacy Member
fout gevonden; de shomessage staat gewoon achter het venstertje; die stayontop maar terug verwijderd... :)

edit: nieuwe versie online

killgore

Legacy Member
:offtopic:Ik heb geen tijd om de nieuwe versie te checken, maar toch even dit:

knap dat je "zwaar afbrekende" commentaar nog goed opneemt :niceone:! De meesten zullen dergelijke commentaar niet appreciëren (om het mooi te verwoorden :p).

(complimenten mogen ook eens gegeven worden imho)

Jan_DM

Legacy Member
zolang die commentaar nuttig kan zijn voor men project en niet persoonlijk is, ben ik allen dankbaar :)

kdenk da nu alle foutjes eruit zijn, behalve 1tje da ik ga voorleggen aan de assistent (kvind er geen oplossing voor)

although, moest er nog iets zijn... spui maar uit!

Messias.

Legacy Member
Vanuit een meer esthetisch oogpunt, Comic Sans is not done. Da's een letterype voor COMICS. Die grote buttons zijn onpraktisch en niet mooi.

Always keep in mind, less is more.

dJeez

Legacy Member
Als ik zo ff naar die source code kijk krijg ik koude rillingen en de neiging om ze naar de prullenmand te verwijzen. Lazarus enkel als RAD tool misbruiken is niet echt ok wat mij betreft. Ik zou toch wat properder proberen te werken.

Dingen die mij storen :
- Allemaal identieke aparte bestanden met records van 200 kars lang terwijl alles in 1 bestand kan? Begrijpe wie kan. Waarom trouwens nog met losse bestandjes werken als er een embedded DB als SQLite bestaat die je op zowat elk denkbaar platform kan gebruiken? Het wiel moet niet altijd opnieuw uitgevonden worden.
- Geen OO? En daarbovenop zelfs zo goed als niet gestructureerd, ik zie gigantisch veel copy/paste werk.
- Naamgeving (nu ja, het ontbreken ervan) van variabelen, controls, bestanden
- Totaal gebrek aan code indentation - die code is niet te onderhouden op lange termijn
- Hardcoded vaste paden = big NONO
- Schijnbaar totaal gebrek aan exception handling

En hier stop ik maar :p. Kortom van iemand die blijkbaar op univ zit zou ik toch echt wel een beter resultaat verwachten...

Jan_DM

Legacy Member
oei, was ik vergeten vermelden dat het Toegepaste Economische Wetenschappen zijn, niet informatica? :p
we hebben 7 weken delphi-lessen achter de rug van telkens 2 uur. we hebben welgeteld 1 les gespendeerd aan databases :), so don't blaim me als ik gemakkelijkheidshalve alles in aparte filetjes heb gezet..

even ter vergelijking: dit is de doelstelling om een 14/20 te hebben:
http://student.vub.ac.be/~jdemesse/Bank_c_drive.exe
paswoord: céline

oja, op algemeen verzoek ben ik net aan een aangepaste, minder grootse lay-out begonnen.. Over 1 week gaat het geheel trouwens uit de beta-phase en zal het volmondig de final version gedoopt worden


aan dJeez:

wat is OO eigenlijk?

weet jij misschien hoe het volgende op te lossen (je klinkt als een kenner); een listbox, selectionchange (zie "show all"); hoe kun je de procedure stoppen wanneer er onder het laatste item geklikt wordt?

Jan_DM

Legacy Member
na een nachtje door gedaan te hebben is de nieuwe layout volledig af... enkele functies moeten nog wel terug in gang gezet worden maar de basis is er.

bovendien gaat er nu ook de mogelijkheid zijn een eigen user te maken maar aangezien ik nog problemen heb met nieuwe users creeren is het voorlopig gesimuleerd door user 'jan'
alles bevindt zich in 1 venster, geen full screen... nuja, neem zelf maar een kijkje.

Vanaf nu spreken we trouwens van 'ProjectMovieOrganizer' (PMO) :)

bestand van 13 naar 6MB :p
code van 4.5 naar 0,6 :p

Cypher

Legacy Member
ik heb t ook even ne keer gerunned:
- "Welcome to Movie organizer. Navigate through the program with the tabs above this text"

naja veel verder heb ik niet gekeken, maar onder wine loopt t niet zo lekker ;)
Het archief is een bevroren moment uit een vorige versie van dit forum, met andere regels en andere bazen. Deze posts weerspiegelen op geen enkele manier onze huidige ideeën, waarden of wereldbeelden en zijn op sommige plaatsen gecensureerd wegens ontoelaatbaar. Veel zijn in een andere tijdsgeest gemaakt, al dan niet ironisch - zoals in het ironische subforum Off-Topic - en zouden op dit moment niet meer gepost (mogen) worden. Toch bieden we dit archief nog graag aan als informatiedatabank en naslagwerk. Lees er hier meer over of start een gesprek met anderen.
Terug
Bovenaan