Przypisywanie przedziałom wartości słownej - tablica Opcje

[pawdoh]

Witam mam następujący problem otóż, pobieram z bazy danych wartość będącą liczbą i teraz mam problem ponieważ potrzebuję to przypisać mam kod

[PHP] pobierz, plaintext 
<?php
$ranga = (
range(0, 3999) => 'Początkujący',
range(4000, 7999) => 'Młodzian',
range(8000, 11999) => 'Doświadczony',
range(12000, 20000) => 'Weteran',
)
?>
[PHP] pobierz, plaintext 

następnie to przypisuje w pliku do którego załączam tą tablice, $ranga[$r[rank]]. I po tym właśnie zwraca mi błąd

Warning: Illegal offset type in ... on line 3,4,5,6

Bardzo bym prosił o pomoc gdyż jestem początkującym programistą i nie wiem za bardzo gdzie zrobiłem błąd. Pozdrawiam

[kulczycki]

Nie możesz ID z tablicy przypisać funkcji. Może być jedynie zmienna. Jeśli chcesz uzyskać podobny efekt to byś musiał zrobić

[PHP] pobierz, plaintext 
foreach (range(0, 3999) as $number)
   $ranga[$number] = "Początkujący";
 
foreach (range(4000, 7999) as $number)
   $ranga[$number] = "Młodzian";
 
foreach (range(8000, 11999) as $number)
   $ranga[$number] = "Doświadczony";
 
foreach (range(12000, 20000) as $number)
   $ranga[$number] = "Weteran";
[PHP] pobierz, plaintext 

Ale to i tak jest błędne. Jeśli pobierasz X wartość z bazy to po prostu robisz

Kod

if($wartosc >= 0 && $wartosc <= 3999)
$ranga = "Początkujący";
elseif($wartosc >= 4000 && $wartosc <= 7999)
$ranga = "Młodzian";

i tak dalej.

Ten post edytował kulczycki 2.01.2011, 22:56:23

[cycofiasz]

Kluczem w tablicy może być string lub liczba całkowita - w Twoim kodzie próbujesz wstawić tam rezultat działania funkcji range czyli tablicę.

[Crozin]

Zapisujesz sobie jedynie górną granicę i etykietę, a później sprawdzasz po kolei czy ranga użytkownika jest wystarczająco wysoka:

[PHP] pobierz, plaintext 
$ranges = array(
  20000 => 'Weteran',
  11999 => 'Doświadczony',
  7999  => 'Młodzian',
  3999  => 'Początkujący'
);
 
foreach ($ranges as $range => $label) {
  if ($user->getRange() < $range) {
    continue;
  }
 
  echo $label;
 
  break;
}
[PHP] pobierz, plaintext 

Ten post edytował Crozin 3.01.2011, 09:21:08

[pawdoh]

Nie możesz ID z tablicy przypisać funkcji. Może być jedynie zmienna. Jeśli chcesz uzyskać podobny efekt to byś musiał zrobić

[PHP] pobierz, plaintext 
foreach (range(0, 3999) as $number)
   $ranga[$number] = "Początkujący";
 
foreach (range(4000, 7999) as $number)
   $ranga[$number] = "Młodzian";
 
foreach (range(8000, 11999) as $number)
   $ranga[$number] = "Doświadczony";
 
foreach (range(12000, 20000) as $number)
   $ranga[$number] = "Weteran";
[PHP] pobierz, plaintext 

Ale to i tak jest błędne. Jeśli pobierasz X wartość z bazy to po prostu robisz

Kod

if($wartosc >= 0 && $wartosc <= 3999)
$ranga = "Początkujący";
elseif($wartosc >= 4000 && $wartosc <= 7999)
$ranga = "Młodzian";

i tak dalej.

Zrobiłem jak kolega powiedział ale jeśli mam poniżej Ranga: $ranga to nic nie wyświetla popełniłem gdzieś błąd?

[PHP] pobierz, plaintext 
	elseif($array['ranga] > -999 && $array['ranga] < 999)
		$ranga = 'ranga1';
		elseif($array['ranga] > 1000 && $array['ranga'] < 3999)
$ranga = 'ranga2';
		elseif($array['ranga'] > 4000 && $array['ranga'] < 7999)
	$ranga = 'ranga3';
		elseif($array['ranga'] > 8000 && $array['ranga'] < 11999)
	$ranga = 'ranga4';
[PHP] pobierz, plaintext 

[Crozin]

[...] popełniłem gdzieś błąd?

Posłuchałeś @kulczycki, który zaproponował utworzenie tablicy złożonej z 20 000 elementów tylko po to, by sprawdzić którą z 4. etykiet przypisać użytkownikowi.

[kulczycki]

Posłuchałeś @kulczycki, który zaproponował utworzenie tablicy złożonej z 20 000 elementów tylko po to, by sprawdzić którą z 4. etykiet przypisać użytkownikowi.

To wróć do góry i zobacz co zaproponowałem. Pokazałem mu tylko jak by wygładał jego przykład a nie zaproponowałem . A rozwiązanie podałem najłatwiejsze z możliwych. Do tego twój przykład letko waldiwy jest.

Ten post edytował kulczycki 3.01.2011, 07:40:10

[thek]

Ja bym nie ufał przedziałom górnym. Kulczycki ma rację co do pewnego małego babola u Ciebie Crozin. Nie reagujesz na przekroczenie zakresu maksymalnego w ostatnim wypadku dla weterana. Gdy ktoś przekroczy 20000 - nie pokaże nic.

[PHP] pobierz, plaintext 
if($wartosc >= 0) {
  if( $wartosc > 3999 ) {
    if( $wartosc > 7999 ) {
      if( $wartosc > 7999 ) {
        if( $wartosc > 11999 ) {
          $ranga = 'Weteran';          
        } else {
          $ranga = 'Doświadczony';
        }      
      } else {
        $ranga = 'Doświadczony';
      }  
    } else {
      $ranga = 'Młodzian';
    }
  } else {
    $ranga = 'Początkujący';
  }
} else {
  $ranga = 'Cheater'; //jeśli wrzucasz kogoś na minus to na bank coś przeskrobał :)
}
[PHP] pobierz, plaintext 

Tu masz wersję zagnieżdżona dla rang. Nie jest ładna, ale zadziała jak trzeba. Jeśli chcesz, by funkcja była dynamicznie modyfikowalna (co jakiś czas zmieniasz nazwy rang lub wartości) to stwórz funkcję, gdzie jako parametr przekazujesz tablicę z zakresami dolnymi jako klucze Przykład masz tutaj mniej więcej...

[PHP] pobierz, plaintext 
<?php
$tabela_zakresow = array(
    0 => 'Początkujący',
    12000 => 'Weteran',
    4000 => 'Doświadczony',
    8000 => 'Młodzian'
);
//ważne są klucze i nazwy. Nie muszą być po kolei.
 
function ranga( $staty, $rangi ) {
    //sprawdzam czy mi podano tablicę rang. Jeśli nie - rzucam false, ale można też tutaj sypnąć wyjątkiem, jak kto woli.
    if( !is_array( $rangi ) ) {
        return false;
    }
    //tu sprawdzam czy mi podano liczbę postów/artykułów/czegoś usera, tu też może być wyjątek.
    if( !is_numeric( $staty ) ) {
        return false;
    }
    //sortowanie kluczy! dzięki temu nie muszę ich podawać w tablicy po kolei :) Te sortowanie załatwia mi ów problem
    ksort( $rangi, SORT_NUMERIC );
    //magia  
    end($rangi);
    while ( !is_null( $key = key($rangi) ) ) {
        if( $key > $staty ) {
            prev($rangi);
        } else {
            return current($rangi);
        }
    }
    return 'Cheater!';// To jeśli w tablicy brak rangi, a więc poniżej progu najniższego
}
 
echo ranga( 'test', $tabela_zakresow ).'<br />';//powinno zwrócić pustkę lub false jeśli zrobisz var_dump( ranga( 'test', $tabela_zakresow ) ) :)
echo ranga( -666, $tabela_zakresow ).'<br />';
echo ranga( 10000, $tabela_zakresow ).'<br />';
echo ranga( 4000, $tabela_zakresow ).'<br />';
echo ranga( 20000, $tabela_zakresow ).'<br />';
echo ranga( 500, $tabela_zakresow ).'<br />';
?>
[PHP] pobierz, plaintext 

Oczywiście "magia" to żart. Posortowaną tablicę przeszukuję od końca tak długo, aż nie dojdę do początku. Gdy jestem w pętli to porównuję wartość liczbową podaną w funkcji do aktualnego progu i jeśli klucz jest większy to lecę do elementu wcześniej w tablicy. Jeśli nie jest to prawda, to znaczy, że wpadłem we właściwy zakres i zwracam wartość dla tego klucza. Jeśli dojdę do końca i nie znajdę pasującego to wywalam rangę domyślną dla oszusta

Zwracaj uwagę na to co zwraca funkcja ranga. jeśli dała w wyniku false to sygnalizuj błąd. Uważaj na 0... Jeśli zrobisz głupie

[PHP] pobierz, plaintext 
if( ranga(0, $tabela) ) {
} else {
   else 'error';
}
[PHP] pobierz, plaintext 

to wywali Ci error jeśli będziesz zwracał jako nazwę 0, '0' lub cokolwiek innego dające się do false przekonwertować w locie. Powinno być:

[PHP] pobierz, plaintext 
if( ranga(0, $tabela) !== false ) {
} else {
   else 'error';
}
[PHP] pobierz, plaintext 

Do latania po pętli mogłem też użyć foreach, co jest wygodne przy małych petlach. Ale tutaj dla celów edukacyjnych użyłem while, byś wiedział, że tak też można, a dodatkowo przy ogromnych pętlach while jest wydajniejsze. Tutaj to tylko sztuka dla sztuki i ukazanie że można. Równie dobrze możesz to foreach zastąpić, bo wątpliwe byś miał kiedykolwiek więcej niż kilka tysięcy rang Dodatkowo jeśli masz pewność, że klucze są już podane w odpowiedniej kolejności od najmniejszej do największej to możesz usunąć z wnętrza ksort, który spowalnia całość.

Powód edycji: [thek]: [thek]: Poprawiłem literówki małe i dodałem komentarze :)

[Crozin]

@thek: Ty to się potrafisz rozpisać.

1. Zapewne przekroczenie rangi "weteran" jest niemożliwe. Zresztą... zawsze, można zamienić znak mniejszości na większości i porównywać minimalny zakres rangi, bo jak się domyślam, ów "weteran" to maksimum.
2. Do wywalania błędów używaj wyjątków, a nie "flagi" w postaci zwrócenia false - to jest chyba jedna z bardziej "błędotwórczych" praktyk.
3. Żeby się już nikt nie czepiał:

[PHP] pobierz, plaintext 
function getRange($userRange) {
  static $ranges = array(
    'Początkujący' => array(0, 3999),
    'Doświadczony' => array(4000, 7999),
    'Młodzian'     => array(8000, 11999),
    'Weteran'      => array(12000, 20000)
  );
 
  foreach ($ranges as $label => $range) {
    if ($userRange >= $range[0] && $userRange <= $range[1]) {
      return $label;
    }
  }
 
  throw new OutOfRangeException("Value '$userRange' does not match any range.");
}
[PHP] pobierz, plaintext 

Ten post edytował Crozin 3.01.2011, 21:58:06

Powód edycji: [thek]: poprawiłem Citablicę statyczną boś mialsamych weteranów :)

[thek]

@Crozin:
ad1) "Zresztą... zawsze, można zamienić znak mniejszości na większości i porównywać minimalny zakres rangi" -> właśnie to zaproponowałem jako lepszy pomysł. Zakres górny w przypadku liczb rosnących jest źródłem potencjalnych błędów w przyszłości.

ad2) Wyjątki także sugerowałem w komentarzach w kodzie, ale zostawiłem wybór. Poza tym dobrze napisany skrypt nie sprawi problemu. Wiesz co dostajesz na wyjściu, co dajesz na wejściu i nie martwisz się logiką biznesową wnętrza. Wyjątki są bardzo zdradliwe. Możesz mieć wszystko cacy i nagle coś sprawi, że poleci wyjątek, a wysypie Ci aplikację z powodu nie przechwyconego fatala. Ty w trakcie testowania nie musiałeś na taką sytuację trafić, ale kto wie co się stanie po pół roku na serwerze produkcyjnym? Użycie false do sygnalizacji błędu jest w takim momencie "bezpieczne", ponieważ większość skryptów, funkcji php-owych false daje domyślnie jako sygnalizację, że coś poszło nie tak i pod tym kątem większość osób testuje returny z funkcji. Owszem, wyjątki są o wiele lepsze do tego celu i dlatego powstały, ale wymaga to przejrzenia kodu aplikacji by wiedzieć gdzie te wyjątki mogą być rzucone, a więc i gdzie je próbować łapać oraz jakiej są klasy, bo przecież wyjątki także podlegają dziedziczeniu, a czasem zależy nam na łapaniu nie wszystkich, ale tylko określonej klasy wyjątków

ad3) Chyba lepszym wyjściem niż robić break; byłoby od razu walnięcie return. Wtedy spada konieczność sprawdzania czy userRangelabel jest nullem. Dodatkowo przesłałbym tablicę zakresów jako parametr z tej przyczyny, że wtedy funkcja staje się uniwersalną i możesz ją stosować nie tylko dla rzeczy na sztywno i potem definiować osobną do Postów, osobna do artykułów itp. Po prostu byłaby jakąś w helperze do sprawdzania czy liczba mieści się w określonym zakresie. A więc byłaby bardzo elastyczna.

PS: Popraw też static $ranges na coś innego niż weteran

[Crozin]

3) Oczywiście, dobra uwaga - już poprawiłem.
2) PHP niestety nie informuje na poziomie języka o tym jakie wyjątki może dana metoda wyrzucić, ale w miarę dobrze udokumentowany już kod tak. Z return false; jest ten problem, że bardzo często nie sprawdza się czy taka właśnie wartość została zwrócona, wyjątki zaś wymuszają (w przypadku PHP nie jest to do końca prawdą) na nas obsługę tego błędu, albo przynajmniej świadome zignorowanie go.

[thek]

ad3) nadal przypisanie nulla przed foreachem jest zbędne. Ta linia nie robi kompletnie nic i nigdzie nie jest wykorzystywana. No i nadal jako indeksy tablicy statycznej miałeś samych Weteranów, ale poprawiłem Ci to już sam.

ad2) No niestetynie informuje i to jest moim zdaniem pewien problem. Nie zagląda się bowiem nieraz głębiej niż ostatni poziom dziedziczenia. Potem nam z powietrza się we wnuczku pojawia exception z powietrza bo dziadek go implementował lub tatuś Ja tak miałem w Swift mailerze, gdy mi się skrypt mailingu wysypał i jak się okazało wyjątkiem sypnęła jakaś klasa transportu, która robiła za podstawę dla innych Nie zauważyłbym tego, gdyby nie przypadkowa uwaga kumpla, że mailing się nie dostarczył mu na konto testowe. Normalnie caly mailing idzie cronem, z możliwościa wywołania poprzez http, a wyjście normalnie sypie się na /dev/null Dopiero gdy skierowałem wyjście na przeglądarkę i zobaczyłem nieobsłużony wyjątek klasy transportowej przy errorze 500 nagłówka podczas wysyłki to mi kopara opadła A dodam, że zablokowany skrypt nie przeskakuje błędnego adresu, tylko próbuje do oporu. Cały mailing więc był zablokowany.

[Crozin]

3) Tak to jest jak się kod kopiuje i poprawia na szybko. Co do rang - uznałem, że autor nie jest skończonym idiotą i nie skopiuje-wklei tego bez rzucenia okiem. No i nie wyrównałeś ładnie elementów tablicy.

2) Ale dokładnie to samo przytrafi Ci się gdy gdzieś wystąpi błąd, zostanie zwrócone false, a Ty tego nie obsłużysz - takie sytuacje zdarzają się znacznie częściej niż w przypadku wyjątków. Co do nieprzechwyconych wyjątków - m.in. dlatego dobrze jest cały kod objąć blokiem try-catch(Exception), ewentualnie użyć exception handlera by nieobsłużone wyjątki (do czego nigdy nie powinno dojść) gdzieś zapisać, poinformować o ich wystąpieniu.

[kulczycki]

1. co do exception, po co on ?. Od tej funkcji nie zależy działanie całego serwisu, a tylko przypisanie rangi danemu użytkownikowi. Jeśli nie ma x przedziału to po prostu ustawia najbardziej bliską, a jeśli dalej nie może ustawić to wywala pusty albo "No range" - to przecież tylko tekst o range.

2. Cozin twoja pierwsza wersja była dobra ale trzeba lekko dopracować. I ta ostatnia to samo. Bo jeśli ostatnia ranga się gdzieś kończy to przecież nie odbierzesz rangi użytkownikowi tylko dlatego że ma więcej punktów

Może takie niechlujstwo ale trudno:

[PHP] pobierz, plaintext 
function getRange($userRange) {
  static $ranges = array(
    'Początkujący' => array(0, 3999),
    'Doświadczony' => array(4000, 7999),
    'Młodzian'     => array(8000, 11999),
    'Weteran'      => array(12000, 20000)
  );
 
  $max = count($ranges);
  $i = 0;
 
  foreach ($ranges as $label => $range) {
    if($i == $max-1)
        if($userRange >= $range[1])
            return $label;
 
    if ($userRange >= $range[0] && $userRange <= $range[1]) {
      return $label;
    }
 
    $i++;
  }
 
  return "No range";
}
 
echo getRange(50);
echo getRange(5000000000000000);
[PHP] pobierz, plaintext 

[Crozin]

1. Oczywiście możesz zwrócić wartość domyślną w momencie kiedy nie da się określić innej, ale gdyby brak możliwości określenia rangi miał skutkować błędem to powinieneś wykorzystać wyjątek, bo nadaje się on do tego dobrze. I nie musi to być jakiś błąd krytyczny.
2. Zwróć tylko uwagę na jedno: Twoje rozwiązanie jest lekko zdradzieckie, bo działanie kodu jest lekko nielogiczne. Najpierw jasno określasz konkretne przedziały dla danych rang, a później je ignorujesz. Tutaj nie jest to jakoś specjalnie groźne, bo jest to bardzo prosty przykład, ale w bardziej skomplikowanych przypadkach może to prowadzić do pewnych nieścisłości, zaciemnienia sposobu działania kodu, błędów itp. Innymi słowy: jest to zła praktyka.
   
   Tutaj lepszym pomysłem wydaje się bezpośrednie przewidzenie sytuacji, gdzie dana ranga ma dolny bądź górny przedział otwarty. Przez otwarty mam na myśli, plus minus nieskończoność, którą mógłbyś sygnalizować przy użyciu wartości NULL:

   [PHP] pobierz, plaintext 
     static $ranges = array(
       'Początkujący' => array(null, 3999),
       'Doświadczony' => array(4000, 7999),
       'Młodzian'     => array(8000, 11999),
       'Weteran'      => array(12000, null)
     );
   [PHP] pobierz, plaintext 

   Oczywiście późniejszy kod nieco się zmienia.
   
   Może Ci się to wydawać niepotrzebnym komplikowaniem kodu, ale w rzeczywistości sprawia to, że jego działanie jest łatwiejsze do zrozumienia.

[kulczycki]

Bez bicia przyznaje racje

[thek]

Ja, właśnie z racji tego, że musimy uważać na zakresy uważam, że tak naprawdę konieczne do poznania są tylko dolne granice. Weteran będzie nim, niezależnie czy posiada 15tysięcy czy 150tysięcy punktów. Stąd dla niego zakres górny jest zbyteczny i sygnalizowanie błędu mija się z celem. Za to istotne jest zejście poniżej wartości 0, która jest wartością graniczną i w normalnej sytuacji nie powinna wystąpić. Może miec to miejsce w przypadku przekręcenia licznika w bazie. A takie coś już jest na pewno sytuacją nienormalną i powinno być oznaczone jako wyjątek. W końcu własnie po to ten mechanizm powstał. Niektórzy używają wyjątków do zwracania wartości, ale to moim zdaniem zła praktyka. Nie po to powstały elementy w stylu rethrow, który pozwala obsłużyć wyjatek i rzucić go ponownie w kod (ale to już nie w php)