Нашли или выдавили из себя код, который нельзя назвать нормальным, на который без улыбки не взглянешь? Не торопитесь его удалять или рефакторить, — запостите его на говнокод.ру, посмеёмся вместе!
−161
sub check_license {
my ($license) = @_;
my $is_valid = 0;
if (-f $license) {
open(my $fh, '<', $license);
my $content = do { local $/ = undef; <$fh> };
close($fh);
$content =~ s/\r\n/\n/g;
$is_valid = $content =~ /^-{5}BEGIN LICENSE-{5}$/m;
$is_valid = $content =~ /^-{5}END LICENSE-{5}$/m;
}
return $is_valid;
}Ну ок.
Elvenfighter,
29 Октября 2014
+155
$(function() {
$('#diametr').on('change', function() {
window.location = '/catalog/<?=$arResult['SECTION']['CODE']?>/'+this.value+'/';
});
$('#type_s').on('change', function() {
$('#size_s').find('option:enabled').prop("disabled", true);
$('#size_s').find('option[data-prop-material='+$(this).find('option:selected').data('propMaterial')+']').prop("disabled", false).first().prop("selected", true);
$('#size_s').trigger('change')
});
$('#type_p').on('change', function() {
$('#size_p').find('option:enabled').prop("disabled", true);
$('#size_p').find('option[data-prop-type='+$(this).find('option:selected').data('propType')+']').prop("disabled", false).first().prop("selected", true);
$('#size_p').trigger('change')
});
$(document)
.on('change', '#size_s, #size_p', function() {
$('#selected_prop').val('csone_19_'+$(this).find('option:selected').data('propId'));
})
.on('click', '.but_sita', function() {
console.log($(this).parent().find('select').first());
$(this).parent().find('select').first().trigger('change');
});
$('.but_sita').last().trigger('click');
$('#additional_spoiler').on('click', function(e) {
e.preventDefault();
$('#additional').fadeIn();
});
});
UncaughtException,
29 Октября 2014
+140
virtual QModelIndex parent(const QModelIndex &child) const = 0;
virtual QModelIndex sibling(int row, int column, const QModelIndex &idx) const;
virtual int rowCount(const QModelIndex &parent = QModelIndex()) const = 0;
virtual int columnCount(const QModelIndex &parent = QModelIndex()) const = 0;
virtual bool hasChildren(const QModelIndex &parent = QModelIndex()) const;
virtual QVariant data(const QModelIndex &index, int role = Qt::DisplayRole) const = 0;
virtual bool setData(const QModelIndex &index, const QVariant &value, int role = Qt::EditRole);
virtual QVariant headerData(int section, Qt::Orientation orientation,
int role = Qt::DisplayRole) const;
virtual bool setHeaderData(int section, Qt::Orientation orientation, const QVariant &value,
int role = Qt::EditRole);
virtual QMap<int, QVariant> itemData(const QModelIndex &index) const;
virtual bool setItemData(const QModelIndex &index, const QMap<int, QVariant> &roles);
virtual QStringList mimeTypes() const;
virtual QMimeData *mimeData(const QModelIndexList &indexes) const;
virtual bool canDropMimeData(const QMimeData *data, Qt::DropAction action,
int row, int column, const QModelIndex &parent) const;
virtual bool dropMimeData(const QMimeData *data, Qt::DropAction action,
int row, int column, const QModelIndex &parent);
ну, вы поняли. они написали тысячи говнокода и говорят, что это круто.
они написали говно-пример, и говорят, что это круто.
а вы что думаете по этому поводу?
FadeToBlack,
29 Октября 2014
+145
Connection::Connection(const Options& opts): impl_(new Impl)
{
impl_->isInitialized = false;
impl_->options = std::move(*opts.impl_);
if (!options().lazyInit)
conn();
}Вся соль в строке 4: из объекта opts, переданного по константной ссылке, подло выжимают содержимое. Видимо, никто не пробовал создать два коннекта из одного объекта опций.
roman-kashitsyn,
28 Октября 2014
+158
$element['yandex'] = 14;
unset($element['yandex']);
$element['yandex'] = 60;Разраб упоролся в каштан :)
sautchenkov,
28 Октября 2014
+158
for($i = 0; $i < $db->max; $i++) {
if($i==1) continue;
.....
}Почему бы и нет
UncaughtException,
28 Октября 2014
−99
@login_required
def datadelivery_stats_report(request, campaign_id):
try:
start_date = extract_date_to_default_timezone(request, 'start_date')
except ValidationError:
return HttpResponseServerError("The %s parameter is invalid." % 'start_date')
except AttributeError:
return HttpResponseServerError("The %s parameter is invalid." % 'start_date')
except KeyError:
return HttpResponseServerError("The %s parameter is missing." % 'start_date')
try:
end_date = extract_date_to_default_timezone(request, 'end_date')
except ValidationError:
return HttpResponseServerError("The %s parameter is invalid." % 'end_date')
except AttributeError:
return HttpResponseServerError("The %s parameter is invalid." % 'end_date')
except KeyError:
return HttpResponseServerError("The %s parameter is missing." % 'end_date')Джанга такая джанга... Почему же нельзя выбросить ошибку валидации? 404 можно...
kyzi007,
28 Октября 2014
−124
Результат = Запрос.Выполнить();
Если Результат.Выгрузить().Количество() <> 0 Тогда
Выборка = Результат.Выбрать();
Пока Выборка.Следующий() Цикл
... тут некий код... и внутри контрольный в голову...
Если ТЗ.Количество() <> 0 Тогда
Для Каждого Строка Из ТЗ Цикл
...
КонецЦикла;
КонецЕсли;
...
КонецЦикла;
КонецЕсли;Видимо это такая "оптимизация", другого объяснения подобным проверкам (это не единичный случай) не нашел.
Twilight_Sun,
27 Октября 2014
+88
with dm.Query do
begin
if Active then Close;
//вот так оно работало до ввода параметров
//dm.QStat.SQL.Text := 'select * from T_ORDER where (date between ''' + IntToStr(ys) + '-' + IntToStr(ms) + '-' + IntToStr(ds) + ''' and ''' + IntToStr(yf) + '-' + IntToStr(mf) + '-' + IntToStr(df) + ''')' ;
SQL.Text := 'select * from T_ORDER where (date between :StartDate and :FinishDate)';
Parameters.ParamByName('StartDate').Value := IntToStr(ys) + '-' + IntToStr(ms) + '-' + IntToStr(ds);
Parameters.ParamByName('FinishDate').Value := IntToStr(yf) + '-' + IntToStr(mf) + '-' + IntToStr(df);в комменте - код, ниже - его вменяемая замена
knowy,
27 Октября 2014
+159
if (AllowOperation(array(1,2,3,4,5,9,10,29), array(), array(), array(), -1, -1)) {
} else {
//скрывать определённые поля для дизайнера
$tplCommon->assign("DISPLAY_NONE", "style='display: none;'");
}
В AllowOperation указываются идентификаторы групп пользователей, что забиты в БД.
Таких AllowOperation(array(1,2,3,4,5,9,10,29). .), но с разными идентификаторами во всей системе больше 1000.
pavel87,
27 Октября 2014
Follow us!