Path: csiph.com!news.mixmin.net!border2.nntp.ams1.giganews.com!nntp.giganews.com!peer03.ams1!peer.ams1.xlned.com!news.xlned.com!peer01.am4!peer.am4.highwinds-media.com!peer01.fr7!futter-mich.highwinds-media.com!news.highwinds-media.com!fx18.fr7.POSTED!not-for-mail Subject: Re: shorthand? Newsgroups: dk.edb.internet.webdesign.serverside.php References: <5a96eb42$0$710$14726298@news.sunsite.dk> From: Kim Ludvigsen User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:52.0) Gecko/20100101 Thunderbird/52.6.0 MIME-Version: 1.0 In-Reply-To: <5a96eb42$0$710$14726298@news.sunsite.dk> Content-Type: text/plain; charset=utf-8; format=flowed Content-Language: da Content-Transfer-Encoding: 8bit Lines: 31 Message-ID: X-Complaints-To: abuse@newsdemon.com NNTP-Posting-Date: Wed, 28 Feb 2018 19:32:58 UTC Organization: http://www.NewsDemon.com Date: Thu, 1 Mar 2018 02:32:57 +0700 X-Received-Bytes: 1367 X-Received-Body-CRC: 2494784304 X-Original-Bytes: 1191 Xref: csiph.com dk.edb.internet.webdesign.serverside.php:7077 Den 01-03-2018 kl. 00:47 skrev Dennis Munding: > Kan ovenstående kode optimeres yderligere? > Synes det virker tåbeligt med 3x if statements lige efter hinanden... Måske noget array-noget? Ellers switch/case: switch ($status) { case: 'Inaktiv': $class = "farvel"; break; case 'Reservist': $class = "reservist"; break case 'Pause': $class = "pause"; break; // Hvis der kun er de tre muligheder, så kan den sidste udskiftes med: default: $class = "pause"; } -- Mvh. Kim Ludvigsen