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

a co to ma robić?

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

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 

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

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

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

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...

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

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ę