I have a few lines of jQuery in my web application. This code is inline at the moment because it accepts a couple of PHP variables.
You could always call the function with parameters from PHP:
somefile.js:
function func(params) {
//params is now from PHP
//params.foo == "bar"
}
somefile.php:
<?php
$params = array('foo' => 'bar');
?>
<script type="text/javascript">
$(function() {
func(<?php echo json_encode($params); ?>);
});
</script>
I tend to go with this approach because it avoids global variables while allowing easily transporting variables.
I like to use json_encode also because anything valid JSON is valid JS meaning you don't have to worry about escaping ' or " if you use a string and echo PHP inside of it.
You cannot put it into an external .js file if you still want to include PHP variables. But you can put the code into a separate PHP file that generates valid Javascript output (including content-type in the header set to "text/javascript"!).
In your first PHP file you can refer to the second PHP file generating the Javascript code with
<script src="path/to/generateJS.php" type="text/javascript"></script>
I dont know if a way exists. But if you manage to include variables in external js files, it means that these files will have to be processed as php scripts rendering those variables rather than just rendering them as static files. This will affect performance adversely.
One way you can still use external JS file is declaring a function and passing parameters to it from inline javascript. You code would look like:
<script>
bindPost(<?php echo $post->id; ?>,<?php echo $active_user->id; ?>,'<?php echo $hash; ?>');
</script>
where your bindPost
is declared in external JS file and handles the parameters well.
The way I usually do this is using a few variables in the page itself, then the included JavaScript access those variables.
In your main page:
var post_id=<?php echo $post->id; ?>;
Then in your included JS file:
data: { post_id: post_id,
Without adding a lot more complexity, the easiest way to do this is to have the server treat the Javascript file as a PHP file, so that you can define the PHP variables in that file and then echo them exactly as you're doing in the code in your question.
On apache, that means creating a .htaccess file and adding the follwing line to it:
AddType x-httpd-php .js
(Note that this will process all your javascript files as PHP. If you only want to process some of your Javascript files as PHP you'd need to make a narrower .htaccess rule.)
My solution combines several techniques, some of which are already mentioned within the answers to this question.
First of all: yes, you should separate your JS from PHP. You can put it in a separate file, but you will need to make some changes into your current code. Do not make JS file dynamically generated - it actually kills the advantages of separating JS code into separate file (you cannot cache it etc.).
Next, store your common variables as global variables in the header of HTML file (you do not really have much choice, although there are other alternatives), preferably grouped in one object:
var Globals = <?php echo json_encode(array(
'active_user_id' => $active_user->id,
'token' => $token,
'hash' => $hash,
)); ?>;
Alternatively you can pass all of them as argument(s) to the function you will call, but I assumed you are using them also in other places in your script.
data-
attributesThen use data-
attributes for storing real data associated with containers. You won't need post1
/post2
/post3
-like classes and separate event handlers for them:
<div data-post-id="10">here is something</div>
instead of eg.:
<div class="post10">here is something</div>
data-
attributes from external JS fileAnd then your JavaScript may look like:
$(document).ready(function(){
$('[data-post-id]').click(function() {
var el = $(this);
var data = {
'post_id': el.data('post-id'),
'user_id': Globals.active_user_id
};
data[Globals.token] = Globals.hash;
$.ajax({
'type': 'POST',
'url': 'http://domain.com/ajax/add_love',
'data': data,
'dataType': 'json',
'success': function(response) {
el.html(response.total_loves).toggleClass('loved');
}
});
return false;
});
});
And this should be sufficient (although I did not test it).