[php], function __set() i test sprawdzający przed pozwoleniem na zmiany Opcje

[dawidg87]

Dla jednych to się może wyda dziwne i oczywiste ale jednak ja nie widzę w tym kodzie czegoś co by mogło być nie tak:(

[PHP] pobierz, plaintext 
<?php
 
class mojaklasa
	 {
	   var $atrybut = 6;
	   
		  function __get($nazwa)
		  {
			return $this -> nazwa;
		  }
		  function __set($nazwa, $wartosc)
		  {
			if ( $nazwa = 'atrybut' && $wartosc >= 0 && $wartosc <= 100)
			$this -> atrybut = $wartosc;
		  }
	   
 
 
	 }
	 
$a = new mojaklasa();
 
echo $a -> atrybut;
 
echo "<br>";
 
$a -> atrybut = 101;
 
echo $a -> atrybut;
?>
[PHP] pobierz, plaintext 

Głównie chodzi mi o _set no bo zamieściłem tam instrukcje, a raczej przepisałem gotowy przykład z podręcznika, i według mnie ma to działać na takiej zasadzie, że gdy chcemy zmienić wartość atrybut z poza klasy to instrukcja sprawdza czy nowa właściwość żeczywiście dotyczy tego atrybutu i czy jest liczbą w przedziale 0..100, no i po za klasą zrobiłem sobie mały test tego. Czyli tworze obiekt, później sprawdzam wartość atrybutu, jest to oczywiście 6 tak jak ustawiłem w klasie, no ale zaraz potem zmieniam wartość atrybutu na 101 i znowu sprawdzam wartość atrybutu, i zamiast pokazać błąd to wyświetla atrybut z nową wartością przekraczającą ustawiony zakres :/ no chyba, że to wszystko źle zrozumiałem i to tak ma właśnie działać :/

[qrees]

__set i __get są używane dla atrybutów których nie ma w klasie. Jeżeli klasa ma jakiś atrybut, to jest on po prostu używany.

Poza tym, masz jeszcze jeden błąd: zamiast $nazwa = 'atrybut' powinno być $nazwa == 'atrybut'

[nevt]

[PHP] pobierz, plaintext 
<?php
if ( $nazwa = 'atrybut' && $wartosc >= 0 && $wartosc <= 100)
?>
[PHP] pobierz, plaintext 

ten warunek jest zawsze spełniony bo użyłeś operatora przypisania = zamiast operatora porównania ==, popraw:

[PHP] pobierz, plaintext 
<?php
if (($nazwa == 'atrybut') && ($wartosc >= 0) && ($wartosc <= 100))
?>
[PHP] pobierz, plaintext 

nawiasy dodałem dla podniesienia czytelności.

[dawidg87]

dzięki panowie, ale mam jeszcze jedno pytanko, a chyba szkoda zakładać kolejny temat, no więc mam pewien kod, ale umieszczę tylko wycinki:

[HTML] pobierz, plaintext 
<style>
            <!--
                body {background-color: white}
                h1 {color: white; font-size: 24pt; text-align: center; font-family: arial.sens-serif; front-weight: bold}
                .menu {color: white; font-size: 12pt; text-align: center; font-family: arial.sens-serif; front-weight: bold}
                rd {background: black}
                p {color: black; font-size: 12pt; text-align: justify; font-family: arial.sens-serif)
                p.foot {color: white; font-size: 9pt; text-align: center; font-family: arial.sens-serif; font-weight: bold}
                a:link.a:visited.a:active {color: white}
            -->
            </style>
[HTML] pobierz, plaintext 

i

[HTML] pobierz, plaintext 
<table width="100%" bgcolor="black" cellpading="12" border="0">
              <tr>
                  <td>
                      <p class="foot">&copy TLA Consulting</p>
                      <p class="foot">Prosimy odwiedzić <a href="prawne.php">stronę informacji prawnych</a></p>
                  </td>
              </td>
              </table>
[HTML] pobierz, plaintext 

to jest kod tak samo z podręcznika, mam taki problem, że gdzieś w tym drugim mam błąd bo nie chce mi ustawić tego tak jak mam w <style> opisane, nie wiem już jak to ma być, tak jak wyżej jest w podręczniku i nie jest to dobrze próbowałem już nawet zmieniać <p class="p.foot"> itp. kombinacje, ale nic to nie daje, nie wiem dlaczego :/ strasznie fajny ten podręcznik, same błędy, te kody co skopiowałem są akurat identyczne jak w podręczniku.... pomocy...

[nevt]

trochę nieuważnie przepisujesz... tutaj:

Kod

p {color: black; font-size: 12pt; text-align: justify; font-family: arial.sens-serif)

źle zamknąłeś znacznik - ) zmiast } i zamieniłeś spacje na . powinno być:

Kod

p {color: black; font-size: 12pt; text-align: justify; font-family: arial sens-serif}

natomiast tutaj:

Kod

a:link.a:visited.a:active {color: white}

"posklejałeś" kilka znaczników (gdzieś przepadły ci przecinki i odstępy - zamienione na .), powinno być:

Kod

a:link, a:visited, a:active {color: white}

a na koniec zauwazyłem, że wszędzie jest:

Kod

Font-family: arial.sens-serif ZAMIAST font-family: arial sens-serif

więc kolego - albo nie rozróżniasz separatorów przy przepisywaniu, albo ta książka nadaje sie na podstawkę pod kubek z kawą...