Witam mam problem nie ważne czy podam dobrze czy źle kod z obrazka to i tak jest kiszka i pisze że kod z obrazka jest zły ;[

[PHP] pobierz, plaintext 
<?php
 
if($_POST['creat']== 1 )
{
   function passcheck($pass) {
	   $nr = floor(time() / 3600);
	   for($i = 0; $i < strlen($pass); $i++)
		 if($i % 2 == 0) $a .= $pass[$i]; else $b .= $pass[$i];
	   $a = base_convert(strrev($a), 36, 10);
	   $b = base_convert($b, 36, 10);
	   if($a + $b >= $nr && $a + $b >= $nr - 2) return true;
	   return false;
   }
 
   if(!passcheck($pass)) die("Kod z obrazka jest nie poprawny !");
 
 
   }
 
 
?>
[PHP] pobierz, plaintext 

Nie zgubiłeś czasem w tym skrypcie klamer {} w pętli for
W if rozumiem że jeżeli mamy jedną linijkę za if to klamry nie są potrzebne ale for bez klamer Coś mi tu nie pasuje ale może się mylę....

to nie to ... ; ]

To czy nie to, w pętli i tak masz dwie instrukcje (if oraz else) więc klamry jak najbardziej są potrzebne!

Jak wyliczasz kod z obrazka?
Prześlij go przez sesję zamiast wyliczać przy sprawdzaniu od nowa.

Nie widać funkcji, która umieszcza kod na obrazku - ja kryptografem nie jestem i po funkcji dekodującej nie potrafię zgadnąć jaka jest funkcja kodująca.

to jest cały skrypt

http://4programmers.net/PHP/Zabezpieczenia...82em_na_obrazku.
link poprawiony

Jak chcesz captcha to looknij tutaj:

http://www.ninfo.pl/download/6/

link już nie działa to co ja napisałem to już gotowy skrypt tam jest tylko parę błędów jest

Link który Ci przesłałem mi działa

http://www.ninfo.pl/artykuly/2008/04/26/captcha-class/ - masz całość opisaną


" link już nie działa to co ja napisałem to już gotowy skrypt tam jest tylko parę błędów jest "

Za dużo kombinowania, ze sprawdzaniem kodu z obrazka... zamiast zrobić to na sesjach ;]

czyżby register_globals off? użyj zamiast $pass - $_GET['pass'] czyli

[PHP] pobierz, plaintext 
<?php
if(!passcheck($_GET['pass'])) die("Kod z obrazka jest nie poprawny !");
?>
[PHP] pobierz, plaintext