Autor Beitrag
TomB
Hält's aus hier
Beiträge: 1


Delphi 3 Pro
BeitragVerfasst: Do 21.06.07 21:47 
Filescan

Dieses kleine Programm scannt einfach die Dateien in einem einstellbaren Verzeichnis [incl. Unterverzeichnissen] und gibt sie in einem Memo aus.

wenn jemand noch optimierende Vorschläge hat = immer willkommen.
Einloggen, um Attachments anzusehen!
_________________
PGP ID 0xF73CD23F
Wolle92
ontopic starontopic starontopic starontopic starontopic starofftopic starofftopic starofftopic star
Beiträge: 1296

Windows Vista Home Premium
Delphi 7 PE, Delphi 7 Portable, bald C++ & DirectX
BeitragVerfasst: Fr 22.06.07 16:46 
entscheide dich erstmal, ob du deutsch oder englisch machen willst ;)
"Verzeichnis:", "Paste", "Exit", "File_Search"...

Außerdem sollte man nicht schon bei der Verzeichnis-Asuwahl sehen können was direkt in dem Verzeichnis liegt...

Dann könnte man noch ein Eingabefeld machen, mit dem man dann nur nach bestimmten dateien suchen kann, weil einfach alle Dateien anzeigen lasse ist in meinen Augen eher sinnlos...
Leuchtturm
ontopic starontopic starontopic starontopic starontopic starontopic starhalf ontopic starofftopic star
Beiträge: 1087

Win Vista, Knoppix, Ubuntu
Delphi 7 Pe, Turbo Delphi, C#(VS 2005 Express), (X)HTML + CSS, bald Assembler
BeitragVerfasst: Fr 22.06.07 17:06 
Ich habe mir jetzt nur den Screenshot angekuckt. Dabei ist mir aufgefallen, dass das alles unstruckturiert ist. Unterschiedliche Buttongrößen, unterschiedliche Schriftgrößen, Deutsch und englisch.

_________________
Ich bin dafür verantwortlich was ich sage - nicht dafür was du verstehst.
Wolle92
ontopic starontopic starontopic starontopic starontopic starofftopic starofftopic starofftopic star
Beiträge: 1296

Windows Vista Home Premium
Delphi 7 PE, Delphi 7 Portable, bald C++ & DirectX
BeitragVerfasst: Fr 22.06.07 17:32 
das auch... also nichts wirklich gutes...
JayEff
ontopic starontopic starontopic starontopic starontopic starontopic starontopic starofftopic star
Beiträge: 2971

Windows Vista Ultimate
D7 Enterprise
BeitragVerfasst: Fr 22.06.07 17:45 

  • Komponenten einigermaßen übersichtlich anordnen (Größen aneinander anpassen)
  • Button-Beschriftungen entweder Deutsch oder Englisch
  • Hotkey für Exit auf *nur* x legen ist Schwachsinn, lieber Alt+X
  • Hotkey funktioniert nur, wenn kein Textfeld angewählt ist
  • Paste löscht das Editfeld, auch wenn die Zwischenablage leer ist
  • Das Ordner-Auswählen-Fenster führt bereits die Hauptfunktion des Programms aus
  • Das Ordner-Auswählen-Fenster sieht bescheiden aus. (Linie mitten im Fenster :?!?:)
  • Titel des Fensters: "FileScan 1.0", Titel der Application: "Inno Helber" :?!?:
  • Dateiliste im Ordner-Auswählen-Fenster ist nicht scrollbar (Enabled=False?)
  • Memo im Hauptfenster ist nicht ReadOnly=True, man kann also reinschreiben
  • Erst ein Doppelklick auf einen Ordner im Ordner-Auswählen-Fenster bewirkt die Auswahl. Einfachklick+OK reicht nicht.
  • Während des Scans wird das Programm zur hungapp, "(Keine Rückmeldung)"
  • Ich kann nicht genau sagen, was das bedeutet, aber das Programm kommt schnell über die 5 Mio Seitenfehler im Taskmanager
  • Das Downloadarchiv enthält seltsame Dateien + ein weiteres Archiv, in dem dann erst der Source ist
  • Boolean-Ausdrücke brauchen nicht erst per = in Boolean-Ausdrücke verwandelt werden if (key = 27and (Panel1.Visible = True) sollte man der Übersicht halber als if (key = 27and Panel1.Visible schreiben.
  • Die Datei pgpkey.asc interessiert uns nicht wirklich ;)
  • Die Prozedur für Button4 ist noch vorhanden, obwohl dieser nicht(mehr?) existiert.
  • Anstelle einer neuen Form als Öffnen-Dialog oder gar diesem genialen Code legst du ein Panel über die alten Komponenten, auf dem die Ordnerwahl statt findet :shock: Nicht gut.
Das ist mal alles, was mir aufgefallen ist. Nur negative Punkte, aber lass dich davon nicht entmutigen ;)
Dann mach dich mal ans Verbessern :zustimm:

_________________
>+++[>+++[>++++++++<-]<-]<++++[>++++[>>>+++++++<<<-]<-]<<++
[>++[>++[>>++++<<-]<-]<-]>>>>>++++++++++++++++++.+++++++.>++.-.<<.>>--.<+++++..<+.
chest3rs
Hält's aus hier
Beiträge: 11


Delphi 2007
BeitragVerfasst: Di 18.09.07 13:11 
Hi Leute!

Also eine sachen wollte ich vorab mal fragen: Seid ihr programmierer oder lernt ihr gerade englisch in der schule???
wenn es hier um design gehen würde, dann solltet ihr mal mit Photoshop anfangen, aber hier passt so eine komische beschwerde, dass die buttons nicht groß genug sind und so weiter nicht rein....
es geht um die funktion des programms, und wenn ihr sie nicht versteht, dann lasst es lieber. es ist nämlich nur destruktiv! JayEff, machs erstmal besser und bleib beim thema! es geht nicht darum, wie man es hochläd, sondern was.

so hier hab ich ne neue version geschrieben. diese ist viieel schneller, 160GB in 30 sek. auf nem 3000+ mit 1GB RAM.
sorry, dass die buttons nicht schön sind und das programm nicht von einem designer erstellt wurde... ;-)
Einloggen, um Attachments anzusehen!
Gausi
ontopic starontopic starontopic starontopic starontopic starontopic starofftopic starofftopic star
Beiträge: 8549
Erhaltene Danke: 478

Windows 7, Windows 10
D7 PE, Delphi XE3 Prof, Delphi 10.3 CE
BeitragVerfasst: Di 18.09.07 13:32 
Mal langsam, ok?

Zu einem guten Programm gehört nicht nur, dass es funktioniert, sondern auch, dass es übersichtlich ist, intuitiv bedienbar ist und dass der Bedienung ein gewisses Schema zugrunde liegt. Insofern sind Anmerkungen zum GUI durchaus angemessen und gehören hierhin. Und auch die Bezeichnung der Bedienelemente gehört dazu.

Design bedeutet nicht immer gleich bunte Bilder mit tollen Photoshop-Effekten!

Und: Bitte füge die neue Version in das erste Posting mit ein, damit man die aktuelle Version bekommen kann, ohne den ganzen Thread lesen zu müssen. Danke!

_________________
We are, we were and will not be.
chest3rs
Hält's aus hier
Beiträge: 11


Delphi 2007
BeitragVerfasst: Di 18.09.07 13:35 
schon gemerkt, dass das programm von TomB ist? außerdem ist es leider so, das in den foren nur um die form gestritten wird. ich finde es sehr schade, wenn ich ein programm hochlade und niemand schreibt etwas zu der funktion. zB du, dass ich das programm oben updaten soll... hallo!? was hat das für ein sinn? sollen wir darüber diskutieren, ob alles am rechten platz ist? dafür hab ich mich hier nicht angemeldet...


Zuletzt bearbeitet von chest3rs am Di 18.09.07 13:39, insgesamt 1-mal bearbeitet
Gausi
ontopic starontopic starontopic starontopic starontopic starontopic starofftopic starofftopic star
Beiträge: 8549
Erhaltene Danke: 478

Windows 7, Windows 10
D7 PE, Delphi XE3 Prof, Delphi 10.3 CE
BeitragVerfasst: Di 18.09.07 13:38 
Ehrlich gesagt, nein. :oops:

Der erste Teil meines Postings bleibt aber uneingeschränkt gültig.

Für den zweiten Teil: Dann wäre es evtl. sinnvoll, einen eigenen Thread zu erstellen, um das Programm dort vorzustellen - sofern du vorhast, dass noch etwas auszubauen. Z.B. bei einem Vergrößern des Fensters die enthaltenen Komponenten auch in der Größe anzupassen.

_________________
We are, we were and will not be.
chest3rs
Hält's aus hier
Beiträge: 11


Delphi 2007
BeitragVerfasst: Di 18.09.07 13:42 
ich wollte nur den schreiber auf die verbesserung hinweisen. und schließlich ist es seine aufgabe, den text weiterzuschreiben. klar kann man am aussehen rumnörgeln, aber wenn ich die funktion benutze, dann füge ich sie in ein neues programm ein.
einfach schade, das die kritik an der form das eigentliche thema überschattet und ich nichts vernünftiges zu lesen bekomme.
Kroko
ontopic starontopic starontopic starontopic starontopic starontopic starontopic starhalf ontopic star
Beiträge: 1284

W98 W2k WXP
Turbo D
BeitragVerfasst: Di 18.09.07 13:43 
(a) button1 ist "sehr aussagekräftig"
(b) drückt man drauf dann stockt der Rechner und irgendwann werden bei mir 39924 Dateien im linken Feld aufgelistet (wer soll die durchschauen)
(c) in einem Label(?) erscheint dann auch noch diese Zahl (ja toll)
(d) der Progressbalken steht und steht und steht ...
(e) im oberen Eingabefeld habe ich mal '"C:\*.bmp' eingeben -> er listet alle 39924 Dateien auf
(f) ScreenShot und Programm stimmen nicht mehr überein
(g) Download bitte nur über 1.Eintrag
(h) ...

_________________
Die F1-Taste steht nicht unter Naturschutz und darf somit regelmäßig und oft benutzt werden! oder Wer lesen kann, ist klar im Vorteil!
chest3rs
Hält's aus hier
Beiträge: 11


Delphi 2007
BeitragVerfasst: Di 18.09.07 13:45 
Titel: mhh
hättest du gelesen, wofür es gut ist, dann würdest du es wissen...

bald halte ich es hier nicht mehr aus...

k, habs kapiert. Anscheinend bin ich hier von profis umgeben!
Kroko
ontopic starontopic starontopic starontopic starontopic starontopic starontopic starhalf ontopic star
Beiträge: 1284

W98 W2k WXP
Turbo D
BeitragVerfasst: Di 18.09.07 13:59 
Titel: Re: mhh
user profile iconchest3rs hat folgendes geschrieben:
hättest du gelesen, wofür es gut ist, dann würdest du es wissen...

...


Ich hoffe mal, Du meinst nicht mich, oder
ich habe doch Button1 gelesen, was anderes stand ja nicht da, oder :wink:

_________________
Die F1-Taste steht nicht unter Naturschutz und darf somit regelmäßig und oft benutzt werden! oder Wer lesen kann, ist klar im Vorteil!
JayEff
ontopic starontopic starontopic starontopic starontopic starontopic starontopic starofftopic star
Beiträge: 2971

Windows Vista Ultimate
D7 Enterprise
BeitragVerfasst: Di 18.09.07 16:21 
user profile iconchest3rs hat folgendes geschrieben:
JayEff, machs erstmal besser und bleib beim thema!

Hä? Ich les wohl nicht recht? Ich überlese nochmal kurz meinen Post...

user profile iconJayEff hat folgendes geschrieben:

  • Komponenten einigermaßen übersichtlich anordnen (Größen aneinander anpassen)
  • [...Weitere Verbesserungsvorschläge...]
Das ist mal alles, was mir aufgefallen ist. Nur negative Punkte, aber lass dich davon nicht entmutigen ;)
Dann mach dich mal ans Verbessern :zustimm:

:gruebel: In diesem Thread gehts um das gepostete Programm. Ich habe Tipps gegeben, wie man es verbessern kann. Wo hab ich da das Thema verfehlt? :gruebel:
Zum Thema "Besser machen": Könnte ich, aber was hätte der Programmierer davon? Ich kann's ja schon, er will's lernen...
Tut mir leid, aber dass du dem Programmierer mal eben ein fertiges Programm lieferst, hat ihm sicherlich weniger gebracht als meine Tipps.
Was Kritik betrifft: Nun, die Leute posten hier ein Programm, und erwarten dann, dass wir nur seine Funktion bewerten, die durch Klick auf "Button1" ausgelöst wird?! Hackts? Welcher 0/8/15 Benutzer würde so ein Programm benutzen? Hier gehts nicht darum, eine Funktion zu schreiben, sondern um ein Programm. Wär's nur die Funktion, hätt man's vielleicht besser in Open Source Units packen sollen, nämlich nur den Auszug mit der Funktion, so dass Entwickler testen können, ob die Funktion gut ist oder nicht. Aber wie bereits erwähnt, gehts hier um ein Projekt das für den Endverbraucher gedacht ist.
Sieh dir mal den Nemp von Gausi an und seinen 11 Seiten langen Thread. Du siehst hier massenweise "Kleinkram" wie in letzter Zeit die Englisch Grammatikfehler in seinem Programm, kleinere Designschnitzer, aber auch "richtige" Bugs. Und über beides freut er sich gleichermaßen, da ja eigentlich in jedem ein kleiner Perfektionist steckt, oder? ;)
Wie auch immer, überleg dir mal, ob du dich hier wirklich vorbildlich verhalten hast.
Und bevor ich noch mehr Off Topic poste, lass ichs lieber...

_________________
>+++[>+++[>++++++++<-]<-]<++++[>++++[>>>+++++++<<<-]<-]<<++
[>++[>++[>>++++<<-]<-]<-]>>>>>++++++++++++++++++.+++++++.>++.-.<<.>>--.<+++++..<+.
Wolle92
ontopic starontopic starontopic starontopic starontopic starofftopic starofftopic starofftopic star
Beiträge: 1296

Windows Vista Home Premium
Delphi 7 PE, Delphi 7 Portable, bald C++ & DirectX
BeitragVerfasst: Di 18.09.07 16:39 
Wenn man sich mal den Quellcode anschaut ist EditDir überflüssig... das ist da einfach nur, aber machen tuts nichts...
und diese Abfrage:
ausblenden Delphi-Quelltext
1:
2:
3:
if Length(EditDir.Text) = 3
 then GetFiles(ShellTreeView1.Path)
  else GetFiles(ShellTreeView1.Path + '\');
ist auch sinnlos... wieso denn kontrollieren, ob EditDir 3 Zeichen lang ist??????!?

und wenn du schon JayEff anmeckerst und meinst er sollte es besser machen, dann mach doch erstmal selber zum beispiel ein Application.ProcessMessages rein...

Ich hör lieber jetzt schon auf zu schreiben, sonst wird mein Account noch gelöscht wegen meiner Ausdrucksweise...

Aufregen könnt ich mich... und wie!!

Noch so am Rande: Vielleicht sollte man erst das Thema verstehen, bevor angeblich andere Leute das Thema verfehlen...