Let me tell you something. You don't have to say anything, you know why? Cause you can pick up all your stuff, because you're mother-fucking fired! | Lemeno.se - En blogg om att Tjäna Pengar På Internet | Min blogg om styrketräning och kost
Programmeringsdesign, PHP
Visa signatur
Det är alltid svårt att uttrycka sig kärnfullt men ändå klart när det gäller programmering tycker jag.
Egentligen är det jag försöker komma undan en följande konstruktion med upprepade if-satser:
class Ad {
public $adState = 'active';
}
class AdWriter {
public function displayAd(Ad $ad, $viewmode = 'visitor') {
if ($viewmode =='visitor') {
if ($adState == 'active') {}
else if ($adState == 'paused') {}
else if ($adState == 'inactive') {}
}
else if ($viewmode = 'owner') {
if ($adState == 'active') {}
else if ($adState == 'paused') {}
else if ($adState == 'inactive') {}
}
else if ($viewmode == 'administrator') {
if ($adState == 'active') {}
else if ($adState == 'paused') {}
else if ($adState == 'inactive') {}
}
}
}
Går det att komma undan på något sätt? Visserligen visar alla varianterna av if-satserna olika saker, men som tidigare nämnt känns det som att det går att lösa på ett snyggare sätt.
Visa signatur
Let me tell you something. You don't have to say anything, you know why? Cause you can pick up all your stuff, because you're mother-fucking fired! | Lemeno.se - En blogg om att Tjäna Pengar På Internet | Min blogg om styrketräning och kost
Citera flera
Citera
Här är ett exempel på hur du undviker upprepade if/else. Tycker switch blir lite tydligare i exemplet men den fungerar egentligen som if/else så det är inte direkt någon superfördel.
class ad{
const active = 1;
const paused = 2;
const inactive = 3;
const visitor = 1;
const owner = 2;
const administrator = 3;
public $adState = self::active;
}
class AdWrite{
function displayAd(Ad $ad, $viewmode = ad::visitor)
{
switch($viewmode){
case ad::visitor:
$this->view_visitor_stuff($ad);
break;
case ad::owner;
$this->view_owner_stuff($ad);
break;
case ad::administrator;
$this->view_admin_stuff($ad);
break;
default:
throw new exception("Incorrect viewmode");
break;
}
$this->common_ad_view($ad);
}
function view_administrator_stuff(Ad $ad){
/* Ex. en header med lite länkar till "edit ad", "remove ad", "view users other ads", osv. */
echo $ad->some_adminspecific_options;
}
function view_owner_stuff(Ad $ad){
/* Ex. lite "view other ads I own", "edit this ad", etc. */
echo $ad->owner_stuff;
}
function view_visitor_stuff(Ad $ad){
/* Kanske inte så mycket som en besökare ser som ingen annan ser... */
}
function common_ad_view(Ad $ad){
/* Relevant för alla, ex. hur reklamen ser/såg ut */
echo $ad->stuff_common_for_all_ads;
}
}
Men det verkar ju som att du har massa typiska if/else-grejer så det kan bli svårt att undvika. Du måste förr eller senare handskas med "Om det är en besökare, visa det här. Om det är admin, visa det här!" Om du väljer att göra det med olika funktioner som i mitt exempel, eller i en view-funktion typ:
function view(Ad $ad, $viewmode){
if($viewmode == ad::administrator){
echo "Stäng av användarens reklam";
}elseif($viewmode == ad::owner){
echo "Ta bort reklam";
}elseif($viewmode == ad::visitor){
/* Kommer inte på något exempel */
}
echo $ad->some_ad_property;
}
Är nog lite upp till hur du tycker känns enklast. En stor view-funktion blir snabbt krånglig att underhålla.
Ett annat alternativ är att ex. ge "Ad" en egen "access-mode" och __get funktion, ex:
class ad{
/* Samma consts som ovan */
private $access;
private $adproperties;
function set_access($new_access){
switch($new_access){
case self::administrator:
$this->access = self::administrator;
break;
/* Samma som ovan och exception på default */
}
}
function __get($name){
$owner_properties = array("some_owner_key");
$admin_properties = array("ad_database_id", "other_admin_keys");
if(array_key_exists($name, $this->adproperties){
if(in_array($name, $owner_properties))
{
if($this->access === self::owner || $this->access === self::administrator)
{
return $this->adproperties[$name];
}else{
/* Exception eller false, beroende på hur du väljer att använda funktionen kanske */
throw new exception("Insufficient rights");
}
}
elseif(in_array($name, $admin_properties)){
if($this->access === self::administrator)
{
return $this->adproperties[$name];
}/*Else some ovan */
}
return $this->adproperteis[$name];
}else{
throw new exception("Incorrect name");
}
}
}
Funktionen ovan går bra med en gemensam view_ad funktion i AdWrite för då kan du jämföra med false eller köra try/catch-block så går den igenom allt automatiskt.
Abstraction ftw, fast nu ska jag inte flumma mer!
Visa signatur
Cat funeral! Cat funeral!
>>> 112383 <<<
Citera flera
Citera
Hårdvara
Mjukvara
Datorkomponenter
Ljud, bild och kommunikation
- RT-AC51U slö på 2.4GHz3
- EA-chef vill se mer reklam i spel igen70
- Asus ville ha 30 000 för trasig kontakt på 409031
- 3D skrivare tjänst. Helst i Stockholm område2
- Garantiärende hos Webhallen slutar med att jag blir betalningsskyldig96
- PizzaClockers - pizzans hemligheter!1414
- Bästa komponenterna enligt SweClockers medlemmar54
- Har Telia tagit bort inspelningsfunktionen på Arris?2
- Köpa gamingdator 25-30k budget. Sälja nuvarande dator?5
- Elcykel – samlingstråd315
- Säljes Acer Nitro G550 Gamingprojektor - 4K | 120hz | 8.3ms | HDR + 110" projektorduk
- Säljes MSI Optix AG32CQ - 31,5" | 144hz | 1440p | 2560x1440
- Säljes Vårstädning av Tangentbord, vattenkylning, RAM
- Säljes EVGA GeForce GTX 1060 6GB SC Gaming
- Säljes Helt nya Xtrfy, SteelSeries tangentbord & musmattor
- Säljes Asus RTX 4070 Dual OC
- Säljes Sennheiser GSP 370
- Skänkes Logitech Harmony + Telldus fjärrströmbrytare
- Säljes Beyerdynamic DT 700 Pro X
- Säljes i7-8700K, RTX 2080, 32GB 3200MHz DDR4, 1TB+500GB SSD, EKWB custom loop
- Asus ville ha 30 000 för trasig kontakt på 409031
- Hemmabygge slår samman 256 RISC‑V‑kretsar21
- Microsofts spelbutik för mobiler lanseras i sommar9
- Fullproppat uppdateringspaket på gång till Steam Deck16
- Krönika: Tanken av att prata med AI i spel gör mig deprimerad49
- Stack Overflow-användare ilsknar till efter Open AI-affär20
- Windows skärmdumpverktyg får QR‑kodläsare3
- Helgsnack: Kommer AMD kunna slå Nvidia igen?98
- Microsoft jobbar på ny widget-funktion i Start‑menyn26
- "Långsiktighet" är orsaken till Microsofts nedskärningar24