php kod insert inte till sql

Har inte sysslat med PHP på flera år men tror jag ser felet, du tycks aldrig köra din query. Bör se ut så här om jag inte är helt förvirrad.

<?php 
require('connect.php');
$username= @$_POST['username'];
$password= @$_POST['password'];
$repass = @$_POST['repassword'];
$email= @$_POST['email'];

if(isset($_POST['submit']))
{
	if($username && $password && $repass && $email){
	$query = true;     //Utan denna rad är query aldrig sant och kommer därför aldrig lägga till användaren. 
	
	}else
		{
			echo "Fyll i alla uppgifter";
		}
		
}
	

	
	
	{
		if($query = mysql_query
		("INSERT INTO users ('id','username','password','email') VALUES ('', '".$username."','".$password."','".$email."') "))
		echo "Det funkar";
		
		else 
	{
		echo "Det funkar inte, något är galet";
	}
	}

?>

Om du är nybörjare och endast ska använda servern lokalt fungerar detta bra men om servern är åtkomlig från internet bör ett antal förändringar ske för att göra det någorlunda säkert, nedan är tips på bra sökord om du vill lära dig mer om hur du bör säkra "user registration" sidan.

• password hash - md5 var normalt när jag började men är nu alldeles för lätt att knäcka och bör köras i flera varv eller med starkare algoritm

• Password salt - Varje användare bör få ett unikt salt som används i olika steg tillsammans med "hash" för att undvika "rainbow table" attacker

• SQL injection - När jag var igång senast var det mysql_real_escape_string() som gällde vilket stoppar skadlig kod från att injekteras i dina text rutor

Slutligen bör du köra en kontroll om $password och $repassword är lika då du annars kunnat skippa $repassword helt (Förutsatt att detta inte görs i javascript redan).

SQL injection är något av det besvärligaste som finns för nybörjare och tyvärr finns det även second level SQL injection, vilket egentligen betyder att ingen kod körs första varvet men sparas i data-basen och körs sedan när sidan genereras. Här är en mycket bra video om du är intresserad.

Hoppas jag inte förvirrade dig allt för mycket men om du släpper lös server på internet och framförallt om du får fler användare än dig själv måste ovanstående vara gjort för allas säkerhet då de flesta återanvänder inloggnings uppgifter.

Som tidigare nämnt gå över till PDO för då kan du köra enklare kod som ser ut såhär:

$dbname = "";
$dbhost = "";
$dbuser = "";
$dbpass = "";

$dbh = new PDO("mysql:host=$dbhost;dbname=$dbname;charset=utf8", $dbuser, $dbpass);
// Sätt eventuella inställningar för PDO:
$dbh->setAttribute(PDO::ATTR_ERRMODE, PDO::ERRMODE_EXCEPTION);
$dbh->setAttribute(PDO::ATTR_EMULATE_PREPARES, false);

function insertUserInDB($dbh) {
	$insert_user_sql = "INSERT INTO users (username, password) VALUES (:username, :password)";
	$stmt = $dbh->prepare($insert_user_sql);
	
	// Hasha och lowercase
	$password = sha1($_POST['password']);
	$username = strtolower($_POST['username']);


	$stmt->bindParam(':username', $username);
	$stmt->bindParam(':password', $password);
	$stmt->execute();
}
        // Kör funktionen och nu ska det fungera
	insertUserInDB($dbh);

("INSERT INTO users ('id','username','password','email') VALUES ('', '".$username."','".$password."','".$email."') "))

Felet är gissningsvis att fältet "id" är auto-increment och att du försöker skicka en tom sträng.

("INSERT INTO users ('username','password','email') VALUES ('".$username."','".$password."','".$email."') "))

Men som tidigare sagt så rekommenderas PDO eller mysqli.