- 1
- 2
- 3
- 4
- 5
- 6
- 7
$price = 'Sold!!!';
if ( $row['mail_sent'] != 1 ) {
$price = 'Price: <span style="color:#bb6600;font-weight:bold;"> '.$row['price'].' </span> Euro';
}
echo '<br />';
printf ('%s',$price);
echo '<br />';
Нашли или выдавили из себя код, который нельзя назвать нормальным, на который без улыбки не взглянешь? Не торопитесь его удалять или рефакторить, — запостите его на говнокод.ру, посмеёмся вместе!
+160
$price = 'Sold!!!';
if ( $row['mail_sent'] != 1 ) {
$price = 'Price: <span style="color:#bb6600;font-weight:bold;"> '.$row['price'].' </span> Euro';
}
echo '<br />';
printf ('%s',$price);
echo '<br />';
Паранойя?
+160
http://channel9.msdn.com/Events/BUILD/BUILD2011/TOOL-835T
Часовая презентация Герб Саттера о будущем C++.
От себя хочется добавить: "Где здесь С++, Герб Саттер?"
+160
window.alert = function(){
events.event(arguments[0],arguments[1],arguments[2],arguments[3],arguments[4]);
}
И это - в продакшене. Писано - для военных
+160
function Hehe ($s) {
$str = $s;
$str = preg_replace ("/\\\/","", $str);
$str = preg_replace ("/'/","'", $str);
$str = preg_replace ("/\n/","<br>", $str);
$str = preg_replace ("/\r/","", $str);
return $str;
}
Название функции оригинальное. Лежала в модуле ~5k строк.
+160
$id = $_POST['id'];
//$id = 194;
$temp = $id;
$str = "";
for ( $incr = 0; $incr < 100; $incr++ ) {
$result = $dbs -> query("SELECT * FROM
market_categs
WHERE
categid={$temp} AND
published = 1 AND
status = 1");
if ( mysql_num_rows($result) > 0 ) {
$categ = mysql_fetch_array($result);
$result2 = $dbs -> query("SELECT * FROM
market_categs_trans
WHERE
categid={$categ[categid]}");
$categ_title = mysql_fetch_array( $result2 );
$str2 = "";
$temp2 = $categ[categid];
for ( $incr2 = 0; $incr2 < 100; $incr2++ ) {
$result3 = $dbs -> query("SELECT * FROM
market_categs
WHERE
categid={$temp2} AND
published = 1 AND
status = 1");
if ( mysql_num_rows($result3) > 0 ) {
$categ2 = mysql_fetch_array($result3);
$str2 .= "/".$categ2[url];
$temp2 = $categ2[parent];
}
}
$str2 = substr($str2,1);
$nb2 = substr_count($str2,"/");
$temp_str2 = explode ("/",$str2);
$str2 = "";
$back = "";
for ($i2 = $nb2; $i2 >= 0; $i2--) {
$str2 .="/".$temp_str2[$nb2];
$nb2--;
}
$str .= " / <a href='/".substr($str2,1)."'>".$categ_title[title]."</a>";
$temp = $categ[parent];
}
}
Рекурсия... мать его
+160
uint64_t search(StringList& who, const string& aName, const string& aSize,TypeModes aTypeMode, SizeModes aSizeMode, const string& aToken, const StringList& aExtList, void* aOwner = NULL)
{
return search(who, aName, Util::toInt64(aSize), aTypeMode, aSizeMode, aToken, aExtList,aOwner);
}
А теперь небольшой _опрос_ :
Каково, по вашему мнению, необходимое и достаточное
качество и количество параметров метода/функции, при котором необходимо(по самым разным причинам) объединять их(параметры) в _меньшее_количество_ , используя встроенные/пользовательские типы данных.
Например: https://github.com/negativ/eiskaltdcpp/blob/master/dcpp/SearchManager.h
+160
HRESULT GetOutlookVersionString(LPSTR* ppszVer, BOOL* pf64Bit)
{
HRESULT hr = E_FAIL;
LPSTR pszTempPath = NULL;
LPSTR pszTempVer = NULL;
TCHAR pszaOutlookQualifiedComponents[][MAX_PATH] = {
TEXT("{1E77DE88-BCAB-4C37-B9E5-073AF52DFD7A}"), // Outlook 2010
TEXT("{24AAE126-0911-478F-A019-07B875EB9996}"), // Outlook 2007
TEXT("{BC174BAD-2F53-4855-A1D5-0D575C19B1EA}") // Outlook 2003
};
int nOutlookQualifiedComponents = _countof(pszaOutlookQualifiedComponents);
int i = 0;
DWORD dwValueBuf = 0;
UINT ret = 0;
*pf64Bit = FALSE;
for (i = 0; i < nOutlookQualifiedComponents; i++)
{
ret = MsiProvideQualifiedComponent(
pszaOutlookQualifiedComponents[i],
TEXT("outlook.x64.exe"),
(DWORD) INSTALLMODE_DEFAULT,
NULL,
&dwValueBuf);
if (ERROR_SUCCESS == ret) break;
}
if (ret != ERROR_SUCCESS)
{
ret = MsiProvideQualifiedComponent(
pszaOutlookQualifiedComponents[i],
TEXT("outlook.exe"),
(DWORD) INSTALLMODE_DEFAULT,
NULL,
&dwValueBuf);
}
else
{
*pf64Bit = TRUE;
}
if (ret == ERROR_SUCCESS)
{
dwValueBuf += 1;
pszTempPath = (LPSTR) malloc(dwValueBuf * sizeof(TCHAR));
if (pszTempPath != NULL)
{
if ((ret = MsiProvideQualifiedComponent(
pszaOutlookQualifiedComponents[i],
TEXT("outlook.exe"),
(DWORD) INSTALLMODE_EXISTING,
pszTempPath,
&dwValueBuf)) != ERROR_SUCCESS)
{
goto Error;
}
pszTempVer = (LPSTR) malloc(MAX_PATH * sizeof(TCHAR));
dwValueBuf = MAX_PATH;
if ((ret = MsiGetFileVersion(pszTempPath,
pszTempVer,
&dwValueBuf,
NULL,
NULL))!= ERROR_SUCCESS)
{
goto Error;
}
*ppszVer = pszTempVer;
pszTempVer = NULL;
hr = S_OK;
}
}
Error:
free(pszTempVer);
free(pszTempPath);
return hr;
}
Говнокод от САМОГО Билли... 21 век на дворе, а у нас в C++ коде goto Error написано (точнее - накакано).
Источник - http://msdn.microsoft.com/en-us/library/dd941331.aspx
+160
function addParam(iId)
{
var sData = '';
//Контрольная работа
if (iId==1)
sData = '';
// Реферат
else if (iId==2)
sData = '';
// Курсовая
else if (iId==3)
sData = '';
// Решение задач
else if (iId==4)
sData = '';
else {
$('#addParam').hide().html('');
return false;
}
$('#addParam').html('').append('<td colspan="2">'+sData+'</td>').show();
}
lot's of code, govnocode, govnocode, govnocode...
+160
typedef _My unsigned int UINT_PTR, *PUINT_PTR;
+160
/**
* Заменяет все вхождения короткого тега <param/> на длиную версию <param></param>
* Заменяет все вхождения короткого тега <embed/> на длиную версию <embed></embed>
*
*/
protected function FlashParamParser($sText) {
if (preg_match_all("@(<\s*param\s*name\s*=\s*\".*\"\s*value\s*=\s*\".*\")\s*/?\s*>(?!</param>)@Ui",$sText,$aMatch)) {
foreach ($aMatch[1] as $key => $str) {
$str_new=$str.'></param>';
$sText=str_replace($aMatch[0][$key],$str_new,$sText);
}
}
if (preg_match_all("@(<\s*embed\s*.*)\s*/?\s*>(?!</embed>)@Ui",$sText,$aMatch)) {
foreach ($aMatch[1] as $key => $str) {
$str_new=$str.'></embed>';
$sText=str_replace($aMatch[0][$key],$str_new,$sText);
}
}
/**
* Удаляем все <param name="wmode" value="*"></param>
*/
if (preg_match_all("@(<param\s.*name=\"wmode\".*>\s*</param>)@Ui",$sText,$aMatch)) {
foreach ($aMatch[1] as $key => $str) {
$sText=str_replace($aMatch[0][$key],'',$sText);
}
}
/**
* А теперь после <object> добавляем <param name="wmode" value="opaque"></param>
* Решение не фантан, но главное работает :)
*/
if (preg_match_all("@(<object\s.*>)@Ui",$sText,$aMatch)) {
foreach ($aMatch[1] as $key => $str) {
$sText=str_replace($aMatch[0][$key],$aMatch[0][$key].'<param name="wmode" value="opaque"></param>',$sText);
}
}
return $sText;
}