unemployment Posted April 19, 2011 Share Posted April 19, 2011 What is the cleanest way to write this if? <div <?php if (!empty($otherfans)) {echo "class=\"newstext\"";} else {echo "class=\"newstext pts\""; } ?>> Link to comment https://forums.phpfreaks.com/topic/234118-clean-if-statement/ Share on other sites More sharing options...
dcro2 Posted April 19, 2011 Share Posted April 19, 2011 *scratches chin*... <div class="newstext<?php echo empty($otherfans) ? ' pts' : ''; ?>"> This looks better to me for some reason though: <div class="<?php echo empty($otherfans) ? 'newstext pts' : 'newstext'; ?>"> Link to comment https://forums.phpfreaks.com/topic/234118-clean-if-statement/#findComment-1203320 Share on other sites More sharing options...
blacknight Posted April 19, 2011 Share Posted April 19, 2011 thats a good way or <?php echo (!empty($otherfans) ? "class=\"newstext\"" : "class=\"newstext pts\"" ); ?> that way if it is empty you dont get a css class error... Link to comment https://forums.phpfreaks.com/topic/234118-clean-if-statement/#findComment-1203323 Share on other sites More sharing options...
dcro2 Posted April 19, 2011 Share Posted April 19, 2011 thats a good way or <?php echo (!empty($otherfans) ? "class=\"newstext\"" : "class=\"newstext pts\"" ); ?> that way if it is empty you dont get a css class error... Don't know how you'd get a css class error... I think it looks cleaner without all the backslashes.. but to each his own. (Obviously I meant newstext, not newstest ) Link to comment https://forums.phpfreaks.com/topic/234118-clean-if-statement/#findComment-1203326 Share on other sites More sharing options...
Recommended Posts
Archived
This topic is now archived and is closed to further replies.