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\""; } ?>> Quote 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'; ?>"> Quote 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... Quote 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 ) Quote Link to comment https://forums.phpfreaks.com/topic/234118-clean-if-statement/#findComment-1203326 Share on other sites More sharing options...
Recommended Posts
Join the conversation
You can post now and register later. If you have an account, sign in now to post with your account.