Date: 04/09/07     Keywords: no keywords

Втф?

string snumber1 = txtStampNumber1.Text.Trim();
string snumber2 = txtStampNumber2.Text.Trim();

if (!CheckStampRange(snumber1, snumber2))
{
//....
}


////

bool CheckStampRange(string snumber1, string snumber2)
{
//Далее соответственно в txtStampNumber1 в фунции не встречаеться :/

if (txtStampNumber1.Text.Substring(0, 3) !=
snumber2.Substring(0, 3))
{}
}

Source: http://community.livejournal.com/code_wtf/79331.html

Date: 04/06/07     Keywords: html, sql

Мало, но есть. Давайте вспомним героев большого WTF, а не красноглазых похапистов, опять забывших про sql injection.
Встречайте - Mork DB, худший формат файлов в истории IT. Мой браузер Firefox 1.5.0.11 хранит историю именно в нем (history.dat)

http://www.jwz.org/hacks/mork.pl
http://jwz.livejournal.com/312657.html

Source: http://community.livejournal.com/code_wtf/78965.html

Date: 04/05/07     Keywords: no keywords

private static String makeDigit(byte asciiByte) {
    return new String(new byte[] { asciiByte });
}

Source: http://community.livejournal.com/code_wtf/77797.html

Date: 03/31/07     Keywords: no keywords

На скрине я сравнил стандартные include-файлы из Visual Studio 6.0 и Visual Studio 8.0. Результат превзошёл мои ожидания... :)

"Видать, в MS велели почистить все исходники от любых намёков на ругательства после того, как код win2000 попал с Сеть. =)"

Оригинал

Source: http://community.livejournal.com/code_wtf/77324.html

Date: 03/29/07     Keywords: java

Недавно смотрел один большой кусок Java EE кода, в котором было много мелких приколов, которые затеняются двумя «гениальными» архитектурными решениями.

В коде не было указаний на авторов, по этому я не знаю, в рамках какой культурной традиции это было сделано. Возможно, это родилось на стыке разных традиций, ибо шибко уж нетривиально.

Приложение является Интернет магазином. И деньги снимаются и начисляются прямо в нем. На момент проверки кода, веб-интерфейс еще не был сделан.

На каком-то этапе разработки они озаботились вопросом, как администрировать систему. Веб-интерфейс показался им, наверное, слишком банальным и они решили сделать AdminConsole на Swing-е. В общем, у меня нет сильных возражений супротив этого. Swing так Swing. В некоторых случаях это даже оправдано, если есть на это деньги.

Следующее чем они озадачились это как связать AdminConsole и серверное приложение. По какой-то причине, они выбрали использовать локальные интерфейсы для EJB. Так что удаленно использовать их не получилось.

И тут им пришла в голову «гениальная» идея. Они решили запускать AdminConsole прямо в контексте веб приложения. Есть административный сервлет, где в методе doGet есть такая строчка:

…
new AdminConsoleFrame(adminInterfaceWrapper);
…

…
Account account =accounts.findByPrimaryKey(userid);
Float balance = account.getBalance();
if (balance == null) throw new BusinessLogicException();
float newAmount = balance.floatValue() - price.floatValue();
account.setBalance(new Float(newAmount));
…
Purchase purchase = purchases.create(itemid);
purchase.setAccount(account);
…

Source: http://community.livejournal.com/code_wtf/77025.html

Date: 03/27/07     Keywords: asp

C#, о том как надо писать контролы на ASP.NET не знаем, похапе форева.

#region Номер текущей страницы
protected int _currentPageNumber = 1;
protected bool firstReadCurrentPage = true;
public virtual int CurrentPageNumber
{
    get
    {
        if (firstReadCurrentPage)
        {
            try
            {
                if (Page.Request.QueryString[ID + "CurrentPage"] != null)
                {
                    if (Convert.ToInt32(Page.Request.QueryString[ID + "CurrentPage"]) > 0)
                        _currentPageNumber = Convert.ToInt32(Page.Request.QueryString[ID + "CurrentPage"]);
                }
            }
            catch { }
        }
        firstReadCurrentPage = false;

        return this._currentPageNumber;
    }
    set { this._currentPageNumber = value; }
}
#endregion

Source: http://community.livejournal.com/code_wtf/76115.html

Date: 03/24/07     Keywords: no keywords

Кладем в файл вот такой код:

#include bool f (GUID const &A, GUID const &B) { return IsEqualGUID (A, B);}

Компилируем 64-разрядным кросс-компилятором (из x86_amd64) от VS 2005 (14.00.50727.762) с выдачей ассемблерного листинга:

cl /c /FAs test.cpp

Открываем ассемблерный файл, и видим там вот такую функцию:

?f@@YA_NAEBU_GUID@@0@Z PROC ; f

Забавная фраза появляется только, если компилировать 64-разрядным компилятором

Source: http://community.livejournal.com/code_wtf/75634.html

Date: 03/22/07     Keywords: no keywords

do
{
if(!iterator.hasNext())
break;
...iterator.next();
...
} while(true);

Source: http://community.livejournal.com/code_wtf/74926.html

Date: 03/22/07     Keywords: xml, java, google

Есть такая контора - ThoughtWorks. И есть у неё такая библиотечка - XStream. Довольно широко используемая.

И есть в ней такой замечательный кусок кода. Можете посчитать в нём количество ненужных аллокаций объектов.

/**
     * Unescapes name re-enstating '$' and '_' when replacement strings are found
     * @param name the name of attribute or node
     * @return The String with unescaped name
     */
    public String unescapeName(String name) {
        StringBuffer result = new StringBuffer();
        int length = name.length();
        for(int i = 0; i < length; i++) {
            char c = name.charAt(i);
            if ( stringFoundAt(name, i, underscoreReplacement)) {
                i += underscoreReplacement.length() - 1;
                result.append('_');
            } else if ( stringFoundAt(name, i, dollarReplacement)) {
                i += dollarReplacement.length() - 1;
                result.append('$');
            } else {
                result.append(c);
            }
        }
        return result.toString();
    }

    private boolean stringFoundAt(String name, int i, String replacement) {
        if ( name.length() >= i + replacement.length()
          && name.substring(i, i + replacement.length()).equals(replacement) ){
            return true;
        }
        return false;
    }

Source: http://community.livejournal.com/code_wtf/74748.html

Date: 03/21/07     Keywords: no keywords

Кусок кода из одного совместного проекта:

def ip2long(ip):
    splittedaddr = ip.split('.')
    ipnum = 16777216*int(splittedaddr[0]) + 65536*int(splittedaddr[1]) + 256*int(splittedaddr[2]) + int(splittedaddr[3])
    return ipnum


def long2ip(ipnum):
    w = int ( ipnum / 16777216 ) % 256
    x = int ( ipnum / 65536 ) % 256
    y = int ( ipnum / 256 ) % 256
    z = int ( ipnum ) % 256
    ip = str(w) + "." + str(x) + "."+ str(y) + "." + str(z)
    return ip

Source: http://community.livejournal.com/code_wtf/74034.html

Date: 03/20/07     Keywords: no keywords

Set conn = Server.CreateObject("ADODB.Connection") Set NewCust = Server.CreateObject("ADODB.RecordSet") conn.Open "onenapilicontest" NewCust.Open "SELECT * FROM giveaway5 WHERE EMail = '" & Request.Form("email") & "'", conn , adOpenKeyset , adLockOptimistic

Code syntax highlighting by VIM captured with ScreenShot script

Source: http://community.livejournal.com/code_wtf/73882.html

Date: 03/19/07     Keywords: security

Когда читаешь код в статьях, обычно ожидаешь увидить вылизаный код. Однако бывают примеры когда не понимаешь, как код прошел редактора. Очень странный кусок кода я нашел в статье на MSDN Using Permutations in .NET for Improved Systems Security. Этот пример напомнил мне анекдот про чайник (вресия про програмистов).

Пример из статьи:

public Permutation(int n, int k)  
{
  this.data = new int[n];
  this.order = this.data.Length;

  // Step #1 - Find factoradic of k
  int[] factoradic = new int[n];
         
  for (int j = 1; j <= n; ++j)
  {
    factoradic[n-j] = k % j;
    k /= j;
  }
      
  // Step #2 - Convert factoradic to permuatation
  int[] temp = new int[n];

  for (int i = 0; i < n; ++i)
  {
    temp[i] = ++factoradic[i];
  }

  this.data[n-1] = 1;  // right-most element is set to 1.

  for (int i = n-2; i >= 0; --i)
  {
    this.data[i] = temp[i];
    for (int j = i+1; j < n; ++j)
    {
      if (this.data[j] >= this.data[i])
        ++this.data[j];
    }
  }

  for (int i = 0; i < n; ++i)  // put in 0-based form
  {
    --this.data[i];
  }

}  // Permutation(n,k)

Очевидно, что добавление единицы и последующее вычитание являются избыточными, и код мог бы быть записан способом, указаным ниже. Так же, после инициализации temp, массивы temp и factoradic имеют одинаковые значения элементов. Так что, непонятно зачем temp создавался.

public Permutation(int n, int k)  
{
  this.data = new int[n];
  this.order = this.data.Length;

  // Step #1 - Find factoradic of k
  int[] factoradic = new int[n];
         
  for (int j = 1; j <= n; ++j)
  {
    factoradic[n-j] = k % j;
    k /= j;
  }
      
  // Step #2 - Convert factoradic to permuatation
  this.data[n-1] = 0;  // right-most element is set to 0.

  for (int i = n-2; i >= 0; --i)
  {
    this.data[i] = factoradic[i];
    for (int j = i+1; j < n; ++j)
    {
      if (this.data[j] >= this.data[i])
        ++this.data[j];
    }
  }

}  // Permutation(n,k)

Source: http://community.livejournal.com/code_wtf/73670.html

Date: 03/16/07     Keywords: no keywords

Сегодня прислала подруга код с просьбой "посмотреть что не так".
Открываю и первым делом натыкаюсь на конструкцию:

while (Count < 3)
{
    ...
    ...
    ...

    Count = ++Count;
}

Source: http://community.livejournal.com/code_wtf/73395.html

Date: 03/16/07     Keywords: no keywords

Встретил в чужом коде следующий фрагмент:

boolean isValidBigInteger(String value) {
try {
BigInteger test = new BigInteger(value);
if (test != null)
return true;
} catch(Exception e) {
}
return false;
}

Судя по коду про OutOfMemoryError они еще не читали.

Source: http://community.livejournal.com/code_wtf/73047.html

Date: 03/16/07     Keywords: no keywords

Вот такие подряд стоящие строчки только что выкопал у нас в проэкте.

txtDate.Text = DateTime.Now.ToString();
try
{
DateTime.Parse(txtDate.Text, System.Globalization.DateTimeFormatInfo.CurrentInfo);
}
catch (Exception)
{
MessageBox.Show("Date is not in correct format", ......................

Source: http://community.livejournal.com/code_wtf/72925.html

Date: 03/13/07     Keywords: programming

Примеры найдены в Programming C# by Jesse Liberty (O'Reilly, 2003)

public double Pow(double x, double y){
   double retVal = x;
   for (int i = 0;i < y-1;i++){
      retVal *= x;
   }
   return retVal;
}

private void Order_Click(object sender, System.EventArgs e){
    string msg;
    msg = "Thank you " + txtName.Text +". You chose " ;
    for (int i = 0;i

(UPD: Во втором фрагменте - цикл по списку радиокнопок)

Source: http://community.livejournal.com/code_wtf/71967.html
	

Date: 03/12/07     Keywords: no keywords

Как человеку, испорченному Ораклом, нижеописанная конструкция кажется мне извращенной.

UPDATE #Temp
SET NameOrdering=A.DocName
FROM
 (SELECT FirstThreadID,MIN(RowNum) mrn
    FROM #Temp
  GROUP BY FirstThreadID) X
  JOIN (SELECT * FROM #Temp) A ON A.FirstThreadID=X.FirstThreadID and A.RowNum=X.mrn
  JOIN #Temp B ON B.FirstThreadID=A.FirstThreadID

The table '#Temp' is ambiguous.

Source: http://community.livejournal.com/code_wtf/71868.html

Date: 03/10/07     Keywords: no keywords

Да уж, действительно. Программирование на 1С разрушает мозг. Посмотрите, что написал минуту назад ваш покорный слуга %)

Процедура ЗаполнитьПричиныПовторногоОбращения()
    Для Каждого Строка ИЗ ПричиныПовторногоОбращения Цикл        
        // Вот так вот мы по имени значения перечисления обратились к элементу на форме...
        ЭлементыФормы[Строка.Причина.Метаданные().ЗначенияПеречисления.Получить(Перечисления.ПричиныПовторногоОбращения.Индекс(Строка.Причина)).Имя].Значение = Истина;
    КонецЦикла;    
КонецПроцедуры

Source: http://community.livejournal.com/code_wtf/71329.html

Date: 03/09/07     Keywords: no keywords

Наткнулся на код, писаный моим Project-манагером в довольно серьезном софте:

if ((pageNumber < 0) || (pageNumber >= m_pages.Count)) return;

if (pageNumber != -1)
{
m_pages[pageNumber].OnDeactivate();
m_pages[pageNumber].Hide();
}

m_pages[pageNumber].Show();
m_pages[pageNumber].OnActivate();

if (pageNumber == 0)
backButton.Enabled = false;
else
backButton.Enabled = true;

Source: http://community.livejournal.com/code_wtf/71046.html

Date: 03/02/07     Keywords: no keywords

две цитаты оттуда:
очень понравилось %)

while ($day > 1)
{
$day -= 7;
}

function get_month_name($month)
function get_total_days($month, $year)

Source: http://community.livejournal.com/code_wtf/70301.html