Rzuci ktoś okiem na skrypt?:) Opcje

[vala]

Witam prosze o ocene:


http://royy.net/~valarius/shout.txt

a tu mozna sprawdzic jak dziala

http://royy.net/~valarius/shout.php

z gory dzieki za wszelkie + i -

Pozdrawiam

[nospor]

a co to ma robić?

Jak wciskam ok to mi wyskakuje bląd js (opera):

http://royy.net/~valarius/shout.php
Event thread: onsubmit
Error:
name: ReferenceError
message: Statement on line 1: Reference to undefined variable: No such variable 'validate_shout'
Backtrace:
  In unknown script
    return validate_shout();
  At unknown location
    {event handler trampoline}

[vala]

akurat tej funkcji w shout.php ni ma;)

[PHP] pobierz, plaintext 
<?php
 
function validate_shout()
 {
	var lnick = document.getElementById('snick').value;
	var lmess = document.getElementById('smessage').value;
 
 
	if(lnick.length<1)
	{
 
	 alert('moze podasz jakis nick?:)');
	 return false;
	}
	if(lmess.length<1)
	{
	  alert('a moze napisz wiadomosc?:)');
	  return false;
	}
 
	return true;
 }
 
?>
[PHP] pobierz, plaintext 

[kowal1122]

no to wyglada na zwyczajny skrypt ala księga gosci...
co tu oceniac, grafike?

[nospor]

ja ci mówie ze w js skrypt mi wywala bląd. do php nawet nie dochodzi

[vala]

no to wyglada na zwyczajny skrypt ala księga gosci...
co tu oceniac, grafike?

bardziej mi chodzilo o rzucenie okiem na kod
czy jest czytelny / czy moje rozwiazania sa do tego typu skryptu prawidlowe itp

[trucksweb]

dla mnie to nie jest w 100% Twoj kod, skrypt czemu daty wywala po angielsku??
to jest mniej wiecej ksiega gosci , najprostsza jaka moze byc, choc takze na komentarze sie nada...

[vala]

dla mnie to jest moj w 100% kod/skrypt bo gdybym go zassal skads tam to nie umieszczalbym go TU
nie sadzisz?

ale dobra olac widze ze nie bylo sensu panow modkow prosze o kasacje tematu
EOT

[Chewolf]

Hmm..., to nie nadaje sie na Shoutbox , nawet na "Shout". Nie to żebym był taki że wszystko co zobacze mi się nie podoba, ale jak mówi Kuba Wojewódzki nie ruszyło mnie to :], wygląda jak z plastusiowego pamiętnika Nie jest źle choć mogło by być lepiej zarówno pod względem kodu jak i wyglądu :-). Mogę cie pocieszyć że np. miłym akcentem jest usuwanie bluzgów, choć trochę ich mało, w moim jest koło 80 a to i tak mało.
Najlepiej jest ukazywać kod w plikach .phps a nie txt.
Do kolegów ... jeśli nie macie czego oceniać - to nie oceniajcie ! nie piszczie bezsensownego posta.
No i najwazniejsze - nie moge dodac zadnej wypowiedzi ?!
Dziękuję

Ten post edytował Chewolf 10.06.2005, 18:04:59