Php JQuery循环搜索结果中的javascript。坏主意？

我经常在一堆不同的东西中使用JQuery循环幻灯片。头脑里总是一样的：

<script type="text/javascript">
  $(document).ready(function() {
    $('.slideshow').cycle({
      fx: 'fade',
      etc...'
    });
  });
</script>

$（文档）.ready（函数（）{
$（'.slideshow'）。循环({
外汇：“淡出”，
等等……”
});
});

然后，身体中的图片：

<div class='slideshow'>
  <img src='photo1.jpg' width height />
  <img src='photo1.jpg' width height />
  <img src='photo1.jpg' width height />
  etc...
</div>

等

不过这一次，我正在为供应商构建一个搜索引擎。返回的每个供应商都需要在其显示框中进行小幻灯片放映。我使用vendor_id来保持slideshow类的唯一性，但我必须将javascript向下移动到主体中，这实际上是我查询结果的一部分：

while($rst = mysql_fetch_array( SQL is happening... )) {
print("
  <script type='text/javascript'>
  $(document).ready(function() {
    $('.s$rst[vendor_id]').cycle({ 
    fx:     'fade', 
    etc...
    });
  });
  </script> 
  <div class='s$rst[vendor_id]'>
    <img src='$rst[prime_image]' width height />
    <img src='$rst[second_image]' width height />
    <img src='$rst[third_image]' width height />
    etc...
  </div>
");}

while（$rst=mysql\u fetch\u数组（SQL正在发生…）{
打印（“
$（文档）.ready（函数（）{
$（'.s$rst[vendor_id]'）。周期（{
外汇：“淡出”，
等
});
});
等
");}

效果很好，在我扣上按钮并开始其他操作之前，有人能想到这个方法的问题吗？烧掉不必要的记忆之类的

谢谢，

---

J

为什么不将对

cycle（）

的调用包装到函数中

您可以保存count（$rst）乘以strlen（）-strlen（functionname+parameters），这样看起来会更干净。 您甚至可以将cycle（）保留在标题中。。或者更好。。在专用的file.js中。（如果您愿意，它甚至可能包含一个jquery插件来包装调用！）

while（$rst=mysql\u fetch\u数组（SQL正在发生…）{
打印（“
$（文档）.ready（函数（）{
$（'.s$rst[vendor_id]'）。cyclecaller（cycleoptions）；
});
......

while($rst = mysql_fetch_array( SQL is happening... )) {
print("
  <script type='text/javascript'>
  $(document).ready(function() {
    $('.s$rst[vendor_id]').cyclecaller(cycleoptions);
  });
......