Streamline this PHP snippet - php

Question

I know this is a Wordpress based question, but I think the answer is definitely more stand-alone PHP.
On almost every site I do for a client I have to add some social media links, which come from an options page. I currently use this kind of snippet;
$twt = of_get_option('twitter');
$fcb = of_get_option('facebook');
$ins = of_get_option('instagram');
if ($twt) {
echo '<li class="twitter">Twitter</li>';
}
if ($fcb) {
echo '<li class="facebook">Facebook</li>';
}
if ($ins) {
echo '<li class="instagram">Instagram</li>';
}
This is fine if there are only a couple of links, but recently one of my main clients seems to be including every social media link under the sun to their designs so doing it this way can be a bit clunky.
Is there a way I can combine everything into a foreach or something?

Answer 1

Identify commonalities:
echo '<li class="twitter">Twitter</li>';
^^^^^^^^^^^^^^^^^ ^^^^^^^^^^^^^^^^^^^^^ ^^^^^^^^^^^
Identify varying parts:
echo '<li class="twitter">Twitter</li>';
^^^^^^^ ^^^^^^^
Identify dependencies and relationships:
$twt depends on of_get_option(...) whose parameter is the same as the twitter in 2.
The class and the name of the service are dependent on each other. Their relationship is apparently that the name is just the first-letter-uppercased version of the class, but I wouldn't rely on that necessarily.
Unify:
$services = array(
'twitter' => 'Twitter',
'facebook' => 'Facebook'
...
);
foreach ($services as $service => $name) {
if ($url = of_get_option($service)) {
printf('<li class="%s">%s</li>', $service, $url, $name);
// or, if you can't be sure that the variables are safe for HTML interpolation:
// printf('<li class="%s">%s</li>', htmlspecialchars($service), htmlspecialchars($url), htmlspecialchars($name));
}
}

Answer 2

Don't know if you mean exactly that, but check this code:
foreach(array('twitter', 'facebook', 'instagram') as $source)
{
$data = of_get_option($source);
if($data)
{
echo '<li class="'.$source.'">'.ucfirst($source).'</li>';
}
}

Answer 3

You could try something like this:
$social_media = array(
array( 'name' => 'Twitter', 'href' => of_get_option('twitter') ),
array( 'name' => 'Facebook', 'href' => of_get_option('facebook') ),
array( 'name' => 'Instagram', 'href' => of_get_option('instagram') )
);
foreach( $social_media as $s )
echo '<li class="' . strtolower( $s["name"] ) . '">' . $s["name"] . '</li>';
Then simply add the new social media site into the $social_media array.

Answer 4

Try the following:
$social = array('Twitter', 'Facebook', 'Instagram');
for($i=0;$i<length($social); ++$i) {
$twt = of_get_option($social[$i]);
if ($twt) {
echo '<li class="' . strtolower($social[$i]) . '">' . $social[$i]. '</li>';
}
}
You can add additional social networks by adding them to the array. It's a compact solution.