Forum PHP.pl

Dobra bo tam napisałem tyle co mi się w oczy rzuciło ale to bzdura była.

[PHP] pobierz, plaintext 
<?php
		$znaki = '1234567890qwertyuiopasdfghjkklzxcvbnm';
		$url = '';
 
		for ($i=0; $i<7; $i++)
		{
			$url .= $znaki[rand()%(strlen($znaki))];
		}
 
		echo 'http://www.banan.pl/url/' . $url;
		$file = $url . ".php";
		$sk = '<?php echo "<meta http-equiv=\"refresh\" content=\"1;url=' . $_POST['link'] . '\" />" ?>';
		#<meta http-equiv="refresh" content="1;url='$_POST['link'];" />
		$fp = fopen($file, "a");
		flock($fp, 2);
		fwrite($fp, $sk);
		flock($fp, 3);
		fclose($fp);
?>
[PHP] pobierz, plaintext 

$id = '', zła nazwa zmiennej, dalej używasz $url a z operatorem .= musi być ona wcześniej już "zainicjowana".
Dalej łączenie wartości zmiennych i stringów - w kilku miejscach błąd, poprawiłem.

[HTML] pobierz, plaintext 
 <form action="link.php" method="POST">
	<input type="text" name="link" />
	<input type="submit" name="ok" value="Ślij link!" />
</form>
[HTML] pobierz, plaintext 

W formularzu musisz określić metodę wysyłania na post, domyślnie robi to przez get więc nie znajdziesz przesłanej wartości w $_POST.

Zakładając że masz taki input:

[HTML] pobierz, plaintext 
<input type="hidden" name="title" value="" />
[HTML] pobierz, plaintext 

dodaj za nim:

[HTML] pobierz, plaintext 
<script type="text/javascript">document.getElementsByName('title')[0].value = document.title;</script>
[HTML] pobierz, plaintext 

spowoduje to nadanie wartości atrybutowi value Twojego inputa o nazwie "title" na taką jaki tytuł posiada strona.

To już sam najlepiej wiesz jakie masz tabele oraz co i jak potrzebujesz z nich pobierać. Miłej zabawy.

Żeby było jeszcze ładniej to zrób jeszcze inaczej, bo ogólnie to komplikujesz sobie pracę, pobierasz dane z tabeli userID na podstawie nickname po to aby na podstawie userID pobrać kolejne dane z tej samej tabeli. Używając klasy możesz skrócić sobie pracę.

[PHP] pobierz, plaintext 
<?php #class_User.php
 
// klasa User
 
class User {
 
	private $userID;
	private $username;
	private $email
 
	public function __construct($nickname = "") {
		// konstruktor
 
		if ($nickname != "") {
			$ds = mysql_fetch_array(safe_query("SELECT userID, username, email FROM ".PREFIX."user WHERE nickname='".$nickname."'"));
			$this->userID = $sd['userID'];
			$this->username = $sd['username'];
			$this->email = getinput($sd['email']);
		}
	}
 
	public function __get($arg) {
		// funkcja zwracająca wartości zmiennych
 
		return $this->$arg;
	}
}
 
?>
[PHP] pobierz, plaintext 

i pliczek

[PHP] pobierz, plaintext 
<?php
 
// dołączenie plików z definicjami klas
require_once('../class_User.php');
 
$nickname = "admin" // tak czy inaczej pobrana wartość z formularza przypisana zmiennej nickname
 
// utworzenie obiektu klasy User
$user = new User($nickname);
 
$id = $user->__get("userID");
$username = $user->__get("username");
$email = $user->__get("email");
 
?>
[PHP] pobierz, plaintext 

Tworząc obiekt podajesz mu od razu nickname który pobrałeś czy to z formularza czy sam przypisałeś zmiennej. Konstruktor go przejmuje i z jego pomocą pobiera z bazy danych potrzebne pola od razu gdy tylko obiekt jest utworzony, nie trzeba wywoływać żadnych dodatkowych funkcji. Później tylko z pomocą funkcji __get() odwołujesz się do kolejnych zmiennych prywatnych w klasie.

[PHP] pobierz, plaintext 
$_user->username['admin'];
[PHP] pobierz, plaintext 

Zmień na:

[PHP] pobierz, plaintext 
$_user->username = "admin";
[PHP] pobierz, plaintext 

Zapomniałeś o " ()" po getID? To funkcja a nie zmienna.

Można to zrobić na kilka sposobów w zależności od tego do czego jeszcze będziesz używał tej klasy i jak.

Tak żeby robiła tylko to co wkleiłeś wyżej:

[PHP] pobierz, plaintext 
<?php #class_User.php
 
// klasa User
 
class User {
 
	public function __construct() {
 
	}
 
	public function getUserID($nickname) {
		$ds = mysql_fetch_array(safe_query("SELECT userID FROM ".PREFIX."user WHERE nickname='".$nickname."'"));
		return $ds['userID'];
	}
 
	public function getUsername($userID) {
		$ds = mysql_fetch_array(safe_query("SELECT username FROM ".PREFIX."user WHERE userID='".$userID."'"));
		return $ds['username'];
	}
 
	public function getUserEmail($userID) {
		$ds = mysql_fetch_array(safe_query("SELECT email FROM ".PREFIX."user WHERE userID='".$userID."'"));
		return getinput($ds['email']);
	}
}
 
?>
[PHP] pobierz, plaintext 

i w pliku

[PHP] pobierz, plaintext 
<?php
 
// dołączenie pliku z definicjami klas
require_once('../class_User.php');
 
// utworzenie obiektu klasy User
$user = new User();
 
$id = $user->getUserID['jakis_login'];
 
?>
[PHP] pobierz, plaintext 

Koniecznie musisz użyć case? Nie wiem czy możliwe jest użycie w tym miejscu pętli for, nigdy czegoś takiego nie widziałem, ale zamiast kombinować może lepsze będzie coś takiego:

[PHP] pobierz, plaintext 
<?php
 
$ilosc = 3;
$a = 2;
 
for ($i = 0; $i < $ilosc; $i++) {
	if ($i == $a) echo $i;
}
 
?>
[PHP] pobierz, plaintext 

W php generujesz nowe pole input - ukryte w formularzu przypisując "nazwie_zmiennej" - "wartość" którą pobrałeś z x.html

[HTML] pobierz, plaintext 
<input type="hidden" name="nazwa_zmiennej" value="wartosc" />
[HTML] pobierz, plaintext 

Wada jest taka że ktoś może Ci podstawić formularz i wysłać z inną wartością do ostatniej strony, dlatego musisz się zabezpieczyć w odpowiedni sposób - dodatkowa walidacja treści.