- 1
http://<api-host>/admin/questions?utf8=✓&skill_id=71
Нашли или выдавили из себя код, который нельзя назвать нормальным, на который без улыбки не взглянешь? Не торопитесь его удалять или рефакторить, — запостите его на говнокод.ру, посмеёмся вместе!
+137
http://<api-host>/admin/questions?utf8=✓&skill_id=71
сервер: сер, как у вас с utf8?
клиент: все ОК
−153
var regs:Array;
if ( USE_NEW_SYNTAX )
regs = line.match( /vc\[([vif][acost]?)(\d*)?(\.[xyzwrgba](\+\d{1,3})?)?\](\.[xyzwrgba]{1,4})?|([vif][acost]?)(\d*)?(\.[xyzwrgba]{1,4})?/gi );
else
regs = line.match( /vc\[([vof][actps]?)(\d*)?(\.[xyzwrgba](\+\d{1,3})?)?\](\.[xyzwrgba]{1,4})?|([vof][actps]?)(\d*)?(\.[xyzwrgba]{1,4})?/gi );
игра найди 10 отличий от Adobe :)
+74
# Определение IP адреса
static public function ip() {
# Список возможных заголовков для получения ip адреса
$Sip = array();
$Sip['REMOTE_ADDR'] = getenv('REMOTE_ADDR');
$Sip['HTTP_FORWARDED_FOR'] = getenv('HTTP_FORWARDED_FOR');
$Sip['HTTP_X_FORWARDED_FOR'] = getenv('HTTP_X_FORWARDED_FOR');
$Sip['HTTP_X_COMING_FROM'] = getenv('HTTP_X_COMING_FROM');
$Sip['HTTP_VIA'] = getenv('HTTP_VIA');
$Sip['HTTP_XROXY_CONNECTION'] = getenv('HTTP_XROXY_CONNECTION');
$Sip['HTTP_CLIENT_IP'] = getenv('HTTP_CLIENT_IP');
$Sip['Unknow'] = '0.0.0.0';
# Выбор найденного ip адреса
if(isset($Sip['REMOTE_ADDR'])) {
$Rip = $Sip['REMOTE_ADDR'];
} else if(isset($Sip['HTTP_FORWARDED_FOR'])) {
$Rip = $Sip['HTTP_FORWARDED_FOR'];
} else if(isset($Sip['HTTP_X_FORWARDED_FOR'])) {
$Rip = $Sip['HTTP_X_FORWARDED_FOR'];
} else if(isset($Sip['HTTP_X_COMING_FROM'])) {
$Rip = $Sip['HTTP_X_COMING_FROM'];
} else if(isset($Sip['HTTP_VIA'])) {
$Rip = $Sip['HTTP_VIA'];
} else if(isset($Sip['HTTP_XROXY_CONNECTION'])) {
$Rip = $Sip['HTTP_XROXY_CONNECTION'];
} else if(isset($Sip['HTTP_CLIENT_IP'])) {
$Rip = $Sip['HTTP_CLIENT_IP'];
} else {
$Rip = '0.0.0.0';
}
# Валидация ip адреса
# Получение из секций массив
$ARip = explode( '.', $Rip );
# Пересборка ip адреса
$RARip = array(0 => NULL, 1 => NULL, 2 => NULL, 3 => NULL);
for($n = 0; $n < 10; $n++) {
$RARip[0] .= isset( $ARip[0]{$n} ) ? $ARip[0]{$n} : NULL;
$RARip[1] .= isset( $ARip[1]{$n} ) ? $ARip[1]{$n} : NULL;
$RARip[2] .= isset( $ARip[2]{$n} ) ? $ARip[2]{$n} : NULL;
$RARip[3] .= isset( $ARip[3]{$n} ) ? $ARip[3]{$n} : NULL;
}
$BRip = substr( (int) $RARip[0], 0, 3 ) . '.' . substr( (int) $RARip[1], 0, 3 ) . '.' . substr( (int) $RARip[2], 0, 3 ) . '.' . substr( (int) $RARip[3], 0, 3 );
# Возвращение пересобранного ip адреса
return $BRip;
}
Не столько — «Г», сколько — «Извращение»...
Улыбнуло имя переменной — $Rip;
А раньше этот код был суровее — exit( $string ); в случаи не соответствия...
+93
for (int i = 0; i < wastePeriods.Length; i++)
{
wastePeriods = 0;
docPeriods = 0;
wastePeriodsYear = 0;
}
Диггеры-рефакторщики раскопали очередной оригинальный цикл.
wastePeriods - массив с over300+ записями.
docPeriods, wastePeriodsYear - тоже массивы
+69
function all_sorting(
$ids, //!< Массив ID элемента в карте сайта
$_POST //!< Значения сортировки элементов
) {
Определение локальной переменной с именем суперглобального массива вызывает ошибку "Fatal error: Cannot re-assign auto-global variable _POST in" в PHP 5.4
−146
public function FooClass() {
if (!_isConstructing) throw new Error("Singleton, use FooClass.instance");
}
public static function get instance():FooClass {
if (_instance == null) {
_isConstructing = true;
_instance = new FooClass();
_isConstructing = false;
}
return _instance;
}
Новое слово в создании синглтонов.
+96
string AutoWait;
int AutoStop;
/*...*/
RegistryKey sKey = Registry.LocalMachine.OpenSubKey("Platform\\Settings");
AutoStop=(int)Key.GetValue("AutoStop");
if (AutoStop == 1) { AutoWait = "yes"; wait.Enabled = false; } else { AutoWait = "no"; }
/*...*/
if ((km4<=2)&(onTheWay)&(AutoWait=="yes"))
{
Waiting=true;
label3.BackColor = Color.Red;
label4.BackColor = Color.Black;
}else
if ((km4 > 2) & (onTheWay))
/*...*/
Классика жанра - использование переменной String в качестве bool. Про ООП автор вообще не слышал, весь код оформлен одной большой портянкой с кучей повторений сброса/установки подобных "флажков".
+73
public static function ParseContainer($s,$be,$en,$cb,$ret_beg=false,$re=false)
{
if(!is_callable($cb))
return$s;
$bl=strlen($be);
$el=strlen($en);
for(;;)
{
if(false===$bp=strpos($s,$be) or false===$ep=strpos($s,$en,$bp+1+$bl))
break;
$brp=strrpos(substr($s,0,$ep-$bp+1),$be);
if($brp>$bp)
$bp=$brp;
$ns=substr($s,$bp+($ret_beg ? 0 : $bl),$ep-$bp-($ret_beg ? 0 : $bl)+($re ? $el : 0));
$ns=call_user_func($cb,$ns);
$s=substr_replace($s,$ns,$bp,$ep-$bp+$el);
}
return$s;
}
−116
select cunt(*) from users
Опечаточка по фрейду :)
−122
my (@data) = $content =~ m#href="/show/\d+.*?">(.*?)</a>.*?(Ep: \d+).*?(at \d{2}:\d{2}).*?</small>#gsi;
for (my $i; $i < @data;){
$data .= $data [$i++].' '.$data [$i++].' '.$data [$i++];
$data .= "\n" unless $i == @data;
}
Распарсил.