Benutzer ändern und Email versenden

Mano · 05.04.2005, 14:45

Folgendes Script nutze ich zum Ändern meiner User in der Datenbank. Jedoch ist dieses nicht sehr schön aufgebaut.

Die gleiche Abfrage besteht nochmal für !empty($Passwort), dabei wird zusätzlich das Passwort mit in der Datenbank gespeichert.

Nun möchte ich das, wenn der Benutzername oder Passwort oder User Email geändert er eine Mail versendet. Wie wäre dabei eine vernüftige Abfrage. Genauso gefällt mir nicht das ich zwei if Abfragen habe, wenn das Passwort leer oder nicht leer ist. Hat da jemand eine bessere Idee? :wink:

?>[/php]

kid01 · 05.04.2005, 15:07

naja so ist es zumindes übersichtlich

warum willst du diese schöne strucktur zerstören????

Wo ist bei den Mails dein Problem????? Am ende jeder if kannst du doch ohne Probleme ne mail funktion einbauen

Mano · 05.04.2005, 15:13

Weil ich das nicht "so" machen will, da der Inhalt der Mail immer der gleiche ist?!

PHP-Code:

  <?php

if($blakp1 != $benutzername) {

mail(.....);

}

 
if($blakp2 != $email) {

mail(.....);

}

 
// usw....

?>

Kort · 05.04.2005, 15:18

Du kannst ja den Inhalt vorher in einer Variablen ablegen.

05.04.2005, 15:20

Zitat:

Zitat von Mano

Die gleiche Abfrage besteht nochmal für !empty($Passwort), dabei wird zusätzlich das Passwort mit in der Datenbank gespeichert.

ich würd dir else { ... } empfehlen
und es gibt für if ja auch operatoren:
&& = logisches und
|| = logisches oder
if(($blakp1 != $benutzername) || ($blakp2 != $email)) {
..
}

meintest du sowas?
ansonsten kann ich mich nur kid anschliessen...

Mano · 05.04.2005, 15:25

Ja, das mit dem "oder" klappt jedoch nicht in allen fällen. Mir wird schon was einfallen.

Jedoch ist mir gerade was ganz anderes eingefallen.

Wenn ich den Benutzernamen ändere muss ich ja überprüfen, ob dieser schon besteht. Da die BenutzerID schon übergeben wird kann ich daher den alten Benutzernamen auslesen und in einer Select Abfrage fragen wo dieser alte Benutzername nicht vorhanden ist. Oder würdet ihr dieses anders lösen?

Mano · 05.04.2005, 16:28

Wäre das so korrekt mit der Abfrage? (Logik)

robo47 · 05.04.2005, 16:52

select * from
Warum soll ich nicht SELECT * schreiben?

http://www.php-faq.de/q/q-sql-select.html

mysql error

man sollte in seinen Scripten immer mysql_error() verwenden, in etwa so:

PHP-Code:

  $sql = "SELECT feld1,feld2 FROM table WHERE id=5";

$result = mysql_query($sql) or die(mysql_error().'
Query: '.$sql);

Mano · 05.04.2005, 17:55

PHP-Code:

  <?php

select * from 

?>

Das wird schon mit bedacht verwendet, den das Script umfasst nicht nur die paar Zielen da oben *lol*. Außerdem ist das einfacher als wenn ich 10 Spalten einzeln davorschreibe auf nen paar mehr oder weniger kommts da auch nicht an *lol*.

Ja stimmt das error fehlte noch in dem Beispiel. Es ging ja hierbei auch nicht darum sondern um die Logik

robo47 · 05.04.2005, 18:02

Zitat:

Das wird schon mit bedacht verwendet

Zitat:

Außerdem ist das einfacher als wenn ich 10 Spalten einzeln davorschreibe

klingt danach dass du

a) faul bist :P
b) die seite nicht wirklich komplett gelesen udn verstanden hast

aber muss jeder selbst wissen ob er sauber programmiert und damit quellcode schafft der gut zu durchblicken udn selbst erklären ist, oder halt mit * arbeitet :P

Zitat:

auf nen paar mehr oder weniger kommts da auch nicht an

.... jedes query belatet die datenbank und unnötig muss es nicht sein, weil unter allem die geschwindigkeit leidet.

abgesehen davon was soll das hier bewirken:

PHP-Code:

  <?php

if(mysql_fetch_array($sql_benutzer)) { 

?>

das wird nur false, wenn du nen massiven fehler im query hast. und dann soltle eigentlich das or die mysql_error() eingreifen. Kann es sein dass du eher mit mysql_num_rows überprüfen willst ob ein datensatz ankam?

mfg
robo47

Themen-Optionen
Druckbare Version zeigen Jemanden per E-Mail auf dieses Thema hinweisen
Thema bewerten
Thema bewerten:

Ähnliche Themen
Thema	Autor	Forum	Antworten	Letzter Beitrag
Schleifenproblem bei Registrierung per Benutzer und Email	pullox	PHP Tipps 2006	4	15.03.2006 16:52
PLZ überprüfen und Email versenden		PHP Tipps 2007	8	25.12.2005 10:50
Email mittels PHP versenden		PHP Tipps 2005-2	6	16.09.2005 14:35
problem beim versenden einer pdf als email anhang	Sclot	PHP Tipps 2005-2	6	06.09.2005 11:56
Bild per email über Formular versenden		PHP Tipps 2005-2	8	30.07.2005 14:46
email versenden mit funktion mail()		PHP Tipps 2005	6	20.05.2005 17:38
Fehlermeldungen abfangen und als email versenden?	Calli	PHP Tipps 2005	2	28.02.2005 10:23
[Erledigt] email versenden und empfangen		PHP Tipps 2005	2	21.02.2005 17:44
Link in Email versenden. Sonderzeichen??	bicpi	PHP-Fortgeschrittene	18	03.02.2005 12:20
Zeitversetzte Email versenden	pcschröda	PHP-Fortgeschrittene	9	29.01.2005 05:47
mit php email versenden		PHP Tipps 2005	6	06.01.2005 16:52
[Erledigt] Datenbank per email versenden		PHP Tipps 2004-2	1	23.11.2004 15:46
email mit attachement versenden		PHP Tipps 2004	3	11.11.2004 10:56
Ganz einfach eine Email über Kontaktformular versenden. Wie?	'progman'	PHP Tipps 2004-2	1	07.11.2004 17:11
Versenden der Website per Email		PHP Tipps 2004	2	30.06.2004 14:39

05.04.2005, 14:45
Mano Erfahrener Benutzer Registriert seit: 06.08.2003 Beiträge: 453 PHP-Kenntnisse: Fortgeschritten	Benutzer ändern und Email versenden Folgendes Script nutze ich zum Ändern meiner User in der Datenbank. Jedoch ist dieses nicht sehr schön aufgebaut. Die gleiche Abfrage besteht nochmal für !empty($Passwort), dabei wird zusätzlich das Passwort mit in der Datenbank gespeichert. Nun möchte ich das, wenn der Benutzername oder Passwort oder User Email geändert er eine Mail versendet. Wie wäre dabei eine vernüftige Abfrage. Genauso gefällt mir nicht das ich zwei if Abfragen habe, wenn das Passwort leer oder nicht leer ist. Hat da jemand eine bessere Idee? :wink: ?>[/php]


Mister Ad PHP Code Flüsterer Registriert seit: 21.08.2005 Beiträge: 4682 PHP-Kenntnisse: Fortgeschritten

05.04.2005, 15:07
kid01 Erfahrener Benutzer Registriert seit: 21.05.2008 Beiträge: 643	naja so ist es zumindes übersichtlich warum willst du diese schöne strucktur zerstören???? Wo ist bei den Mails dein Problem????? Am ende jeder if kannst du doch ohne Probleme ne mail funktion einbauen __________________ http://www.dhsmedia.de http://www.buecherfuerdasleben.at

05.04.2005, 15:13
Mano Erfahrener Benutzer Registriert seit: 06.08.2003 Beiträge: 453 PHP-Kenntnisse: Fortgeschritten	Weil ich das nicht "so" machen will, da der Inhalt der Mail immer der gleiche ist?! PHP-Code: `<?php if($blakp1 != $benutzername) { mail(.....); } if($blakp2 != $email) { mail(.....); } // usw.... ?>`

05.04.2005, 15:18
Kort Erfahrener Benutzer Registriert seit: 09.09.2004 Beiträge: 716 PHP-Kenntnisse: Anfänger	Du kannst ja den Inhalt vorher in einer Variablen ablegen. __________________ "Only wimps use tape backup: real men just upload their important stuff on ftp, and let the rest of the world mirror it." - Linus Torvalds, 1996

05.04.2005, 15:25
Mano Erfahrener Benutzer Registriert seit: 06.08.2003 Beiträge: 453 PHP-Kenntnisse: Fortgeschritten	Ja, das mit dem "oder" klappt jedoch nicht in allen fällen. Mir wird schon was einfallen. Jedoch ist mir gerade was ganz anderes eingefallen. Wenn ich den Benutzernamen ändere muss ich ja überprüfen, ob dieser schon besteht. Da die BenutzerID schon übergeben wird kann ich daher den alten Benutzernamen auslesen und in einer Select Abfrage fragen wo dieser alte Benutzername nicht vorhanden ist. Oder würdet ihr dieses anders lösen?

05.04.2005, 16:28
Mano Erfahrener Benutzer Registriert seit: 06.08.2003 Beiträge: 453 PHP-Kenntnisse: Fortgeschritten	Wäre das so korrekt mit der Abfrage? (Logik)

05.04.2005, 16:52
robo47 Moderator Registriert seit: 03.09.2004 Beiträge: 11.792 PHP-Kenntnisse: Fortgeschritten	select * from Warum soll ich nicht SELECT * schreiben? http://www.php-faq.de/q/q-sql-select.html mysql error man sollte in seinen Scripten immer mysql_error() verwenden, in etwa so: PHP-Code: `$sql = "SELECT feld1,feld2 FROM table WHERE id=5"; $result = mysql_query($sql) or die(mysql_error().' Query: '.$sql);` __________________ robo47.net - Blog, Codeschnipsel und mehr \| Caching-Klassen und Opcode Caches in php \| Robo47 Components - PHP Library extending Zend Framework

05.04.2005, 17:55
Mano Erfahrener Benutzer Registriert seit: 06.08.2003 Beiträge: 453 PHP-Kenntnisse: Fortgeschritten	PHP-Code: `<?php select * from ?>` Das wird schon mit bedacht verwendet, den das Script umfasst nicht nur die paar Zielen da oben lol. Außerdem ist das einfacher als wenn ich 10 Spalten einzeln davorschreibe auf nen paar mehr oder weniger kommts da auch nicht an lol. Ja stimmt das error fehlte noch in dem Beispiel. Es ging ja hierbei auch nicht darum sondern um die Logik