Problem s javascript skriptom

Što je ovdje pogrešno? 

// IMG Popup

if (oButtonProperties.sCode == 'img')

{

// Ask them to enter the URL of the image.

var sText = prompt(oEditorStrings['prompt_text_img'], 'http://');

if (!sText)

return;

// Ask them to enter the height of the image.

var sHeight = prompt(oEditorStrings['prompt_text_height']);

if (!sHeight || sHeight == '')

sHeight = sText; 

// Ask them to enter the width of the image. 

var sWidth = prompt(oEditorStrings['prompt_text_width']);

if (!sWidth || sWidth == '')

sWidth = sText; 

// Ask them to enter the alt text of the image. 

var sAlt = prompt(oEditorStrings['prompt_text_alt']);

if (!Alt || sAlt == '')

sAlt = sText;

var bbcode = '[img height=' + sHeight + 'width=' + sWidth + 'alt=' + sAlt + ']' + sText + '[/img]';

replaceText(bbcode.replace(/\\n/g, '\n'), this.oTextHandle);

}

// Replace?

else if (!('sAfter' in oButtonProperties) || oButtonProperties.sAfter == null)

To je samo dio koda. Mene zanima što ne valja u tom dijelu. 

Odokativno, trebalo bi to malo refaktorirati.. Npr. sve varijable koje počinju sa 's' bi mogle biti jedan objekt kojeg se onda može koristit u raznim primjenama tipa "generiraj bbcode". Također, provjeru parametara prebaciti u jednu funkciju jer se svi parametri provjeravaju na jednak način. Varijabla 'Alt' vjerojatno je 'sAlt'.

Dobije se nešt poput ovog onda:

var s = {};// IMG Popupif (oButtonProperties.sCode == 'img') {  // Ask them to enter the URL of the image.  s.Text = prompt(oEditorStrings['prompt_text_img'], 'http://');  if (empty(s.Text)) {    return;  }  // Ask them to enter the height of the image.  s.Height = prompt(oEditorStrings['prompt_text_height']);  if (empty(s.Height)) {    s.Height = s.Text;   }  // Ask them to enter the width of the image.   s.Width = prompt(oEditorStrings['prompt_text_width']);  if (empty(s.Width)) {    s.Width = s.Text;   }  // Ask them to enter the alt text of the image.   s.Alt = prompt(oEditorStrings['prompt_text_alt']);  if (empty(s.Alt)) {    s.Alt = s.Text;  }  var bbcode = gen_bbcode(s);  replaceText(bbcode.replace(/\\n/g, '\n'), this.oTextHandle);}// Replace?else if (!('sAfter' in oButtonProperties) || oButtonProperties.sAfter == null) {}

function gen_bbcode (s) {  return '[img height=' + s.Height + 'width=' + s.Width + 'alt=' + s.Alt + ']' + s.Text + '[/img]';}

function empty (item) {  if (!item || (item === "")) {    return true;  }  else {    return false;  }}
Dalo bi se još po tome pošarafit, al mislim da je ovo dobar početak.