Forum PHP.pl

Niestety, z kawałka kodu który wkleiłeś ciężko wywnioskować co tak właściwie chcesz osiągnąć. Widzę też pewne naleciałości z innych języków.

Ale po kolei:

[PHP] pobierz, plaintext 
if ($k > $n) {
    echo "Nieprawidłowe dane wejściowe";
}
[PHP] pobierz, plaintext 

Słuszne założenie, ale zwykły if i echo nie powstrzymają wywoływania dalszej części skryptu mimo oblania warunku walidacji. Sugeruję jakiś wyjątek, return lub pospolity die/exit.

[PHP] pobierz, plaintext 
$numbers = array($n);
[PHP] pobierz, plaintext 

Nie robi tego co wydaje Ci się, że robi. Właśnie utworzyłeś jednoelementową tablicę zawierającą element $n. W skrócie w PHP jest dynamiczna alokacja pamięci tablic, więc taka deklaracja nie jest kompletnie potrzebna.

[PHP] pobierz, plaintext 
$numbers = array($n);
 
for ($i=0; $i<$n; $i++) {
    $numbers[$i] = $i + 1;
}
[PHP] pobierz, plaintext 

To wszystko da się przede wszystkim zastąpić prostym:

[PHP] pobierz, plaintext 
$numbers = range(1, $n);
[PHP] pobierz, plaintext 

[PHP] pobierz, plaintext 
$r = floor(rand(0, $n));
[PHP] pobierz, plaintext 

Nie ma potrzeby zaokrąglania wartości zwracanej przez rand. PHP zwraca liczbę całkowitą z podanego przedziału (lub [0 .. getrandmax()] jeśli przedziału nie podasz).

[PHP] pobierz, plaintext 
// losowanie k liczb
for ($i=0; $i<$k; $i++) {
 
    // tworzenie losowego indeksu pomiędzy 0 i n - 1
    $r = floor(rand(0, $n));
 
    // wybieramy element z losowego miejsca
    echo $numbers[$r]." | ";
 
    // przeniesienia ostatniego elementu do miejsca z którego wzięliśmy
    $numbers[$r] = $numbers[$n - 1];
 
    //zmniejszamy n
    $n--;
}
[PHP] pobierz, plaintext 

To również robienie na piechotę tego, co w PHP jest wbudowane. Wystarczy prosta funkcja:

[PHP] pobierz, plaintext 
$random = array_rand($numbers, $k);
[PHP] pobierz, plaintext 

Nie jestem przekonany dlaczego chciałeś sortować dla każdego losowania tablicę $numbers, ale zakładam, że chodziło Ci o posortowanie liczb już wylosowanych. Sortowanie wykonujesz więc tylko raz, na końcu. W skrócie, całość da się zapisać:

[PHP] pobierz, plaintext 
<?php
 
function losuj($k, $n)
{
    if ($k > $n) {
        die("Nieprawidłowe dane wejściowe");
    }
 
    $numbers = range(1, $n);
    $random = array_rand($numbers, $k);
    sort($random);
 
    return $random;
}
 
 
$k = 9;
$n = 805;
 
$wylosowane = losuj($k, $n);
 
print_r($wylosowane);
 
?>
[PHP] pobierz, plaintext 

Ten post edytował Nattfarinn 13.04.2015, 13:47:46