Ver Mensaje Individual
  #7 (permalink)  
Antiguo 29/10/2010, 15:55
Avatar de _cronos2
_cronos2
Colaborador
 
Fecha de Ingreso: junio-2010
Mensajes: 2.062
Antigüedad: 14 años, 7 meses
Puntos: 310
Respuesta: Cambiar Background Dinamicamente

Vamos a ver, voy a copiar tu código y lo analizamos:
Código Javascript:
Ver original
  1. <script type="text/javascript">
  2. function rand(min, max){
  3. var argc = arguments.length;
  4. if(argc == 1){
  5. min = 1;
  6. max = 2147483647;
  7. }else if(argc == 1){
  8. return "Warning: rand() expects exactly 2 parameters, 1 given";
  9. }
  10. return Math.floor((Math.random() * (max - min + 1)) + min);
  11. }
  12.  
  13. function change(){
  14. aleatorio = rand(1, 29);
  15. document.getElementById("back_change").rel = '../../css/backgrounds/' + aleatorio + '.css';
  16. }
  17. </script>
#2: A las funciones en las que el número de argumentos es variable no se les pasa ninguno (function rand()).
#6: ¿ max = 2147483647; ? Entonces no hace falta pasarle argumento, tendrías que llamarla como rand(), pero entonces habría 0 argumentos y no entraría en el if
#7: Si hay 1 argumento [...], si no y hay ¿1? . No tiene sentido.
#14: Si por defecto cuando sólo le pasas un argumento min es 1, ¿para qué le pasas 2?
#15: El rel de un elemento (aquí igual meto la pata xD) se usa para declarar qué relación tiene lo que va a traer ese elemento con el mismo. Ej: <link rel='stylesheet' ... />, porque lo que va a traer el link es una hoja de estilos. Por tanto no tiene sentido usar el rel para modificar la url, usa href porque me imagino que será un link.
Saludos (:
__________________
" Getting older’s not been on my plans
but it’s never late, it’s never late enough for me to stay. "
Cigarettes - Russian Red