- 1
- 2
- 3
- 4
- 5
- 6
- 7
....
elseif(!strcasecmp(substr($value['type'],-4),'form'))
{
$class=$value['type']==='form' ? get_class($this->_form) : Yii::import($value['type']);
$element=new $class($value,null,$this->_form);
}
......
Нашли или выдавили из себя код, который нельзя назвать нормальным, на который без улыбки не взглянешь? Не торопитесь его удалять или рефакторить, — запостите его на говнокод.ру, посмеёмся вместе!
+135
....
elseif(!strcasecmp(substr($value['type'],-4),'form'))
{
$class=$value['type']==='form' ? get_class($this->_form) : Yii::import($value['type']);
$element=new $class($value,null,$this->_form);
}
......
В недрах Yii.
Условие довольно странное ,этож надо было до этого додуматься
+145
status = array();
if ( !empty( $_POST['token'] ) && $_POST['token'] !== 'www.zencoder.pro::mailDrop' ) exit(0);
$email = '[email protected]';
$sender = ( !empty( $_POST['sender'] ) && preg_match('/[^@]+\@[^\.]+\.[a-z]{2,4}/i', $_POST['sender']) ) ? text_check( $_POST['sender'] ) : false;
if( !$email ) {
$status[] = array(
'txt' => 'проверьте поле email',
'field_id' => '#recepient',
);
}
echo json_encode( $status );
неДрисо-ориентированное программирование нарушившее заветы долбоебов, на которых дрису похуй
+147
//вынести куда-нибудь в index.php что-ли... а то используется уже в трёх местах
//a: по всему порталу так, не парься
Комментарий к коду где-то в середине файла.
+12
if ( (_gen_statP==tm_Manual) ||
(_gen_statP==tm_OK) && (_cur_statP!=tm_Manual) ||
(_gen_statP==tm_ValidTM) && (_cur_statP!=tm_Manual) && (_cur_statP!=tm_OK) ||
(_gen_statP==tm_Rejected) && (_cur_statP!=tm_Manual) && (_cur_statP!=tm_OK) && (_cur_statP!=tm_ValidTM) ||
(_gen_statP==tm_Bad) && (_cur_statP!=tm_Manual) && (_cur_statP!=tm_OK) && (_cur_statP!=tm_ValidTM) && (_cur_statP!=tm_Rejected) ||
(_gen_statP==tm_Statistic) && (_cur_statP!=tm_Manual) && (_cur_statP!=tm_OK) && (_cur_statP!=tm_ValidTM) && (_cur_statP!=tm_Rejected) && (_cur_statP!=tm_Bad)
) {
_cur_statP = _gen_statP;
}
if ( (_gen_statQ==tm_Manual) ||
(_gen_statQ==tm_OK) && (_cur_statQ!=tm_Manual) ||
(_gen_statQ==tm_ValidTM) && (_cur_statQ!=tm_Manual) && (_cur_statQ!=tm_OK) ||
(_gen_statQ==tm_Rejected) && (_cur_statQ!=tm_Manual) && (_cur_statQ!=tm_OK) && (_cur_statQ!=tm_ValidTM) ||
(_gen_statQ==tm_Bad) && (_cur_statQ!=tm_Manual) && (_cur_statQ!=tm_OK) && (_cur_statQ!=tm_ValidTM) && (_cur_statQ!=tm_Rejected) ||
(_gen_statQ==tm_Statistic) && (_cur_statQ!=tm_Manual) && (_cur_statQ!=tm_OK) && (_cur_statQ!=tm_ValidTM) && (_cur_statQ!=tm_Rejected) && (_cur_statQ!=tm_Bad)
) {
_cur_statQ = _gen_statQ;
}
Вроде работает, но уж больно некрасиво) по идее вычисляет новый статус в соответствии с "приоритетом"
+155
/**
* Конструктор класса
* @param mixed $db
*/
public function __construct($db = null)
{
if ($db instanceof db) {
self::$_db = & $db;
} // передан объект БД
elseif ($db !== null) {
self::$_db = new db($db);
} // создание по схеме БД
else {
self::$_db = new db();
} // создание по умолчанию
return $this;
}
+149
// Проверка наличия интернета по доступности яндекса и рамблера
public function is_inet($wait=false) {
if ($wait) {
while(!($this->ping('yandex.ru')&&$this->ping('rambler.ru'))) {
echo date('Y/m/d H:i:s >> ').'Internet disconnect, wait...'."<br/>\n";
}
return true;
}
elseif ($this->ping('yandex.ru')&&$this->ping('rambler.ru')) return true;
else return false;
}
+125
<a class="buy_btn" href="javascript://228;" onclick="$.l_basket_modify(this.href.replace(/\D+/g, ''), 1);">
<img src="/images/addToBasket.png" alt="Добавить в корзину">
</a>
Сначала привлёк хинт у ссылки при наведении...
+21
if (Utility::compareVersions(myVersionStr, latestVersionStr) < 0)
{
// New version is available. Download and install the new version.
performUpdate();
}
else
{
bool isUpdateRunning = Utility::isUpdateRunning();
if (!isUpdateRunning)
{
performUpdate();
}
}
Нашел в коде после того как у тестера версия 1.0.0.401 обновилась на 1.0.0.395
−101
<div class="book-info">
<strong class="book">${label}</strong>
{{!if desc && desc != "None None"}}
<span class="author">${desc}</span>
{{!/if}}
</div>
Это кусочек из JQuery template. Ему на вход приходит JSON, сформированный в Python.
Кто бы догадался, что означает это "None None"?
Да, это случай, когда имя и фамилия автора книги неизвестны. Вместо того, чтобы в классе Book сделать нормальный метод getAuthor, чуваки предпочли наговнякать такой вот IF в клиентской части.
+110
private void Save(string ThreadID, string Board)
{
string pathL;
if (cbGIF.Checked)
{
pathL = String.Format(path, Board, ThreadID, "-gif");
}
else
{
pathL = String.Format(path, Board, ThreadID, "");
}
string htmlPath = String.Format(threadPath, Board, ThreadID);
WebClient wcli = new GZipWebClient();
string cThread = wcli.DownloadString(htmlPath);
var rx = new Regex(cbGIF.Checked ? regExGif : regEx);
var ms = rx.Matches(cThread);
imgSaved = 0;
imgCount = ms.Count;
saveProgress.Minimum = 0;
saveProgress.Maximum = imgCount;
saveProgress.Value = 0;
if (!Directory.Exists(pathL))
{
Directory.CreateDirectory(pathL);
}
try
{
foreach (Match m in ms)
{
WebClient ccl = new WebClient();
ccl.DownloadFileCompleted += new AsyncCompletedEventHandler(ccl_DownloadFileCompleted);
string[] v = m.Value.Split('"');
string sd = v[1].Split('/').Last();
string a = url + v[1];
string b = pathL + sd;
if (File.Exists(b))
{
imgSaved++;
saveProgress.Value = imgSaved;
lblSaveProgress.Text = imgSaved.ToString() + "/" + imgCount.ToString();
if (imgSaved == imgCount)
{
btnSave.Enabled = true;
lblSaveProgress.Text = "FUKKEN SAVED!";
}
}
else ccl.DownloadFileAsync(new Uri(a),b);
}
}
catch (WebException e)
{
MessageBox.Show(e.Message + e.StackTrace);
}
}
В пределах одного метода бросаемся из крайности в крайность в именовании переменных.