StackOverflow2013

Note that there are some explanatory texts on larger screens.

plurals

PONested IF statement PHP
text
Body
copied!<p>after a bit of adivce -I have a rather lengthy chunk of nested if statement, and I'm looking at ways I can refine it and make it more efficient. I did think of a switch - but I can't see how I could split this up.</p> <p>Any thoughts appreciated</p> <pre><code> $portfolioItems [$i] = array(//assign vars into array 'imagePath' => '/sites/jda_redev/'.$myrow_home->path.'/thumbs/thumbs_'.$myrow_home->filename, 'altText' => $myrow_home->alttext, 'description' => $myrow_home->description, 'client' => $extra_client->field_value, 'job' => $extra_job->field_value, 'channel' => $extra_channel->field_value, 'channeltwo' => $extra_channeltwo->field_value, 'channelthree' => $extra_channelthree->field_value, 'channelfour' => $extra_channelfour->field_value, 'channelfive' => $extra_channelfive->field_value, 'sector' => $extra_sector->field_value, 'workerone' => $extra_workerone->field_value, 'workertwo' => $extra_workertwo->field_value, 'gallery' => $galleryName, 'mediaType' => $media_type->field_value, 'videoName' => $video_name->field_value ); if ( $portfolioItems [$i]['imagePath'] == "" ) {$portfolioItems [$i]['imagePath'] = " ";} if ( $portfolioItems [$i]['altText'] == "" ) {$portfolioItems [$i]['altText'] = " ";} if ( $portfolioItems [$i]['description'] == "" ) {$portfolioItems [$i]['description'] = " ";} if ( $portfolioItems [$i]['client'] == "" ) {$portfolioItems [$i]['client'] = " ";} if ( $portfolioItems [$i]['job'] == "" ) {$portfolioItems [$i]['job'] = " ";} if ( $portfolioItems [$i]['channel'] == "" ) {$portfolioItems [$i]['channel'] = " ";} if ( $portfolioItems [$i]['channeltwo'] == "" ) {$portfolioItems [$i]['channeltwo'] = " ";} if ( $portfolioItems [$i]['channelthree'] == "" ) {$portfolioItems [$i]['channelthree'] = " ";} if ( $portfolioItems [$i]['channelfour'] == "" ) {$portfolioItems [$i]['channelfour'] = " ";} if ( $portfolioItems [$i]['channelfive'] == "" ) {$portfolioItems [$i]['channelfive'] = " ";} if ( $portfolioItems [$i]['sector'] == "" ) {$portfolioItems [$i]['sector'] = " ";} if ( $portfolioItems [$i]['workerone'] == "" ) {$portfolioItems [$i]['workerone'] = " ";} if ( $portfolioItems [$i]['workertwo'] == "" ) {$portfolioItems [$i]['workertwo'] = " ";} if ( $portfolioItems [$i]['gallery'] == "" ) {$portfolioItems [$i]['gallery'] = " ";} if ( $portfolioItems [$i]['mediaType'] == "" ) {$portfolioItems [$i]['mediaType'] = " ";} if ( $portfolioItems [$i]['videoName'] == "" ) {$portfolioItems [$i]['videoName'] = " ";} </code></pre>

Querying!

Guidance

An individual column

Larger individual text columns get their own page to allow for proper reading.

SQuiL has stopped working due to an internal error.

If you are curious you may find further information in the browser console, which is accessible through the devtools (F12).

Reload