Open Source Trash Mail PHP Script

Michael · 21. März 2011, 14:48

Hallo zusammen,

Ich habe ein Open Source Trash Mail Script entwickelt und wollte mal Eure Meinung dazu hören.

Webseite: wegwerf-e-mail-adresse.de/
Repository: github.com/msoftware/Trash-Mail
Demo: 1pad.de/

Ich würde mich über euer Feedback freuen. Natürlich sind auch Wünsche und Kritik immer gerne gesehen.

Gruß
Michael

vince · 21. März 2011, 15:30

Hi,

postiv ist auf jedenfall, das eine Doku, Api und Source vorhanden ist :thumbup:

Was mir nicht gefällt, ist die Namensgebung der Dokumente. Das Denglisch ist gruselig -> "about", "kontakt", "impressum", "empfehlen" ...

Auch nicht schön finde ich die vielen Dokumente. Wir sind im Jahr 2011, da braucht man nicht für jedes Script ein Dokument erstellen und Code mit Ausgaben mischen.
Das Projekt ist nicht riesig, aber man kann auch bei kleineren Projekten das MVC Konzept umsetzen. Ich meine nicht das man mit Kanonen auf Spatzen schießt, es ist aber auch kein Problem alles abgespeckter zu implementieren.

Der mir! fehlende Objektorientierte Ansatz lehnt sich an den vorigen Kritikpunkt an. Mit PHP 5(.3) kann man schon ganz gut Oo-programmieren.
Nichts ist gegen den Imperativen Stil auszusetzen, aber ich persönlich finde das nicht mehr zeitgemäß.

Aber da das ganze ja funktionell fertig zu seien scheint (abgesehen davon, das man niemals fertig wird ^^), wäre das refactoring ja kein Problem

Torben Brodt · 21. März 2011, 20:38

Ich finde es erstens gut, weil es Open Source ist, und außerdem finde ich das Design ganz nett.
Beim Code fehlt natürlich der MVC Ansatz. Die einzelnen Dateien sind sehr wartungsaufwändig, wenn du doch mal etwas ändern willst.
Auf 1pad.de hingegen sehe ich übrigens alles in Times New Roman. Da fehlt eine "Sans" Alternativschrift zu Verdana.
Lg

Michael · 22. März 2011, 14:06

Erst mal Danke für die Antworten. Heute ist auch schon eine neue Version erschienen. Die Version hat die Nummer 0.8.1 und kann jetzt QR Codes. Wie Ihr seht, konzentriere ich mich mehr auf die Features/Benefits für den User als für eine ausgereifte Architektur. Sorry, aber ich bin halt eher der pragmatische Programmierer ....

Zum Denglisch: Sorry, but das werde ich wohl in meinem Leben nicht mehr away kriegen ;-).

Zu MVC: MVC macht da Sinn, wo MVC sinn macht. In dem Fall habe ich mich dagegen entschieden. aus einem einfachen Grund. Bei einem so einfachen Script kann man MVC machen, aber einen echten Vorteil (wie z.B. bei großen JSF Projekten) hat man hier nicht. Werde aber darüber nachdenken, dass evtl. mal nachzurüsten. Z.B. mit dem Zend Framework. Da habe ich gute Erfahrungen gemacht.

Sans: OK, die Linux Gemeinde muss momentan noch etwas leiden

- wird in der nächsten Version beseitigt.

OO: Ok, habe ich auch drüber nachgedacht, aber in PHP habe ich keinen echten Vorteil durch OO und das Projekt ist bei weitem nicht komplex genug um durch den OO Ansatz einen besseren Überblick zu erhalten. Ist aber sehr einfach nachzurüsten. Da gibt es sogar Scripte, die so was automatisch machen. Wenn das Trash Mail Script weiter wächst, werde ich das auf jeden Fall mal umstellen.

Gruß
Michael

rswhite · 22. März 2011, 16:22

Warum hast du in PHP keinen echten Vorteil von OO?
Klare, getrennte Daten, wiederverwendbar, leichter zu warten, ich sehe nur Vorteile.
Und naja, wenn ein User das mal erweitern/anpassen will/muss, muss er sich in statischen und durcheinander gewürfelten Code einlesen, dann hat er mehr Probleme als Vorteile.
Da wäre eine OO API auch in angespeckter Form hilfreicher.

Szabo · 22. März 2011, 22:41

OOP ist immer wartungsfreundlicher und gerade für andere ist es einfacher sich einzuarbeiten. Würde also auch bei - egal wie klein das Projekt ist - auch in PHP mit OOP arbeiten.

Torsten · 22. März 2011, 22:51

Sieht sehr nett aus!

Bis auf den zusammengewürfelten Code.

Eventuell falls du doch mal in Richtung MVC gehen möchtest. Ich hab mir als Basis für ein Projekt kissmvc.com/ (hat auch ein oder zwei Mängel) genommen ...

Das mit den QR Codes ist eine geile Idee!

Eventuell noch nen schöneres Interface, aber darum kann man sich ja auch später kümmern ...

Michael · 23. März 2011, 12:28

@rswhite: Natürlich hat man einen Vorteil durch OO, egal in welcher Programmiersprache. Aber in meinem Fall habe ich mich dagegen entschieden, weil das Projekt sehr klein und übersichtlich ist. Es gibt keinen Grund hier Klassen zu definieren oder Objekte einzusetzen. Ist halt eine Architekturentscheidung. Außerdem habe ich die Erfahrung gemacht, dass es einfacher ist in einem Projekt OOP nachzurüsten, als es später wieder rückzubauen. Ist zwar in diesem Fall kein Problem, da Open Source und kein Auftraggeber im Nacken, der kein OOP haben will.

@Torsten: Danke, dass es Dir gefällt. QR Codes sind im Moment einfach cool und da wollte ich auch einen haben. Das mit dem zusammengewürfelten Code ist halt so eine Sache, ich baue auf Funktionen und Code-Schnipsel auf, die in meinem Repository seit fast 5 Jahren wachsen und gedeihen. kissmvc.com/ kannte ich noch nicht, habe immer framework.zend.com/manual/1.11…ing.quickstart.intro.html genutzt. Für das Interface habe ich ja das Template vorgesehen. Ist zwar auch keine richtige Template Engine, aber da werde ich später wohl mal Smarty einführen.

@All: Allen ein herzliches Dankeschön für die vielen Anregungen.

Gruß
Michael

Michael · 28. März 2011, 15:38

Hi,

habe mir die Mühe gemacht, und das Löschen von E-Mail implementiert. Habe ein wenig mit jQuery gespielt und finde, ich habe das ganz passabel hinbekommen.

Feedback ist wie immer sehr willkommen

Gruß
Michael

Torsten · 29. März 2011, 00:19

Bug? Feature?

1pad.de/?search=

Also das ich alle Emails einsehen kann?

lg

Michael · 29. März 2011, 08:37

Bug ... behoben

Szabo · 29. März 2011, 08:58

Ich finde die Mailanzeige ein wenig unübersichtlich. Erstell doch (wie bei jedem Mailprogramm) erstmal nur eine Liste/Tabelle mit Datum, Absender, Betreff und wenn man auf eine Mail drauf klickt, kannst du ja (via JavaScript evtl.) den Content der Mail anzeigen.

djonny · 29. März 2011, 09:49

muss ich szabo recht geben also ich hab mir das gerade angsehen, so ist es ja eh schön aufgebaut aber wenn ich dann ein Postfach abrufe hätte ich gerne eine kurze Übersicht mit Mails und nicht so große kästchen mit Inhalt.

Weiteren Bug bzw. Feature? wenn ich das Postfach "*" abrufe kommen ziemlich viele e-mails

Michael · 29. März 2011, 10:22

OK, jetzt habe ich eine E-Mail Validierung ergänzt. Damit sollte der Bug endgültig behoben sein - hoffe ich.

Michael · 29. März 2011, 10:26

@djonny, @Szabo: Kann ich verstehen. Aber in der Regel hat man ja nur eine oder maximal 2-3 E-Mail Adressen im Posteingang so dass die Übersicht nicht so sehr leidet und man alle E-Mails auf einer Seite sehen kann, ohne auf die Folgeseite springen zu müssen. Die Idee mit dem Aufklappen per JS finde ich gut. Werde mal darüber nachdenken und evtl. noch ein wenig mit jQuery spielen.

djonny · 29. März 2011, 12:13

also der Bug is noch immer da zwar nicht mehr mit * aber dafür funktioniert % / - + ....

allerdings nach 5x kommt die Fehlermeldung "Cannot connect to Gmail: Too many login failures"

Michael · 29. März 2011, 12:43

Da muss ich mir wohl noch mal was anderes überlegen ...

Michael · 29. März 2011, 13:11

Für jedes Problem gibt es eine Lösung ... mal sehen, was als nächstes kommt

PS: Vielen Dank schon mal für die fleißigen Tester, die mir helfen, die Software besser zu machen.

Szabo · 29. März 2011, 14:40

Mach beim Login einfach folgende Überprüfung:

Quellcode

if(preg_match('#[a-zA-Z0-9.-_]+#', $_POST['foobar'])) {
// true
}

Dann kann man sich mit %, +, ... etc. nicht mehr einloggen.

Michael · 29. März 2011, 14:56

@Szabo:
Aber noch mit .

Open Source Trash Mail PHP Script

Open Source Trash Mail PHP Script

Quellcode

Teilen

Benutzer online 1

Tags