Уменьшить строки кода

Question

Я не гуру JavaScript (пока). Я пытаюсь выяснить способ сократить количество строк ниже ... Есть ли какие-либо ярлыки для let say утверждения if?

function showDialog(divID) 
{ 
    var dialogDiv = $("#" + divID); 

    var height = 500; 
    var width = 400; 
    var resizable = false; 

    if (dialogDiv.attr("height") != "") 
    { 
     height = parseInt(dialogDiv.attr("minHeight")); 
    } 

    if (dialogDiv.attr("width") != "") 
    { 
     width = parseInt(dialogDiv.attr("minWidth")); 
    } 

    if (dialogDiv.attr("resizable") != "") 
    { 
     resizable = dialogDiv.attr("resizable"); 
    } 

    dialogDiv.dialog 
    (
     { 
      resizable: resizable, 
      width: width, 
      height: height, 
      bgiframe: true, 
      modal: true, 
      autoOpen: false, 
      show: 'blind' 
     } 
    ) 

    dialogDiv.dialog("open"); 
} 

Answer 1

Вы можете сократить его немного вниз, как это:

function showDialog(divID) 
{ 
    var dialogDiv = $("#" + divID); 

    dialogDiv.dialog({ 
      resizable: dialogDiv.attr('resizable'), 
      width: dialogDiv.attr('width') || 400, 
      height: dialogDiv.attr('height') || 500, 
      bgiframe: true, 
      modal: true, 
      autoOpen: false, 
      show: 'blind' 
    }); 
    dialogDiv.dialog("open"); 
} 

Это имеет преимущество в том, Javascript слабо типизированных , на самом деле это своего рода злоупотребление чертой, но это работает.

Я до сих пор считаю это довольно удобочитаемым, но я привык к синтаксису, если вы не ... решаете, что читаемо, а что-то краткое и что для вас важнее, во многих случаях есть компромисс.

Answer 2

Вы могли бы использовать:

var x = first || default; 

делать инициализацию. Он в основном проверяет, является ли значение first правдоподобным и назначает его x, если оно есть, в противном случае назначить default на x. Пример из кода:

height = dialogDiv.attr('height') || 500; 

заменяет

if (dialogDiv.attr("height") != "") 
{ 
    height = parseInt(dialogDiv.attr("height")); 
} 

Answer 3

Поскольку вы имеете только одно заявление, в каждом случае блок, вы можете избавиться от скобок, окружающих его, но это в основном эстетическим.

Answer 4

Вы имели в виду, что хотите использовать тройные операторы?

Совет: Сценарии написаны для человека, а не для машины. Более важно, чтобы ваш код был легко читаемым и понятным, чем предварительно оптимизированный, а затем вы забыли, что вы там написали.

В любом случае, вот один из способов сократить если заявления:

height = dialogDiv.attr("height") != "" ? parseInt(dialogDiv.attr("height")) : height; 
width = dialogDiv.attr("width") != "" ? parseInt(dialogDiv.attr("width")) : width; 
resizable = dialogDiv.attr("resizable") != "" ? parseInt(dialogDiv.attr("resizable")) : resizable; 

Answer 5

var height = dialogDiv.attr("height") || 500; 
var width = dialogDiv.attr("width") || 400; 
var resizable = dialogDiv.attr("resizable") || false; 

Answer 6

Не делайте этого. Вы хотите, чтобы ваш код JavaScript был максимально читабельным и содержательным. Если вы пытаетесь быстро выполнить время загрузки и быстро сделать свой JavaScript, то что вы должны делать, это создать копию кода JavaScript minified ... не разрушая ваш исходный код JavaScript. Когда дело доходит до минимизации и оптимизации, Google Closure Compiler - лучший инструмент для работы. Чтобы узнать больше о Google Closure Tools (and Compiler Service) и Google Closure Library, см. Ссылки.