View Single Post
Old 01-17-2013, 11:56 AM   PM User | #3
samhor
New Coder

 
Join Date: Mar 2012
Location: USA
Posts: 30
Thanks: 0
Thanked 0 Times in 0 Posts
samhor is an unknown quantity at this point
Question Update

Quote:
Originally Posted by devnull69 View Post
Oh my ... there are several issues in your code

1 - language="Javascript" should be replaced by type="text/javascript"
2 - You should get used to correct indentation of code and related brackets
Code:
var count = 0
var images = new Array('bblogo.jpg','applelogo.jpg','alogo.jpg','iphone45.jpg')

function imageslideshow( btnselection){

 if ( btnselection == 'next' && count <=2){
   count++
 }

 if ( btnselection == 'back' && count >=1){
   count --;
 }
 
 if ( btnselection == 'firstslide' && count >=count+1){
   count 
 
 if ( btnselection == 'lastslide' && count >=count+4){
   count 
 }
 
 document.getElementById("imagecount").innerHTML = (count+1)+" of 4 <br> <img src='images/"+images[count]+"'>"
  
}
Then you can immediately see that you are missing a closing } for one of the if statements

3 - In two of the if statements you just have "count" ... what do you want to do for those if's? You want to SET a certain value for count, no matter what the current value is
Code:
 if ( btnselection == 'firstslide'){
   count=0;
 }
 if ( btnselection == 'lastslide'){
   count=3;
 }
Hi

In regards to 3 -
Code:
 if ( btnselection == 'firstslide' && count >=count+1){
   count 
 
 if ( btnselection == 'lastslide' && count >=count+4){
   count 
 }
I wanted the above to take me to first and last slide as marked in bold. I thought by putting in count shown in red would take the count and take me to the first and last slide respectedly. How would i do it?
samhor is offline   Reply With Quote