Forum PHP.pl

mb_ to prefix od 'Multibyte' (a same funkcje pochodzą z modułu mbstring) uwzględniający przeróżne kodowania łańcuchów znaków na których operują.

Więcej informacji "co, jak i dlaczego" w samej dokumentacji: http://php.net/manual/en/intro.mbstring.php

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 

I jesteś pewien, że parser nie mówi Ci, że w linii 12 brakuje średnika?

Edit: Jak można programować z wyłączonym powiadamianiem o błędach i zadawać na forum pytania "Dlaczego mi nie działa"?

Możesz to zrobić w dowolnym momencie. W skrócie, robisz to tak:



I w sumie tyle. Branch 'core' tak naprawdę prowadzi do brancha 'master' repozytorim 'git@github.com:core.git'

Jak napiszesz na branchu 'myfeature' jakiś feature który chciałbyś wrzucić do 'git@github.com:core.git':



To ostatnie polecenie jest po to, żebyś miał 'core' na dole, a wszystkie swoje zmiany projektowe na górze. To powinno wywalić commit który cherry-pickowałeś, bo nie jest już potrzebny (jest częścią 'core'). Tylko musisz pamiętać, że jeśli Twój projekt dorobił się tysięcy commitów, to zmiana bazy 'core' może trochę potrwać. To wygląda tak, że git commit po commicie nakłada Twoje zmiany na bazę. To może spowodować konflikty, jeśli commit który cherry-pickowałeś jest zależny od struktury którą zbudowałeś dopiero w projekcie, ale to oczywiste. Jeśli w pewnym momencie zatrzyma Ci się rebase na konflikce, to po konflikcie nie zaczynasz procesu od nowa (bo dalej 'trwa' rebase) tylko kontynuujesz od miejsca w którym się zatrzymało przez polecenie:



Może się też zdarzyć, że proces rebaseowania zatrzyma Ci się w momencie commitu który cherry-pickowałeś. Tutaj po prostu nie dam sobie głowy uciąć, bo zwyczajnie nie pamiętam, czy git załatwi to sobie automatycznie i pominie pusty commit czy też powie Ci, że commit jest pusty (bo zmiany przez tego nakładane zostały już nałożone wcześniej). Jeśli to drugie, to ignorujesz commit jako niepotrzebny poleceniem:



Więcej Ci nie potrzeba w sumie.

Edit: Jeśli rebase nie idzie po Twojej myśli, konfliktów pełno i końca nie widać, to zawsze możesz go przerwać za pomocą:



To w ułamku sekundy przywróci Ci stan sprzed polecenia rebase.

Ogólnie rebase to najpotężniejsze narzędzie jakie stoi za gitem. Za tydzień nie będziesz mógł bez niego żyć. Przenoszenie, łączenie, zmiana nazw commitów jest mega wygodna za pomocą trybu rebase interactive.

Piractwo to nie jedyna droga łamania prawa. Zastanów się, czy Twój serwis nie narusza regulaminu serwisu Wrzuta.pl, w tym punktu II.4.g:

Od takich operacji jest funkcja str_pad();

[PHP] pobierz, plaintext 
setlocale(LC_TIME, 'pl_PL');
echo strftime('%F', time());
[PHP] pobierz, plaintext 

@Down: Owszem, jest wiele sposobów . Chciałem tylko powiedzieć, że str_pad() to funkcja przeznaczona do takich i tylko takich celów, dająca w tej kwestii nieco większe możliwości

No brakuje Ci kilku podstawowych rzeczy w tym skrypcie, a inne są umieszczone w złym miejscu. Śmiem też wątpić w to googlowe borykanie się, a sam kod jest lekko... posklejany . To nie jest Twój kod, prawda?

[PHP] pobierz, plaintext 
<form action="logowanie.php" method="post">
 
<?php
 
$directory = '../images/galeria/';
 
echo 'Pliki w '.$directory.': <select name="file">';
foreach ( glob($directory.'*') as $file ) {
    if ( is_file($file) ) {
        echo '    <option>'.$file.'</option>';
    }
}
echo '</select>';
if ( !empty($_POST['file']) ) {
    unlink($directory.$_POST['file']);
}
 
?>
 
<input type="submit" value="Usuń">
</form>
[PHP] pobierz, plaintext 

Wersja poprawiona, ale ostrzegam, że to niebezpieczna rzecz.

Wersja alternatywna, nie tak prosta (chyba, że nie widzisz przeszkód w przechowywaniu $a jako stringu od początku):

[PHP] pobierz, plaintext 
$a = 1234;
$aStr = (string)$a;
$b = $aStr[2];
[PHP] pobierz, plaintext 

Usuń spację pomiędzy dwukropkiem a wyrazem Location. Poprawny nagłówek wygląda:

[PHP] pobierz, plaintext 
header("Location: xxxx");
[PHP] pobierz, plaintext 

Nie mam w tej chwili możliwości przetestowania czegokolwiek, ale dopisz return do onClick, czyli:

[PHP] pobierz, plaintext 
$_SESSION["przedstaw"] = $login;  
[PHP] pobierz, plaintext 

A jesteś pewien, że przypisujesz tutaj właściwą zmienną? Bo na moje oko, wg. Twojego kodu nadpisujesz wartość zmiennej sesyjnej, niezadeklarowaną wcześniej zmienną $login. Nie widzę, reszty kodu, ale powinno to być raczej:

[PHP] pobierz, plaintext 
  $login = $_SESSION["przedstaw"];    
[PHP] pobierz, plaintext 

Ależ nazwy plików można poznać (php: glob()), a wtedy i usunąć

[PHP] pobierz, plaintext 
$path = 'sciezka/do/plikow/';
$xmls = glob($path.'*.xml');
foreach ($xmls as $file) {
    unlink($path.$file);
}
[PHP] pobierz, plaintext 

@Down: Przepraszam za to - pisałem z pamięci i nie pamiętałem czy glob zwraca pełną ścieżkę, czy tylko pełną nazwę pliku/katalogu

Skoro dopiero plik miałby Ci podpowiedzieć jakich zmiennych używać, to w jaki sposób napisałabyś resztę kodu?


A nie lepiej użyć tablicy wielowymiarowej? Przecież, skoro zależy Ci tylko i wyłącznie na tym, by wczytać dane z pliku i na nich operować, a jedynym wyznacznikiem tych danych jest numer kolumny w której się znajduje, to tablica jest przecież elementem który do takich celów został właśnie stworzony .

[PHP] pobierz, plaintext 
$plik = file('twoj_plik_csv.txt');
$dane = array();
foreach ( $plik as $number => $line ) {
    $dane[] = explode('|', $line);
}
[PHP] pobierz, plaintext 

No i teraz przecież łatwo dostaniesz się do odpowiedniej linii i wartości kolumny (obrazowo):

[PHP] pobierz, plaintext 
$linia = 12;
$kolumna = 3;
$to_co_chce = $dane[$linia][$kolumna];
[PHP] pobierz, plaintext 

Trzeba tylko pamiętać, że klucze tablicy liczone są od 0, więc odnoszenie sie do klucza 3 de facto będzie prowadziło do kolumny czwartej [0, 1, 2, 3] (sytuacja ma się analogicznie dla linii). Da się to oczywiście obejść, ale... po co. Taka jest specyfika tablic i tego trzeba się trzymać, a nie obchodzić.

PS: Od takich operacji na plikach (zwanych plikami CSV) są odpowiednie funkcje (fgetcsv() i fputcsv()) ale nie zagłębiając się w szczegóły, można sobie bez nich poradzić nie najgorzej .
PS2: Z tablicami i iteracja jest łatwiejsza i życie prostsze... Zalet jest mnóstwo a wad bardzo niewiele

Twoja sekcja:

[PHP] pobierz, plaintext 
case 'required':
    (...)
[PHP] pobierz, plaintext 

Jest źle napisana. Sprawdzanie walidacji jest ograniczone do pierwszego poprawnego elementu (linia 89 i 90: return true;). Ten warunek jest tam zupełnie nie potrzebny. Nie zagłębiając się w resztę:

[PHP] pobierz, plaintext 
  (...)
  public function Validation() {
    foreach($this -> rules as $key => $rule) {
      foreach($rule as $post => $validRule) {
        $methodRule = explode('|', $validRule);
        $methodRule = $methodRule[count($methodRule)-1];
        switch($methodRule) {
          case 'required':
            if(method_exists(get_class($this), $methodRule)) {
              $required = $this -> $methodRule($post);
              if(!$required) {
                $this -> msg =  'Pole '.$key.' nie moze byc puste';
              }
            } 
            break;
        }
      }    
    }
  }
  (...)
[PHP] pobierz, plaintext 

I będzie śmigało...

Czyli możesz napisać wprost, że zgodne nie jest, bo autor w pierwszym poście pytał przecież:


Ale wracając do Twojego ostatniego pytania, tak jak nie wprost powiedział kolega: Ze wzorcem MVC zgodne nie będzie. Najpoprawniej i najwydajniej byłoby tworzyć obiekty klasy Human z już istniejących, pobranych przez odpowiedni model danych. Tworzenie obiektu, który dane pobiera sobie indywidualnie przez różnorakie metody (chociaż konstruktor byłby bardzo odpowiedni) ma niewielką rację bytu i nie widzę większego sensu tworzenia takich elementów. Owszem, jako pytanie teoretycznie przykład dobry, ale uczulam na takie zagrania . Widzę, że największe problemy masz ze zrozumieniem czym właściwie jest wzorzec MVC i jak go zastosować. Po krótce mogę powiedzieć Ci tylko tyle, że to obiekty będące Modelami powinny wykonywać wszelkie operacje we/wy na danych z zewnętrznych źródeł (tj. baza danych, plik etc) i dopiero same dane przekazywać poszczególnym, wyspecjalizowanym obiektom. Model działa tutaj jako pośrednik i chociaż składnię może mieć niewiele bardziej skomplikowaną od Twojego przykładu i na dobrą sprawę jest zupełnie zbędny (obiektowość też jest przecież zbędna) - ujednolica projekt i sprowadza do stosowania metod modelu zamiast każdorazowego operowania zapytaniami czy plikami. To tyle jeśli chodzi o sam model. Literatury elektronicznej o MVC google podpowie o wiele więcej niż jestem w stanie sobie wyobrazić - dlatego nie będę próbował się z Google siłować .

Podsumowując: nawet jeśli przykład...

[PHP] pobierz, plaintext 
class Human {
    public $id = 12345;
    (...)
    public function getHuman($id) {
        $sql = "select * from tabela wher eid = 'id'";
        return (...);
    }
}
[PHP] pobierz, plaintext 

... nie jest poprawny, to ten już jest:

[PHP] pobierz, plaintext 
class PopulationModel {
    public function getHumanById($id) {
        $sql = "select * from tabela wher eid = 'id'";
        return (...);
    }
}
 
class Human {
    public $id = 12345;
    public function getHuman($id) {
        $population = new PopulationModel();
        return $population->getHumanById($this->id);
    }
}
[PHP] pobierz, plaintext 

Nie skupiam się w tym momencie nad sensownością, ale zachowaniem funkcjonalności przy jednoczesnej zgodności z MVC.

Nie bardzo rozumiem, co chcesz powiedzieć przez oryginał/kopia bazy. Domyślam się, że chodzi Ci o objekt klasy bazy danych, o jego instancje. Wg. przykładu który podałeś, wszystko jest w porządku. Obiekt tworzysz tylko raz (... = new database();) a dalej operujesz tylko na jego uchwytach. Rejestr służy do przechowywania takich instancji w klasie statycznej i sprawuje nad nimi pewną kontrolę. Nie ma znaczenia czy operujesz na Registry::get('db'); czy bezpośrednio na $db, bo obie wartości są uchwytami tego samego obiektu. Więc, jeśli na końcu pliku dasz przykładowo Registry::get('db')->QueryCount(); czy $db->QueryCount(); wyświetli tą samą wartość, uwzględniającą wszystkie użycia obiektu klasy database, niezależnie gdzie i kiedy były używane.


Początkowo używanie rejestru może wydać sie niejasne, ale niewątpliwie ma wiele zalet (chociaż są opozycjoniści tego punktu widzenia). Przede wszystkim zapewnia pewną spójność, a nawet pokusiłbym sie o stwierdzenie: wygodę, pozwala sprawować pełną kontrolę na przepływem instancji czy operacje na nich. Podstawową funkcjonalnością jest możliwość dostępu do instancji danej klasy z dowolnego miejsca aplikacji (klasa statyczna) bez każdorazowego tworzenia nowych obiektów oraz bez użycia zmiennych globalnych (podoba funckjonalność). Tą funkcjonalność można rozszerzyć o wyżej wymienione rzeczy (kontrolę, operacje, wyjątki etc. - czego dusza zapragnie). Prosty, podstawowy sposób użycia:

Zamiast używania zmiennych globalnych (przykład poniższy):

[PHP] pobierz, plaintext 
<?php
$db = new database();
...
class przyklad
{
	public function funkcja()
	{
		global $db;
		$db->exeQuery(...);
	}
}
?>
[PHP] pobierz, plaintext 

Użyjesz:

[PHP] pobierz, plaintext 
<?php
Registry::setObj("baza", new database());
  ...
  class przyklad
  {
	  public function funkcja()
	  {
		  Registry::getObj("baza")->exeQuery(...);
	  }
  }
?>
[PHP] pobierz, plaintext 

Pozostałych użyć opisywać nie będę, bo wychodzę z założenia że: "jeśli ich nie znasz to znaczy, że jeszcze ich nie potrzebowałeś". Przyjdzie samo z czasem

Pozdrawiam!
Natt.

[PHP] pobierz, plaintext 
<?php
 
	class moja_klasa
	{
		private $tajne;
		public static $statycznie = "na starcie";
		public function setZmienna($wartosc)
		{
			$this->tajne = $zmienna;
		}
		public function teczkaIPN()
		{
			 return $this->tajne;
		}
		public function __construct($poczatkowa)
		{
			echo "Tworzenie obiektu";
			$this->tajne = $poczatkowa;
		}
	}
 
	$objekt = new moja_klasa("Jestem agentem");
	echo $objekt->teczkaIPN();
	$objekt->setZmienna("Nie jestem juz agentem! Nigdy nim nie bylem!");
	echo $objekt->teczkaIPN();
	echo moja_klasa::$statycznie;
	moja_klasa::$statycznie = "Zmienna statyczna";
 
?>
[PHP] pobierz, plaintext 

Może i chaotycznie, ale na przykład wystarczy.

Polecam lekturę: Rozdział 19. Classes and Objects (PHP 5)

Dane wejściowe:

[PHP] pobierz, plaintext 
<?php
 
	$imageData = imagecreatefromjpeg($imageFile);
	list($imageWidth, $imageHeight) = getimagesize($imageFile);
	$imageRatioWidth = $imageWidth > $maxImageWidth ? $maxImageWidth / $imageWidth : 1;
	$imageRatioHeight = $imageHeight * $imageRatioWidth > $maxImageHeight ? $maxInameHeight / $imageHeight : 1;
	$newImageSizeWidth = floor($imageWidth * $imageRatioWidth * $imageRatioHeight);
	$newImageSizeHeight = floor($imageHeight * $imageRatioWidth * $imageRatioHeight);
	$newImage = imagecreatetruecolor($newImageSizeWidth, $newImageSizeHeight);
	imagecopyresampled($newImage, $imageData, 0, 0, 0, 0, $newImageSizeWidth, $newImageSizeHeight, $imageWidth, $imageHeight);
	$savePath = $newImageName.".jpg";
	imagejpeg($newImage, $savePath, $imageQuality);
 
?>
[PHP] pobierz, plaintext 

Maksymalny rozmiar docelowy ($maxImageWidth i $maxImageHeight) to właściwie rozmiar pliku który chcesz uzyskać po zmianie rozmiaru z tą tylko różnicą, że jeśli grafika wejściowa będzie mniejsza niż wyjściowa, nie ulegnie zmianie (zapobiega rozciąganiu małych obrazków).

Cudowna literówka a jak zmienia sens zdania .

Najprostszy przykład:

[PHP] pobierz, plaintext 
<?php
 
	$mysql_grip = mysql_connect("localhost","uzytkownik", "haslo");
	mysql_select_db("nazwa_bazy", $mysql_grip);
 
	if(!empty($_GET["delete_id"])) { mysql_query("DELETE FROM nazwa_tabeli WHERE id_pola = ".$_GET["delete_id"], $mysql_grip); }
 
	$mysql_result = mysql_query("SELECT * FROM nazwa_tabeli", $mysql_grip);
	echo "<table>";
	while($mysql_wynik = mysql_fetch_assoc($mysql_result))
	{
		echo "<tr><td>".$mysql_wynik["id_pola"]."</td><td>".$mysql_wynik["jakies_pole"]."</td><td>".$mysql_wynik["inne_pole"]."</td><td><input type='button' onclick='document.location = index.php?delete_id=".$mysql_wynik["id_pola"]."' value='Usuń'></td></tr>";
	}
	echo "</table>";
 
?>
[PHP] pobierz, plaintext 

Wywal cudzysłów z lini 7, ale nie z przypisywanych wartości jak w pierwszym poście bo to nie ma sensu, tylko z argumentu warunku IF, tak by było:

[PHP] pobierz, plaintext 
<?php
if ($HTTP_REFERER==$dan[0])
?>
[PHP] pobierz, plaintext 

Dalej. Raczej korzystaj z $_SERVER["HTTP_REFERER"].
Zamiast takiej formy pętli używaj:

[PHP] pobierz, plaintext 
<?php
while(($dan = fgetcsv($handle, 100)) !== false)
{
...
}
?>
[PHP] pobierz, plaintext 

Wtedy zobacz co się dzieje.

Tak jak napisał b_chmura:

A blokują bo takie wyskakujące okienka to uciążliwe paskudztwo i zły nawyk...

Ale blokuje nie blokuje, rozwiazanie masz takie [JS]:

Jak korzystasz z tego co ja wkleiłem ($users_in_group) to dodaj przed: $users_id = array(); linię $users_in_group = ""; i powinno być OK.