هل يمكن تحسين / تنظيم هذا المقتطف؟
سؤال
من سؤالي ذي الصلة هنا في ذلك ، توصلت إلى مقتطف PHP التالي:
$url = parse_url($url);
if (is_array($url))
{
$depth = 2;
$length = 50;
if (array_key_exists('host', $url))
{
$result = preg_replace('~^www[.]~i', '', $url['host']);
if (array_key_exists('path', $url))
{
$result .= preg_replace('~/+~', '/', $url['path']); // normalize a bit
}
if (array_key_exists('query', $url))
{
$result .= '?' . $url['query'];
}
if (array_key_exists('fragment', $url))
{
$result .= '#' . $url['fragment'];
}
if (strlen($result) > $length)
{
$result = implode('/', array_slice(explode('/', $result, $depth + 2), 0, $depth + 1)) . '/';
if (strlen($result) > $length)
{
$result = implode('/', array_slice(explode('/', $result, $depth + 1), 0, $depth + 0)) . '/';
}
$result = substr($result, 0, $length) . '...';
}
}
return $result;
}
يبدو كيندا متسللًا ، وخاصة المكررة if (strlen($result) > $length)
كتل الكود. لقد فكرت في الانخفاض parse_url()
إجمالا ، لكني أريد تجاهل مخطط, المستعمل, يمر و ميناء.
أنا أتساءل عما إذا كان بإمكانك يا رفاق الخروج بحل أكثر أناقة / منظمًا له نفس التأثير.
لقد لاحظت للتو ، هناك خطأ - إذا $depth != 2
تتأثر الكتلة التالية:
if (strlen($result) > $length)
{
$result = implode('/', array_slice(explode('/', $result, $depth + 2), 0, $depth + 1)) . '/';
if (strlen($result) > $length)
{
$result = implode('/', array_slice(explode('/', $result, $depth + 1), 0, $depth + 0)) . '/';
}
$result = substr($result, 0, $length) . '...';
}
أعتقد أن أفضل حل هو استخدام حلقة ، سأحاول إصلاح هذا في أسرع وقت ممكن. :س
حلها ، من خلال استبدالها بهذا المقتطف الجديد:
if (strlen($result) > $length)
{
for ($i = $depth; $i > 0; $i--)
{
$result = implode('/', array_slice(explode('/', $result), 0, $i + 1)) . '/';
if (strlen($result) <= $length)
{
break;
}
}
$result = substr($result, 0, $length) . '...';
}
المحلول 2
إليكم النسخة الأقل تشوشًا التي توصلت إليها:
$url = preg_replace('~^www[.]~i', 'http://www.', array_shift($url));
$parts = parse_url($url);
if (is_array($parts))
{
$depth = 2;
$length = 50;
if (array_key_exists('host', $parts))
{
$result = preg_replace('~^www[.]~i', '', $parts['host']);
if (array_key_exists('path', $parts))
{
$result .= preg_replace('~/+~', '/', $parts['path']);
}
if (array_key_exists('query', $parts))
{
$result .= '?' . $parts['query'];
}
if (array_key_exists('fragment', $parts))
{
$result .= '#' . $parts['fragment'];
}
if (strlen($result) > $length)
{
while ($depth > 0)
{
$result = implode('/', array_slice(explode('/', $result), 0, $depth-- + 1)) . '/';
if (strlen($result) <= $length)
{
break;
}
}
$result = substr($result, 0, $length) . '...';
}
return $result;
}
}
نصائح أخرى
للمبتدئين ، يمكن أن يكون لديك:
if (is_array($url))
كرر لجميع الخاص بك === true
عمليات. لماذا تقارن الأنواع؟
لا تنتمي إلى StackOverflow