- 1
- 2
- 3
- 4
- 5
- 6
- 7
- 8
__inline BOOL IsNumeric(LPTSTR lpsBuf)
{
int i, nSize = _tcslen(lpsBuf);
for (i = 0; i < nSize; i++)
if ((lpsBuf[i] < _T('0')) || (lpsBuf[i] > _T('9')))
return (FALSE);
return (TRUE);
}
Нашли или выдавили из себя код, который нельзя назвать нормальным, на который без улыбки не взглянешь? Не торопитесь его удалять или рефакторить, — запостите его на говнокод.ру, посмеёмся вместе!
+13
__inline BOOL IsNumeric(LPTSTR lpsBuf)
{
int i, nSize = _tcslen(lpsBuf);
for (i = 0; i < nSize; i++)
if ((lpsBuf[i] < _T('0')) || (lpsBuf[i] > _T('9')))
return (FALSE);
return (TRUE);
}
Серъёзный алгоритм.
+135.2
#define DBGLEVEL2 2
#define ER2 if (o.dbglvl>=DBGLEVEL2) fprintf(stderr,
some_fun(int errorcode)
{
ER2 " Error %d", errocode);
}
Использование незакрытых скобок внутри #define
+69.9
public void ejbPassivate() throws EJBException, RemoteException {
return;
}
Истинный китайский девелопер никогда не доверяет компилятору! Ато если ж в функции тело пустое, мало ли, вдруг она никогда не возвратиться. Какой выход? Написать об ентом явно.
+17
std::string sError = "";
try
{
fIn = fopen(cszFileName, "rb");
if (fIn == 0)
{
sError = "Can not open file: ";
sError += cszFileName;
throw std::string("");
}
if (!ParseFile(fIn, pLookup))
{
sError = "Bad file format. File: ";
sError += cszFileName;
throw std::string("");
}
sError = "";
throw std::string("");
}
catch (std::string& e)
{
if (fIn != 0)
{
fclose(fIn);
fIn = NULL;
}
if (sError.length() != 0)
{
throw std::exception(sError.c_str());
}
}
Там же нашел еще более шикарное продолжение гавнокода http://govnokod.ru/1459
+18.3
while(0<1)
{
...
}
В универе встречал такой вот бесконечный цикл
+133.1
/// <summary>
/// General handler for all buttons
/// </summary>
private void FormButtons_Click(object sender, EventArgs e)
{
Control control = (Control) sender;
if (control.Handle == btnCreateInvoices.Handle)
ExportOrders();
else if (control.Handle == btnFirstUsageInvoices.Handle)
ExportFirstUsageInvoices();
else if (control.Handle == btnImportCustomers.Handle)
ImportCustomers();
else if (control.Handle == btnImportProdcuts.Handle)
ImportProducts();
else if // и так далее...
}
WinForms приложение, на все кнопки навешен 1 обработчик события OnClick.
А внутри вот....
+154
$max_b = 0;
for($i=1;$i<=40;$i++){
$max_b +=(41-$i);
}
Нашел у товарища в коде, данный кусочек вызывался в цикле порядка десятка тысяч раз
+168.2
if ($obj->id == 17718 || $obj->id == 16579 || $obj->id == 16849 || $obj->id == 16527 || $obj->id == 16866 || $obj->id == 16864 || $obj->id == 16863 || $obj->id == 16549 || $obj->id == 16809 || $obj->id == 16155 || $obj->id == 16852 || $obj->id == 16808 || $obj->id == 17750 || $obj->id == 17754 || $obj->id == 16993 || $obj->id == 16720 || $obj->id == 16594 || $obj->id == 16595 || $obj->id == 16703 || $obj->id == 16704 || $obj->id == 16806 || $obj->id == 16807 || $obj->id == 16712 || $obj->id == 16716 || $obj->id == 16722 || $obj->id == 16830 || $obj->id == 16831 || $obj->id == 16867 || $obj->id == 17412 || $obj->id == 16690 || $obj->id == 16039 || $obj->id == 16797 || $obj->id == 16749 || $obj->id == 16829 || $obj->id == 16571 || $obj->id == 16592 || $obj->id == 16136 || $obj->id == 17307 || $obj->id == 16846 || $obj->id == 17308 || $obj->id == 16577 || $obj->id == 16630 || $obj->id == 17405 || $obj->id == 17403 || $obj->id == 16042 || $obj->id == 16131 || $obj->id == 16631 || $obj->id == 16139 || $obj->id == 16575 || $obj->id == 16576 || $obj->id == 16574 || $obj->id == 16573 || $obj->id == 16535 || $obj->id == 16536 || $obj->id == 16570 || $obj->id == 16528 || $obj->id == 16526 || $obj->id == 17409 || $obj->id == 16622 || $obj->id == 16623 || $obj->id == 16619 || $obj->id == 16620 || $obj->id == 16805 || $obj->id == 16151 || $obj->id == 16804 || $obj->id == 16026 || $obj->id == 16613 || $obj->id == 16681 || $obj->id == 16682 || $obj->id == 17414 || $obj->id == 16618 || $obj->id == 16626 || $obj->id == 16627 || $obj->id == 17413 || $obj->id == 16628 || $obj->id == 16629 || $obj->id == 16593 || $obj->id == 16165 || $obj->id == 16166 || $obj->id == 17404 || $obj->id == 16168 || $obj->id == 16174 || $obj->id == 17402 || $obj->id == 16140 || $obj->id == 16624 || $obj->id == 16625 || $obj->id == 16621 || $obj->id == 16832 || $obj->id == 16584 || $obj->id == 16718 || $obj->id == 16719 || $obj->id == 16530 || $obj->id == 16847 || $obj->id == 16585 || $obj->id == 16756 || $obj->id == 16825 || $obj->id == 16823 || $obj->id == 16824 || $obj->id == 17783 || $obj->id == 17787 || $obj->id == 17788 || $obj->id == 17782 || $obj->id == 17789 || $obj->id == 17790 || $obj->id == 17784 || $obj->id == 17791 || $obj->id == 17792 || $obj->id == 17785 || $obj->id == 17793 || $obj->id == 17794) continue;
Кусок скрипта генерации каталога на CD. Пропуск позиций, которые надо исключить из каталога.
+142.9
public Аккаунт НайтиУчётнуюЗаписьПользователя(string логин, string пароль)
{
lock (_пользователи)
return _пользователи.Find(пользователь => пользователь.НеТыЛиЭто(логин, пароль));
}
+68.9
public char getChar() throws ErrorLexAn {
delSpace();
int simb = p.bufStr[p.posStr] & 0xFF;
return charl = simb < 128 ? (char) simb :
(new String(new byte[]{
convert == false ? (byte) simb : arrayConv[simb]})).charAt(0);
}
Читает следующий символ из буфера (хранятся в байтах) и конвертит по таблице.
Я сам джаву плохо знаю, но уверен, что byte -> char можно сконвертить как-то проще.