Нашли или выдавили из себя код, который нельзя назвать нормальным, на который без улыбки не взглянешь? Не торопитесь его удалять или рефакторить, — запостите его на говнокод.ру, посмеёмся вместе!
+154
trait TPerson {
public $fname;
public $lname;
public $mname;
// ...
}
class BUser extends EMongoDocument {
public $first_name;
public $last_name;
public $middle_name;
// ...
}
class User extends BUser {
public $firstname;
public $lastname;
// ...
}
Попалось такое наследие на одном рабочем месте. Было два типа юзеров, каждый из них наследовал BUser, в одном из них были firstname и lastname, в другом не были. В монге документы юзера, у которого свои firstname и lastname были вида:
{
_id: ObjectId("..."),
...
first_name: '',
last_name: '',
middle_name: '',
fname: '',
lname: '',
mname: '',
firstname: '',
lastname: ''
}
LubarVV,
26 Апреля 2014
+72
if(error == CABtEngineErrors.K_ERR_BTENGINE_CANCEL) {
mObserver.btEngineEncryptError(CABtEngineErrors.K_ERR_BTENGINE_CANCEL);
} else {
mObserver.btEngineEncryptError(error);
}
busylee,
26 Апреля 2014
+154
$massiv = array(1,2,3,4,5,6,7,8);
$her = 0;
$idRazdelov = array();
$idRoditela = $idrazdelf;
foreach($massiv as $chislo){
$db_list = CIBlockSection::GetList(Array(SORT=>"ASC"), $arFilter = Array("IBLOCK_ID"=>"7", "ID"=>$idRoditela), true);
while($ar_result = $db_list->GetNext()){
if ($her == 0){
if ($ar_result["IBLOCK_SECTION_ID"]){
$idRazdelov[].= $ar_result["ID"];
$idRoditela = $ar_result["IBLOCK_SECTION_ID"];
}else{
$idRazdelov[].= $ar_result["ID"];
$her = 1;
}
}
}
}
Cotard,
25 Апреля 2014
+125
'q<->5.,q<->r<0>3.,q<T>r<0>3.,q<:>r<0>3.,q<:>r<0>3.,r<0>2.' 8!:2<.2014 3 2 3 30 46.034Еще одни даты на J.
Abbath,
22 Апреля 2014
+122
}:;(1,5#3)<@}."_1'--T:: ',.~":,.10000+<.6!:0''Даты. Теперь на J.
Abbath,
22 Апреля 2014
+153
print implode('-', array_reverse(explode('-', trim(substr($project->start_date, 0, count($project->start_date) - 9)))));
Форматирует дату с Y-m-d в d-m-Y. Альтернатива для
date_format(new DateTime($project->start_date), 'd-m-Y');
djumpen,
22 Апреля 2014
+72
if (match) {
formCell(sheet, rowPlus2, 0, 0, fulBorderCalignFont10, 1, false);
formCell(sheet, rowPlus2, 8, 8, fulBorderCalignFont10, 2, false);
formCell(sheet, rowPlus2, 14, 14, fulBorderCalignFont10, 3, false);
formCell(sheet, rowPlus2, 20, 20, fulBorderCalignFont10, 4, false);
formCell(sheet, rowPlus2, 26, 26, fulBorderCalignFont10, 5, false);
formCell(sheet, rowPlus2, 32, 32, fulBorderCalignFont10, 6, false);
formCell(sheet, rowPlus2, 42, 42, fulBorderCalignFont10, 7, false);
formCell(sheet, rowPlus2, 48, 48, fulBorderCalignFont10, 8, false);
formCell(sheet, rowPlus2, 54, 54, fulBorderCalignFont10, 9, false);
formCell(sheet, rowPlus2, 60, 60, fulBorderCalignFont10, 10, false);
formCell(sheet, rowPlus2, 66, 66, fulBorderCalignFont10, 11, false);
formCell(sheet, rowPlus2, 72, 72, fulBorderCalignFont10, 12, false);
} else {
formCell(sheet, rowPlus2, 0, 0, fulBorderCalignFont10, 1, false);
formCell(sheet, rowPlus2, 8, 8, fulBorderCalignFont10, 2, false);
formCell(sheet, rowPlus2, 14, 14, fulBorderCalignFont10, 3, false);
formCell(sheet, rowPlus2, 20, 20, fulBorderCalignFont10, 4, false);
formCell(sheet, rowPlus2, 26, 26, fulBorderCalignFont10, 5, false);
formCell(sheet, rowPlus2, 32, 32, fulBorderCalignFont10, 6, false);
formCell(sheet, rowPlus2, 38, 38, fulBorderCalignFont10, 7, false);
formCell(sheet, rowPlus2, 42, 42, fulBorderCalignFont10, 8, false);
formCell(sheet, rowPlus2, 48, 48, fulBorderCalignFont10, 9, false);
formCell(sheet, rowPlus2, 54, 54, fulBorderCalignFont10, 10, false);
formCell(sheet, rowPlus2, 60, 60, fulBorderCalignFont10, 11, false);
formCell(sheet, rowPlus2, 66, 66, fulBorderCalignFont10, 12, false);
formCell(sheet, rowPlus2, 72, 72, fulBorderCalignFont10, 13, false);
formCell(sheet, rowPlus2, 78, 78, fulBorderCalignFont10, 14, false);
}
evg_ever,
17 Апреля 2014
+157
switch ($vs_TmpStr)
{
case "230";
$vb_isCompl = true;
break;
case "11619";
$vb_isCalcul = true;
break;
case "11660";
$vb_isSevice = true;
break;
case "11668";
$vb_isCalcul = true;
break;
case "11670";
$vb_isCalcul = true;
break;
case "11669";
$vb_isCalcul = true;
break;
case "11678";
$vb_isSevice = true;
break;
case "11679";
$vb_isSevice = true;
break;
case "11681";
$vb_isSevice = true;
break;
...
}125-ти строчный switch.
any0ne2567,
17 Апреля 2014
+152
if (ctype_space($text[$pos] && $pos < $len)) {
while (ctype_space($text[$pos++]) && $pos < $len);
$pos--;
}
Написал я когда-то такое и подумал: а какой смысл в $pos++ в последней итерации, если потом сразу $pos--?
И тут до меня дошло, что
while (ctype_space($text[$pos++]) && $pos < $len);
это не то же самое, что
while (ctype_space($text[$pos]) && $pos < $len) $pos++;
ибо в первом случае $pos++ выполнится в последней итерации, даже если ctype_space возвратит false.
И в итоге заоптимизировал до while (ctype_space($text[$pos]) && $pos < $len) $pos++; (уже без ифа и декремента).
Мораль: нефиг выпендриваться (ставить инкременты и декременты в условия), если можно написать по-простому.
arzeth,
14 Апреля 2014
+154
var setIdToGridElements = function (gridId, gridConfig, idPrefix) {
var gridEl = Ext.get(gridId);
if (!gridEl) {
return;
}
var nodes = gridEl.dom.childNodes; // это массив всех элементов грида
var el;
for (var i = 0; i < nodes.length; i++) {
el = Ext.get(nodes[i].id);
if (el) {
var elements = el.dom.childNodes[i];
if (elements) {
var rowParameters = elements.children; // это массив всех елементов строки
for (var j = 0; j < rowParameters.length; j++) {
var parameterEl = rowParameters[j].childNodes;
var newId = gridEl.id + '-' + gridConfig[0][j].key[1].name.bindTo + '-' + i;
parameterEl[1].className = "gridValue";
for (var k = 0; k < parameterEl.length; k++) {
parameterEl[k].id = newId + '-' + parameterEl[k].className;
}
}
}
}
// Выбираем все строки - элементы грида с ненулевым Id
}
};Авторские комментарии сохранены. Понимаю, что надо править и боюсь
alex123098,
11 Апреля 2014
Follow us!