- 1
- 2
- 3
- 4
- 5
- 6
- 7
private function fürDieLuLz(event:String, params:StringHash):void
{
if (ExternalInterface.available)
ExternalInterface.call(
"function(){window.fireEvent('" + event +
"',JSON.stringify(" + params + "));}");
}
Нашли или выдавили из себя код, который нельзя назвать нормальным, на который без улыбки не взглянешь? Не торопитесь его удалять или рефакторить, — запостите его на говнокод.ру, посмеёмся вместе!
−241
private function fürDieLuLz(event:String, params:StringHash):void
{
if (ExternalInterface.available)
ExternalInterface.call(
"function(){window.fireEvent('" + event +
"',JSON.stringify(" + params + "));}");
}
Я долго пытался объяснить человеку пишущему JS часть проекта, что не нужно перекодировать объекты каждый раз из JSON и обратно при общении с Флешем, если они все равно проходят через eval(). Но у человека ЧСВ победил все. В итоге пришел "главный инженер" и "решил", что "если мы раньше так делали, то мы будем так делать и дальше, и не волнует". В итоге родился этот шедевр. Я сегодня с утра для поднятия настроения показал этот код JS кодеру. Долго объяснял, что он делает. После чего он сказал, что больше не будет со мной разговаривать :)
Я счастлив
−235
if( end.x % 2 != 0 )
{
//20 строк кода
}
else if( end.x % 2 == 0 )
{
//20 строк кода
}
С того же места, чуток ниже
−104
if( mIsTarget )
{
...
if( !mTargetUnit )
{
panel.isTarget = true;
}
else
{
mTargetUnit.isTarget = true;
}
if( targetUnitId )
{
targetId = targetUnitId;
}
else
{
if( !mTargetUnit )
{
targetId = panel.getId();
}
else
{
targetId = mTargetUnit.getId();
}
}
}
else if( !mIsTarget )
{
if( mUnitManager.getUnitList())
{
for( i = 0; i < mUnitManager.getUnitList().length; i++ )
{
( mUnitManager.getUnit( mUnitManager.getUnitList()[ i ]) as BattleUnit ).hasTarget = false;
( mUnitManager.getUnit( mUnitManager.getUnitList()[ i ]) as BattleUnit ).isTarget = false;
}
}
}
отак от
−99
if (((vo.metricId == 3) && ((columnName == "A/c Balance CCY & Amount") || (columnName == "No. of Trades Impacted") ||
(columnName == "Value of Trades") || (columnName == "Facility CCY & Amount") || (columnName == "MTM/ Net Position") ||
(columnName == "Utilisation CCY & Amt"))) ||
((vo.metricId == 5) && (columnName == "Amount")) ||
((vo.metricId == 5) && (columnName == "Balance in Local")) ||
((vo.metricId == 5) && (columnName == "Balance in EUR")) ||
((vo.metricId == 5) && (columnName == "Sub Balance EUR")) ||
((vo.metricId == 5) && (columnName == "Unsub Balance EUR")) ||
((vo.metricId == 5) && (columnName == "PFE EUR")) ||
((vo.metricId == 5) && (columnName == "Provision EUR")) ||
((vo.metricId == 13) && ((columnName == "EUR Net Cost") || (columnName == "Nominal/ Position"))) ||
((vo.metricId == 14) && ((columnName == "CCY Value") || (columnName == "EUR Value"))) ||
((vo.metricId == 15) && ((columnName == "Value") || (columnName == "EUR Value"))) ||
((vo.metricId == 16) && ((columnName == "Value") || (columnName == "EUR Value"))) ||
((vo.metricId == 17) && ((columnName == "Value") || (columnName == "EUR Value"))) ||
((vo.metricId == 18) && ((columnName == "Value") || (columnName == "EUR Value"))) ||
((vo.metricId == 19) && ((columnName == "Value") || (columnName == "EUR Value"))) ||
((vo.metricId == 20) && ((columnName == "Value") || (columnName == "EUR Value"))) ||
((vo.metricId == 21) && ((columnName == "Value") || (columnName == "EUR Value"))) ||
((vo.metricId == 22) && ((columnName == "Value") || (columnName == "EUR Value")))
Королевский IF!
−106
public class XMLParser extends Sprite
{
private var _xmlData:XML;
private var _xmlLoader:URLLoader;
public function XMLParser() {}
public function loadXML(url:String):void
{
_xmlLoader = new URLLoader();
_xmlLoader.addEventListener(Event.COMPLETE, onXMLLoad);
_xmlLoader.load(new URLRequest(url));
}
private function onXMLLoad(e:Event):void
{
_xmlData = new XML(e.target.data);
parent.dispatchEvent(new Event("LOAD"));
}
public function parseXML():Array
{
var infoData:Array = new Array();
for(var i:int=0; i <= _xmlData.item.length()-1; i++)
infoData.push([ _xmlData.item.attribute("src")[i],
_xmlData.item[i].title,
_xmlData.item[i].image,
_xmlData.item[i].call,
_xmlData.item[i].description,
_xmlData.item[i].place,
_xmlData.item[i].file]);
return infoData;
}
}
XML Parser, ёпт =\
−116
var strLabel:String = (_numCurrentUpload + 1) + "/" + _filesToUpload.length;
strLabel = (_numCurrentUpload + 1 <= _filesToUpload.length && numPerc > 0 && numPerc < 100) ? numPerc + "% - " + strLabel : strLabel;
strLabel = (_numCurrentUpload + 1 == _filesToUpload.length && numPerc == 100) ? "Upload Complete - " + strLabel : strLabel;
strLabel = (_filesToUpload.length == 0) ? "" : strLabel;
Вот так просто и незатейливо обновляется статус загруженного файла :)
−102
if( !mainLoop() == true )
{
return null;
}
Красота, нашлось в проекте от прежних разработчиков
−115
stop();
subt.textColor = 0xFF0000;
subt.text = "Good afternoon!";
sunmain.addEventListener(MouseEvent.CLICK, play_next_afternoon);
sunmain.addEventListener(MouseEvent.MOUSE_OUT, stop_afternoon);
sunmain.addEventListener(MouseEvent.MOUSE_OVER, over_afternoon);
function play_next_afternoon(evt:Event)
{
sunmain.removeEventListener(MouseEvent.CLICK, play_next_afternoon);
sunmain.removeEventListener(MouseEvent.MOUSE_OUT, stop_afternoon);
sunmain.removeEventListener(MouseEvent.MOUSE_OVER, over_afternoon);
sunmain.stop();
subt.textColor = 0xCCCCCC;
subt.text = "";
gotoAndPlay(21);
}
function stop_afternoon(evt:Event)
{
sunmain.stop();
}
function over_afternoon(evt:Event)
{
aft.play();
sunmain.play();
}
Мое, годовой давности. А можно ли было написать это как-то менее говнокодисто?
−116
public class NeaveNetStream extends Sprite
Может не очень смешно, но жизненно. Понять почему / зачем NetStream может наследоваться от Sprite мне не удалось... (но его где-то добавляют в дисплей лист, просто так не переделать).
−105
var cam:Camera;
/*
TODO: camerName here is actually camerIndex - this is what view is sending back, it can send the name and in that case, we will need
to find the index by name
*/
Logger.getInstance().debug("startSetCamera: " + cameraName);
if(!cameraName || cameraName == "")
{
cam = getDefaultCamera();
}
else if(Camera.getCamera(cameraName))
{
cam = Camera.getCamera(cameraName);
}
else
{
cam = null; // А вдруг?!
}
Человек был просто мастером головоломок....