[SQL] pobierz, plaintext 
<?php
$connect = mysql_connect("x.x.x.100", "root", "123456");
?>
<form action="?action=change" method="post">
Nick: <input type="text" name="name"><br>
Czas: <input type="text" name="availDt"><br>
<input type="submit" value="Banuj">
</form>
<?php
	/********** KONFIGURACJA KOMUNIKATÓW *************/
	$msg['success'] = "Zbanowany";
	$msg['not_exist'] = "Niema Takiego gracza";
	$msg['not_some'] = "Podane hasła do siebie nie pasują!";
	/*************************************************/
	$nick = $_REQUEST['name'];
	$czas = $_REQUEST['availDt'];
	$db = mysql_select_db("account" "player");
	IF($_REQUEST['action']=='ban')
	{
		{
			$szukaj = mysql_num_rows(mysql_query("SELECT accountid FROM player WHERE name='$nick')) > 0)
 
			$ban1 = mysql_num_rows(mysql_query("SELECT * FROM account WHERE id='$szukaj')) > 0)
 
				$update = mysql_query("UPDATE account SET status=('BLOCK') AND availDt=('$czas') WHERE $szukaj='$ban1'") OR die(mysql_error());
				IF($update==true)
				{
					echo $msg['success'];
				}
 
			else
			{
				echo "<b>Błąd</b><br>".$msg['not_exist'];
			}
		}
	}
?>
[SQL] pobierz, plaintext 

Napisałem coś takiego i czy jest dobrze napisany
ma robic cos takiego

1. wyszukuje danych w bazie "player" o danym "name" i kopiuje accountid z tej tabeli
2. odczytuje wynik z pk 1 i wyszukuje ten winyik z pkt, 1 w rekordzie "id"
3. jak znajdzie to zmieni dane tabeli na te co podałem "status=('BLOCK') AND availDt=('$czas')"

prosze o sugestie co zle a co dobrze

PHP przemieszane z HTML - jak ktoś lubi oldschool to może być.

Błędy składniowe: brakuje zamkniętych cudzysłowów i średników na końcu polecenia.

Zero zabezpieczenia przed SQL injection.

Składnia UPDATE jest trochę inna:

tak jak masz teraz, ustawiasz pole `status` na wynik koniunkcji ('BLOCK') AND availDt=('$czas')

Tu jest źle IF($update==true)
jak już to po wykonaniu UPDATE należy wywołać funkcję int mysql_affected_rows ([ resource $link_identifier ] )
i w zależności od tego czy zwróci 0, 1 sterować dalszym kodem..
----
Tak poza tym ten kod jest tak naładowany psarser eror'ami, że szok!

I na dokładkę: "Nie ma" to dwa słowa.

A sensu tego: w ogóle nie rozumiem.

[SQL] pobierz, plaintext 
<?php
$connect = mysql_connect("x.x.x.100", "root", "123456");
?>
<form action="?action=change" method="post">
Nick: <input type="text" name="name"><br>
Czas: <input type="text" name="availDt"><br>
<input type="submit" value="Banuj">
</form>
<?php
	/********** KONFIGURACJA KOMUNIKATÓW *************/
	$msg['success'] = "Zbanowany";
	$msg['not_exist'] = "Niema Takiego gracza";
	$msg['not_some'] = "Podane hasła do siebie nie pasują!";
	/*************************************************/
	$nick = $_REQUEST['name'];
	$czas = $_REQUEST['availDt'];
	$db = mysql_select_db("account" "player");
	IF($_REQUEST['action']=='ban')
	{
		{
			$szukaj = mysql_num_rows(mysql_query("SELECT accountid FROM player WHERE name='$nick')) > 0)
 
			$ban1 = mysql_num_rows(mysql_query("SELECT * FROM account WHERE id='$szukaj')) > 0)
 
				$update = mysql_query("UPDATE account SET status = 'BLOCK', availDt = '$czas' WHERE $szukaj=$ban1") OR die(mysql_error());
 
	}
?>
[SQL] pobierz, plaintext 

Teraz? poprawione Jak coś jest źle proszę powiedzcie

Na 6 czy 7 wuwag, poprawiłeś jedną rzecz. Jak dla mnie super.

NO weź proszę dopiero zaczynam z tym nie wiem wszystkiego ;/

Masz włączone raportowanie błędów?

Jeśli nie wstaw sobie:



na początku. Będą Ci leciały komunikaty błędów z numerem linii gdzie masz błąd. Po kolei naprawiaj, aż będziesz miał czysto.

Będziesz wtedy na etapie skryptu który przynajmniej nie rozsypuje się w połowie. Potem zajmiesz się poprawianiem logiki.

[SQL] pobierz, plaintext 
$ban = mysql_num_rows(mysql_query("SELECT * FROM account WHERE id='$szukaj')) > 0)
[SQL] pobierz, plaintext 

CO w tym jest źle?

[PHP] pobierz, plaintext 
$ban = mysql_num_rows(mysql_query("SELECT * FROM account WHERE id='$szukaj'")) > 0);
[PHP] pobierz, plaintext 

I tak w zasadzie powinieneś się z tym udać do działu PHP, bo z MySQL to niewiele ma wspólnego.

cos jeszcze zle jest