Question

我有以下代码：

$image_1 = $value->getElementsByTagName("Image1");
$image1  = $image_1->item(0)->nodeValue;

$image_2 = $value->getElementsByTagName("Image2");
$image2  = $image_2->item(0)->nodeValue;

有没有更简单的方法，不重复代码如果我需要$ image_3？

即。我怎么能重构这个？

由于

更新：

我在更多代码中使用$ images_x变量，这也需要重构：

更新2： - 我的完整代码：

$image_1 = $value->getElementsByTagName("Image1");
$image1  = $image_1->item(0)->nodeValue;

$image_2 = $value->getElementsByTagName("Image2");
$image2  = $image_2->item(0)->nodeValue;

$image_3 = $value->getElementsByTagName("Image3");
$image3  = $image_3->item(0)->nodeValue;

$filename_1 = basename($image1);
$ch = curl_init ($image1);
curl_setopt($ch, CURLOPT_HEADER, 0);
curl_setopt($ch, CURLOPT_RETURNTRANSFER, 1);
curl_setopt($ch, CURLOPT_BINARYTRANSFER,1);
$rawdata_1=curl_exec ($ch);
curl_close ($ch);
$fp = fopen(Mage::getBaseDir('media') . DS . 'import/'.$filename_1,'w');
fwrite($fp, $rawdata_1);
fclose($fp);

$filename_2 = basename($image2);
$ch = curl_init ($image2);
curl_setopt($ch, CURLOPT_HEADER, 0);
curl_setopt($ch, CURLOPT_RETURNTRANSFER, 1);
curl_setopt($ch, CURLOPT_BINARYTRANSFER,1);
$rawdata_2=curl_exec ($ch);
curl_close ($ch);
$fp = fopen(Mage::getBaseDir('media') . DS . 'import/'.$filename_2,'w');
fwrite($fp, $rawdata_2);
fclose($fp);

$filename_3 = basename($image3);
$ch = curl_init ($image3);
curl_setopt($ch, CURLOPT_HEADER, 0);
curl_setopt($ch, CURLOPT_RETURNTRANSFER, 1);
curl_setopt($ch, CURLOPT_BINARYTRANSFER,1);
$rawdata_3=curl_exec ($ch);
curl_close ($ch);
$fp = fopen(Mage::getBaseDir('media') . DS . 'import/'.$filename_3,'w');
fwrite($fp, $rawdata_3);
fclose($fp);

$product->addImageToMediaGallery(Mage::getBaseDir('media') . DS . 'import/' .  $filename_1, array('image', 'small_image','thumbnail'), false, false);
$product->addImageToMediaGallery(Mage::getBaseDir('media') . DS . 'import/' .  $filename_2, array('image', 'small_image','thumbnail'), false, false);
$product->addImageToMediaGallery(Mage::getBaseDir('media') . DS . 'import/' .  $filename_3, array('image', 'small_image','thumbnail'), false, false);

Answer 1

你可以使用循环：

$images = array();
for ($i = 1; $i <= 2; $i++) {
   $images[] = $value->getElementsByTagName("Image" . $i)->item(0)->nodeValue;
}

// and then you can get an image via $images[0], $images[1] and so on

所有代码都可以改写为Brad F Jacobs建议：

function downloadAndSave($image) {
    $filename = basename($image);
    $ch = curl_init ($image);
    curl_setopt($ch, CURLOPT_HEADER, 0);
    curl_setopt($ch, CURLOPT_RETURNTRANSFER, 1);
    curl_setopt($ch, CURLOPT_BINARYTRANSFER,1);
    $rawdata=curl_exec ($ch);
    curl_close ($ch);
    $fp = fopen(Mage::getBaseDir('media') . DS . 'import/'.$filename,'w');
    fwrite($fp, $rawdata);
    fclose($fp);
    return $filename;
}
// here you should have another loop, suppose foreach ($products as $product) {
    for ($i = 1; $i <= 2; $i++) {
       $filename = downloadAndSave($value->getElementsByTagName("Image" . $i)->item(0)->nodeValue);
       $product->addImageToMediaGallery(Mage::getBaseDir('media') . DS . 'import/' .  $filename, array('image', 'small_image','thumbnail'), false, false);
    }
// end of foreach }

Answer 2

$images = array();
for ($x = 1; $x <= 3; $x++) {
    $images[] = $value->getElementsByTagName("Image$x")->item(0)->nodeValue;
}

Answer 3

您可以将其捆绑到一个函数中：

$image = getImage("Image1");

function getImage($path)
{
  $image_raw = $value->getElementsByTagName($path);
  return $image_raw->item(0)->nodeValue;
}

此外，您可以从文件中读取资源名称（如果这在您希望使用它的上下文中是合适的），在foreach循环中返回值数组：

伪代码：

$imgArray = array();

foreach($pathNames as $path)
{
   $imgArray[] = getImage($path);
}

Answer 4

使用循环？

$max_ids = 5; // or whatever
$images = array();

for( $i=1; $i <= $max_ids; $i++ ) {
    $images[$i]['tag'] = $value->getElementsByTagName("Image$i");
    $images[$i]['value'] = $image[$i]['tag']->item(0)->nodeValue;
}

还为图像创建数据结构。

Answer 5

最佳使用数组。这将为您提供一系列图像：

for ($i = 1; $i < 3; $i++)
{
  $imageElement = $value->getElementsByTagName("Image".$i);
  $imageElements[] = $imageElement;
  $images[] = $imageElement->item(0)->nodeValue;
}

Answer 6

创建一个组合两行的函数。

PHP - 简单的重构

6 个答案: