Показать сообщение отдельно
  #4 (permalink)  
Старый 25.08.2010, 13:02
Аватар для Riim
Рассеянный профессор
Отправить личное сообщение для Riim Посмотреть профиль Найти все сообщения от Riim
 
Регистрация: 06.04.2009
Сообщений: 2,379

То, что сразу в глаза бросается:

Сообщение от Илюшенций
functiоn InputFileFake()
используя innerHTML можно сильно сократить функцию, она станет заметно понятней.

Сообщение от Илюшенций
functiоn HandleChanges()
{
fileInput = document.getElementById('file_input');
fileType
где var?

Сообщение от Илюшенций
switch (ext.toLowerCase())
{
case 'doc': pos = '0'; break;
case 'bmp': pos = '16'; break;
case 'jpg': pos = '32'; break;
case 'jpeg': pos = '32'; break;
case 'png': pos = '48'; break;
case 'gif': pos = '64'; break;
case 'psd': pos = '80'; break;
case 'mp3': pos = '96'; break;
case 'wav': pos = '96'; break;
case 'ogg': pos = '96'; break;
case 'avi': pos = '112'; break;
case 'wmv': pos = '112'; break;
case 'flv': pos = '112'; break;
case 'pdf': pos = '128'; break;
case 'exe': pos = '144'; break;
case 'txt': pos = '160'; break;
default: pos = '176'; break;
};
я бы сделал через объект, пример:
var obj = {doc: 0, bmp: 16/*, ....*/};
var pos = obj[ext.toLowerCase()];
Ответить с цитированием