Reikia nuomonės apie pakvietimų sistema

[*DeMo*]

Sukūriau paprasta pakvietimų sistema su Orinto pagalba

Daryta su MySQLi

 

Laukiu nuomonių kritikos ir patarimų ką patobulint

DEMO : http://www.tesst.wc.lt/

 

Github : https://github.com/Demonas/Pakvietimas

Redaguota Balandžio 2, 2015, nario *DeMo*

[Estafetingo^^]

Jai įmanoma, tai pakeisk šriftą į tahoma.

[#PovilasC]

Dinaminius IP turintiems GG. Geriau padaryk prisijungimą / registraciją.

[Bladvorkas]

Man asmeniškai gražu, viskas OK

[*DeMo*]

Jai įmanoma, tai pakeisk šriftą į tahoma.

Pamėginsiu Sėkmingai pakeista į tahoma gal ir gražiau dabar atrodo

 

Dinaminius IP turintiems GG. Geriau padaryk prisijungimą / registraciją.

 

Šita žinau bus ateityje login/register

Man asmeniškai gražu, viskas OK

Ačiū

Redaguota Balandžio 1, 2015, nario *DeMo*

[nesigilink]

šaunuolis, dizainas neblogas, šeip galėtum kažką sugalvoti su backgroundu, kažkiek pagyvintų, ir mažokas blokelis pagrindinis, galėtum praplėsti (aišku ir tadą reikėtų pridėti navigaciją ir ten dar kažką, galima daug ką padaryti jeigu tik bus noro ) sekmės!!

[*DeMo*]

šaunuolis, dizainas neblogas, šeip galėtum kažką sugalvoti su backgroundu, kažkiek pagyvintų, ir mažokas blokelis pagrindinis, galėtum praplėsti (aišku ir tadą reikėtų pridėti navigaciją ir ten dar kažką, galima daug ką padaryti jeigu tik bus noro ) sekmės!!

Dėl panelės tai užtenka ir tokios nes nėra per daug ką ir dėti   o dėl navigacijos tai nėra ką joje dėti nebent į tinklapį atgal o daugiau kaip ir nieko sistemoje nėra nei Top15 nieko

[nesigilink]

Dėl panelės tai užtenka ir tokios nes nėra per daug ką ir dėti   o dėl navigacijos tai nėra ką joje dėti nebent į tinklapį atgal o daugiau kaip ir nieko sistemoje nėra nei Top15 nieko

štai kodėl negali praplėsti, pats atsakei... Tai, sukurk kažką ;D paįvairink sistemą, patobulink ją

[leoleva]

Pasidalink kodu

[nesigilink]

Pasidalink kodu

jo išties galėtų įkelti į kokį github, tadą galėtume dar daugiau patarimų duoti

[*DeMo*]

Pasidalink kodu

 

 

jo išties galėtų įkelti į kokį github, tadą galėtume dar daugiau patarimų duoti

Negražus tas kodas bet jei taip norit https://github.com/Demonas/Pakvietimas

[darrel.]

Tai kad neveikia tas psl.

[Vaidelio]

Tai kad neveikia tas psl.

Gal ištrine..

[*DeMo*]

 

Tai kad neveikia tas psl.

 

Pataisiau buvau suklydes ir įkėles kitos sistemos index.php

[leoleva]

Kalbant apie php pusę, daug kas iš `rimtų programuotojų` mane muš už žinių stoką, tačiau pasakysiu ką aš matau blogai :
 

Bereikalingai naudojama tokia sudėtinga captcha, galėjai naudot paprastesnę.
Geriau naudoti require, nei include.
Jeigu naudoji mysqli, vis tiek reiktų naudot prepare statement, nes :

if(isset($_GET['pakviete']) && !empty($_GET['pakviete'])) 
{
   $pakviete = $_GET['pakviete']; 

   // ...

   $pav = "SELECT * FROM `ip` WHERE `id`='$pakviete'";
   $db = $conn->query($pav);

}

Toks variantas nėra labai geras. Reiktų patikrinti bent su is_int

Query rašymas :

$conn->query("UPDATE  ip  SET pakviete = pakviete +1 WHERE  id ='$pakviete'");
$conn->query("UPDATE  ip  SET pakvietimas ='$data' WHERE  id ='$pakviete'");

// -->

$conn->query("UPDATE `ip` SET `pakviete` = pakviete +1, `pakvietimas` = "'.$data.'" WHERE  `id` = "'.$pakviete);

Strcasecmp naudojimas :

Jeigu žmogus išsiųs array, o tu dėsi $_POST į pačią funkciją gausi klaidą, kad `Warning: strcasecmp() expects parameter 1 to be string, array given` ar kaip ten.

Nuorodos sudarymas :

$page = $_SERVER['SERVER_NAME']; 
$file = $_SERVER['PHP_SELF']; 

// ...

echo "www.$page$file?pakviete=".$id."";

Jeigu į puslapį bus įeita per www, tai SERVER_NAME gražins www. pakvietimusistema.lt, o vartotojui atvaizduos ->
"www.www. pakvietimusistema.lt/failas.php?pakviete=1"
Ir šiaip toks variantas keistas, patarčiau iš praktikos naudot ką nors tokio (aišku pasitaisyk pagal save)
 

function url()
{
	$s = substr(strtolower($_SERVER['SERVER_PROTOCOL']), 0, strpos($_SERVER['SERVER_PROTOCOL'], '/'));

	if (!empty($_SERVER["HTTPS"])) 
		$s .= ($_SERVER["HTTPS"] == "on") ? "s" : "";
	

	$s .= '://'.$_SERVER['HTTP_HOST'];

	if (!empty($_SERVER['SERVER_PORT']) && $_SERVER['SERVER_PORT'] != '80') 
		$s .= ':'.$_SERVER['SERVER_PORT'];
	
	return $s;
}
// PaySera

Pati kodo tvarka. Daug kur su echo atvaizduoji html, o kituose vietose išskiri html tagus (pvz meta).
------------------

Klydau, klystu ir klysiu. Jeigu kažkur nusišnekėjau, pataisykit.

 

 

Redaguota Balandžio 3, 2015, nario leoleva