Shoutbox adminfunktionen erstellen

derdefeckter

soweit ich gelernt habe
muss bei else keine klammer hin

mfg

no_code

falsch gelernt! es muß nur keine klammer hin wenn danach nur ein statement folgt bei dir sind es aber mehr z.14 soll doch auch noch mindestens mit rein...

derdefeckter

allerdings hatt das voher auch geklappt ohne die klammern

also auf meinen anderen seiten klappt das genau so

nur jetzt hier und ich denke das er die js datei nicht ricjtih mit rein nimmt

mfg

no_code

denk doch was du willst...

PRIEST

sicher? Kann ich grad nicht verstehen. Was man aber machen kann ist..

if ($foo == $bar):
 echo 'lustig lustig';
else:
 echo 'trotzdem lustig';
endif;

no_code

ja also es ist so ne sache es ist nicht direkt falsch aber nur solange das exit() da ist sobald einer auf die idee kommt das raus zu nehmen hast den fehler. also kann das für mich zu folge fehlern führen und sollte ausgebessert werden.

Hallo,

Zeile 10:
Bereits angesprochen. Entweder du setzt die Klammern, dann kannst du dir exit sparen, oder du lässt exit stehen und entfernst das else. Deine Variante verwirrt nur den Leser nur weil das else unsinnig ist.

Zeile 12:
Kein Fehler, aber schöner wäre die Verwendung von require_once. Bei require_once wird geprüft ob die Datei bereits eingebunden ist. Wenn ja, wird sie kein zweites mal dazugeholt.

Zeile 18/33:
Was ist wenn die Ergebnismengen leer sind? Erweitere den Code besser um

if (!$row) exit;

Zeile 50:
<form action="" ...> verweist auf die im Verzeichnis liegende index-Datei und nicht zwangsläufig auf sich selbst. Soll das passieren, lass action="" weg, oder nutz $_SERVER['script_name'].

Zeile 117/118:
Du machst den PHP-Tag zu um ihn direkt danach wieder zu öffnen. Wieso?

2-3 kleine Anmerkungen und Fragen:

1. Wieso wechselst du zwischen mysql_fetch_array und mysql_fetch_assoc?
Wenn du sowieso nur auf die assoziativen Indizes zurückgreifst verwendet doch gleich mysql_fetch_assoc oder mysql_fetch_array($result, MYSQL_ASSOC).

2. Nutzt du persitente Verbindungen? Wenn nicht, vermisse ich mysql_free_result um den Speicher wieder für neue Daten frei zu machen. Ein fehlendes mysql_free_result verursacht allerdings auch nur ein Warning im Log (sofern Warnings aktiviert sind)

3. Versuche nicht immer die gleichen Variablennamen zu verwenden. Bei der miesen Codeeinrückung erkennt man am Ende nicht mehr, welche du wann verwendest. Noch ist es überschaubar, aber spätestens bei deinem ersten 1000-Zeilen Script wird's kompliziert. Vor allem wächst dann auch die Gefahr, bereits bestehende Ergebnismengen zu überschreiben.

4. Versuch dich in Zukunft mal ein wenig mit MVC (Model-View-Controlling) auseinander zusetzen. Das Mischen von View und Controlling find ich persönlich grauenvoll.

LG

derdefeckter

danke!,

für die hilfe bis jetzt.

allerdings läd er mir die shoutbox.js nicht! voran liegt das?

mfg

p.s. ich werden denn code nochmal komplett überarbeiten das else ist weg

Hallo,

möglicherweise liegt die shoutbox.js nicht im selben Verzeichnis.

Btw: Mach deinen Server sicher. Ich sag dir auch wieso:

Schleifunk-Ostseewelle

Auch auf diesem Server gibt es, bei den Server-Regeln, merkwürdige Einschränkungen, und basieren nicht auf den FRN-Regeln![...]

Es klingt nicht so als wäre es geheim, aber genauso schnell wie daran, kommt man auch an andere Dinge.

LG

derdefeckter

Hallo,

Doch die shoutbox.js liegt im selben Verzeichnis ich kann sie sogar abrufen wenn ich direkt denn link oben eingebe

dann noch ne frage:

hmmz schrieb:

Hallo,

möglicherweise liegt die shoutbox.js nicht im selben Verzeichnis.

Btw: Mach deinen Server sicher. Ich sag dir auch wieso:

Schleifunk-Ostseewelle

Auch auf diesem Server gibt es, bei den Server-Regeln, merkwürdige Einschränkungen, und basieren nicht auf den FRN-Regeln![...]

Es klingt nicht so als wäre es geheim, aber genauso schnell wie daran, kommt man auch an andere Dinge.

LG

Und zwar wo wurde die Datei bzw die Sätze gefunden?
und wie? mir kommt die Datei bzw die Sätze nicht bekannt vor

Und wie kann ich denn Server sicherer machen

mfg

Hallo,

Und zwar wo wurde die Datei bzw die Sätze gefunden?

da!

und wie?

so!

Einfach mal eine leere index.php in den Stamm legen, oder die "startseite.php" die ja augenscheinlich vorhanden ist, als index eintragen (sofern du an die Config kommst).

Welchen Link gibst du wo ein um an die Script-Datei zu kommen? Laut deinem Code, müsste sie im selben Verzeichnis liegen, wie die Datei, die den gezeigten Code beinhaltet. Vorausgesetzt, die Datei mit dem gezeigten Code wird nicht irgendwo via include( oder require( eingebunden. Dann wäre der Pfad relativ zur einbindenden Datei.

LG

derdefeckter

Danke für den Hinweis!

ich werde das nacher ändern werde jetzt erstmal schlafen gehen

mfg

PRIEST

derdefeckter schrieb:

Danke für den Hinweis!

ich werde das nacher ändern werde jetzt erstmal schlafen gehen

mfg

guten Tag.

ach, btw deine Signatur ist nicht richtig ^^

PRIEST schrieb:

derdefeckter schrieb:

Danke für den Hinweis!

ich werde das nacher ändern werde jetzt erstmal schlafen gehen

mfg

guten Tag.

ach, btw deine Signatur ist nicht richtig ^^

dumdidum

derdefeckter

PRIEST schrieb:

derdefeckter schrieb:

Danke für den Hinweis!

ich werde das nacher ändern werde jetzt erstmal schlafen gehen

mfg

guten Tag.

ach, btw deine Signatur ist nicht richtig ^^

Was ist da denn Falsch?

also die javascript datei wird in html mit einbezogen mit den css datein geht es ja auch
mfg

EDIT:
Ohh ja es muss heissen "=='True'" und nicht "= True"

Hallo,

was du noch versuchen kannst ist ein vollständiger Tag

<script type=\"text/javascript\" language=\"javascript\" src=\"shoutbox.js\" ></script>

Wenn das auch nicht klappt, weiß ich nicht mehr weiter. Die Zeile wird im generierten Quellcode aber auch angezeigt? Und die Script-Datei liegt wirklich im selben Verzeichnis wie das aufrufewnde Script? Hast du eventuell einen Fehler in der shoutbox.js? Woher weißt du eigentlich dass sie nicht eingebunden wird?

LG

inflames2k

If (Beitrag.Hilfreich == 'True')

Also der Vergleich ist unsinnvoll.

Besser:

If (Beitrag.Hilfreich == true)

noch besser:

If (Beitrag.Hilfreich)

PRIEST

und du gibst Bewertung.Send("Positiv") als text aus. Ist das so gewollt :)?
Sag ja ^^

PRIEST schrieb:

und du gibst Bewertung.Send("Positiv") als text aus. Ist das so gewollt :)?
Sag ja ^^

Falls er "Nein" sagt -> Es gibt in PHP keinen Punktoperator der die Aufgabe "Zuweisungsoperator" hat. Der Punktoperator ist in PHP ein Verkettungsoperator. Anstelle Klasse.Methode muss man hier also Klasse->Methode verwenden :p

Noch einmal etwas zu Operatoren, wobei wahrscheinlich das gesamte Tutorial hilfreich ist

LG

Naja, hier fehlt ohnehin eine gehörige Portion Grundwissen.
Meint ihr nicht, ihr verwirrt ihn eher noch mehr?