[PHP]pomoc przy zadaniu, czy jest ktoś w stanie zrobić i wytłumaczyć co i jak powinno być w tym Opcje

[loooki92]

Witam, mam problem z poniższym zadaniem. Dostałam kilka zadań ale z tym jednym nie mam kompletnie pojęcia jak się za nie zabrać. Znajdzie się ktoś na tyle wyrozumiały zeby je zrobić i możliwie jak najdokładniej skomentował?

Napisz funkcję, która:
- zbierze możliwe informacje o użytkowniku z $_SERVER
- na ich podstawie utworzy ich stosunkowo unikalny identyfikator (maks 32 znaki) i go zwróci
/function userUniqId(){

}
$userId=userUniqId();
/*

[mar1aczi]

Zacznij od tego: http://php.net/manual/en/reserved.variables.server.php
Wyszukaj, które dane z tej tablicy odnoszą się do użytkownika i można z nich skorzystać do wygenerowania unikalnego identyfikatora.

[KsaR]

Hm.
Możesz pobrać np.

[PHP] pobierz, plaintext 
$userAgent=$_SERVER['HTTP_USER_AGENT'];
$ip=$_SERVER['REMOTE_ADDR'];
[PHP] pobierz, plaintext 

I zahashować je jakimś algorytmem który zrobi to jak najbardziej unikalnie.
Np.

[PHP] pobierz, plaintext 
echo hash_hmac('sha512', $userAgent, $ip); # $ip jako sól 
[PHP] pobierz, plaintext 

Jednak nie mam pewnosci czy o takie cos chodzilo

Ten post edytował KsaR 23.09.2015, 13:18:04

[loooki92]

Zrobiłam jednak coś takiego:

[PHP] pobierz, plaintext 
$agent = "X".$_SERVER['HTTP_USER_AGENT'];
$przegladarka = array('Internet Explorer' => 'MSIE', 'Mozilla Firefox' => 'Firefox'
				,'Opera' => 'Opera', 'Chrome' => 'Chrome');
 
foreach ($przegladarka as $nazwa => $id)
	if (strpos($agent, $id)) $przegladarka = $nazwa;
echo "Twój IP: <b>".$_SERVER['REMOTE_ADDR']."</b><BR>";
echo "Przegladarka: <b>".$przegladarka."</b><BR>";
[PHP] pobierz, plaintext 

Pokazuje mi teraz moje IP oraz przeglądarke z jakiej korzystam tylko teraz jak zrobić ten uniwersalny identyfikator?

[Pyton_000]

tobie wystarczy md5()

[redeemer]

tobie wystarczy md5()

(maks 32 znaki)

Tak naprawdę to już jest podpowiedź jakiego hashowania użyć.

[KsaR]

Ano, nie doczytalem tego 32.
Kierowalem sie powtarzalnoscia.

[loooki92]

Czyli mniej więcej taki kod powinien być już ok? :

[PHP] pobierz, plaintext 
$userAgent= "X".$_SERVER['HTTP_USER_AGENT'];
$ip=$_SERVER['REMOTE_ADDR'];
$przegladarka = array('Internet Explorer' => 'MSIE', 'Mozilla Firefox' => 'Firefox'
				,'Opera' => 'Opera', 'Chrome' => 'Chrome');
$hash = md5($userAgent);
 
foreach ($przegladarka as $nazwa => $id)
	if (strpos($userAgent, $id)) $przegladarka = $nazwa;
echo "Twój IP: <b>".$_SERVER['REMOTE_ADDR']."</b><BR>";
echo "Przegladarka: <b>".$przegladarka."</b><BR>";
echo ($hash);
[PHP] pobierz, plaintext 

Zrobię to tak jak jest wyżej, a teraz mam kolejne pytanie, a w zasadzie prośbę o sprawdzenie:
Treść:
funkcja, która przyjmie jako parametr, wynik funkcji stringRepeat i
- sprawdzi czy string zawiera napis 'koniecznie' i umieści wynik tego sprawdzenia
w zmiennej $stringExist o typie boolean
- policzy ilość wystąpięń stringu podanego jako parametr $newString i umieści wynik
w zmiennej $stringCount o typie int

Zrobiłam coś takiego ale nie do końca jestem przekonana co do poprawności.

Z góry dziękuje za pomoc

[PHP] pobierz, plaintext 
function stringExistCount($multipliedString, $newString) 
{
    if (strpos($multipliedString, "koniecznie") === true)
    {
        $stringExists = True;
    }
     else
    {
        $stringExists = False;
    }
    $stringCount = substr_count($multipliedString, $newString);
        return array($stringExists, $stringCount);
}
$multipliedString = str_repeat('koniecznie', 20);
$result = stringExistCount($multipliedString, 'koniecznie');
echo $result[1] ? 'true ' : 'false ';
echo $result[1];
[PHP] pobierz, plaintext 

Ten post edytował loooki92 23.09.2015, 17:02:13

[thek]

Ogólnie podejście dobre, choć zapomniałaś, że indeksowanie tablicy jest od 0, a nie 1. No i można nieco sprawę zoptymalizować przyjmując pewne wartości domyślne. Funkcję masz jednak wykonaną w miarę poprawnie ( poza sprawdzeniem wystąpienia... sprawdź co i kiedy zwraca ta funkcja)i to o czym wspominam to już nieco "czepialstwo". Ogólnie można też zamiast posługiwać się w wyniku indeksem tablicy asocjacyjnej, by wiadomo było czym są 0 i 1 indeks wyniku. Stąd bardziej optymalne jest moim zdaniem wyrzucenie osobnych zmiennych (choć to pogwałcenie założeń zadania ) i od razu posłużenie się tablicą. W efekcie powstanie nam:

[PHP] pobierz, plaintext 
function stringExistAndCount($search, $newString) {
    $result = ['stringExists' => false, 'stringCount' => 0]; /* wartości domyślne */
    if (strpos($search, "koniecznie") !== false) {
        $result['stringExists'] = true;
    };
    $result['stringCount'] = substr_count($search, $newString);
    return $result;
}
[PHP] pobierz, plaintext 

A teraz pytanie jakie możesz zadać nauczycielowi, by go zaskoczyć Czemu gdy wywołamy funkcję w postaci stringExistAndCount('alalalala', 'ala') wynik będzie ['stringExists' => false, 'stringCount' => 2] a nie przykładowo ['stringExists' => false, 'stringCount' => 4] Jeśli tego sama nie wiesz, to sprawdź jak działa substr_count.