Inefficient php / javascript code?

Question

I have the following code as shown below.
The code is working but the bottom part seems highly inefficient to me so I was wondering if anyone has any ideas how I can rewrite the code so that its not required to create a

baseUrl + '<?php echo $urls[0]; ?>',

for every row?

(the $urls contain only text like in this way: something-more (so no http: etc))

<?php
    include("includes/connect.php");
    $sql = mysql_query("SELECT url FROM urls ORDER BY listorder ASC");

    $urls = array();

    while ($rw = mysql_fetch_array($sql)) {
        $urls[] = $rw['url'];
    }
?>

<script>
var baseUrl = '/';

var mySwipe = $('#pageBody').responsiveSwipe({
    edition: [
        baseUrl,
        baseUrl + '<?php echo $urls[0]; ?>',
        baseUrl + '<?php echo $urls[1]; ?>',
        baseUrl + '<?php echo $urls[2]; ?>',
        baseUrl + '<?php echo $urls[3]; ?>',
        baseUrl + '<?php echo $urls[4]; ?>',
        baseUrl + '<?php echo $urls[5]; ?>',
        baseUrl + '<?php echo $urls[6]; ?>',
        baseUrl + '<?php echo $urls[7]; ?>',
        baseUrl + '<?php echo $urls[8]; ?>',
        baseUrl + '<?php echo $urls[9]; ?>',
        baseUrl + '<?php echo $urls[10]; ?>'
    ],
    widthGuess: 0,
    emulator: window.location.hash.match(/emulator/)
});

$(document).ready(function(){
    $(document).on('click', '.next', function (e) {
        e.preventDefault();
        mySwipe.gotoNext();
    });
    $(document).on('click', '.prev', function (e) {
        e.preventDefault();
        mySwipe.gotoPrev();
    });
})
</script>

There's a separate site for these types of questions, over at codereview.stackexchange.com. This should probably be moved. — Ivo Coumans
– Ivo Coumans, Commented Sep 8, 2014 at 10:00
what's not efficient about it ? If your array should have baseurl then it should have it. do you want something like factorization ? — Aysennoussi
– Aysennoussi, Commented Sep 8, 2014 at 10:01
@Sekai: What is inefficient about it is that this code is unsafe (PHP to JS values are best json_encode-ed. fetching an array, and passingit on to JS is easily done by writing var someArr = <?= json_encode($urls); ?> job done — Elias Van Ootegem
– Elias Van Ootegem, Commented Sep 8, 2014 at 10:04

Sebastien C. · Accepted Answer · 2014-09-08 10:13:23Z

1

You could use json_encode after generating the array (including baseUrl) server-side :

<?php
    ...

    $baseUrl = '/';

    $urls = array($baseUrl);

    while ($rw = mysql_fetch_array($sql)) {
        $urls[] = $baseUrl.$rw['url'];
    }
?>

<script>
var mySwipe = $('#pageBody').responsiveSwipe({
    edition: <?php echo json_encode($urls); ?>,
    widthGuess: 0,
    emulator: window.location.hash.match(/emulator/)
});

...
</script>

answered Sep 8, 2014 at 10:13

Sebastien C.

4,8612 gold badges24 silver badges33 bronze badges

Sign up to request clarification or add additional context in comments.

Comments

andrew · Accepted Answer · 2014-09-08 10:09:19Z

0

You can do something like this in your php:

$java_urls = '';
foreach ($urls as $url)
{
$java_urls .= "'/".$url."',";
}
// remove last , from the string
if ($java_urls!='') { $java_urls = substr($java_urls, 0, -1); }

Now in your javascript:

var mySwipe = $('#pageBody').responsiveSwipe({
    edition: [
        baseUrl,
        <?php echo $java_urls;?>
    ],
    widthGuess: 0,
    emulator: window.location.hash.match(/emulator/)
});

edited Sep 8, 2014 at 10:09

answered Sep 8, 2014 at 10:03

andrew

2,1182 gold badges27 silver badges33 bronze badges

1 Comment

Cerbrus Over a year ago

You probably want to wrap $url in quotes.

SuperDJ · Accepted Answer · 2014-09-08 10:13:51Z

This:

var mySwipe = $('#pageBody').responsiveSwipe({
    edition: [
        baseUrl,
        baseUrl + '<?php echo $urls[0]; ?>',
        baseUrl + '<?php echo $urls[1]; ?>',
        baseUrl + '<?php echo $urls[2]; ?>',
        baseUrl + '<?php echo $urls[3]; ?>',
        baseUrl + '<?php echo $urls[4]; ?>',
        baseUrl + '<?php echo $urls[5]; ?>',
        baseUrl + '<?php echo $urls[6]; ?>',
        baseUrl + '<?php echo $urls[7]; ?>',
        baseUrl + '<?php echo $urls[8]; ?>',
        baseUrl + '<?php echo $urls[9]; ?>',
        baseUrl + '<?php echo $urls[10]; ?>'
    ],

Could also be something like

var mySwipe = $('#pageBody').responsiveSwipe({
   edition: [
   <?php
   foreach($urls as $url) {
      echo '/'.$url.',';
   }
   ?>
   ]
})

Collectives™ on Stack Overflow

Inefficient php / javascript code?

3 Answers 3

Comments

1 Comment

Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

3 Answers 3

Comments

1 Comment

Comments

Your Answer

Sign up or log in

Post as a guest

Related