The Daily "wtf?!"

Jul 02, 2007 12:33

Наткнулся только что вот на такой код одного из программистов наших клиентов...
Интересно, кто больше кривых мест в коде найдет? Сыграем в такую игру?

public boolean validate()
{
    VendInvoiceJour        vendInvoiceJour;
    VendInvoicePurchLink vendInvoicePurchLink;
    UserGroupList            userGroupList;
    boolean ret;
;

ret = super();

// Сотрудники группы казначейстав могут изменять значение поля.
    select firstonly userGroupList
        where userGroupList.UserId   == curUserId() &&
                 userGroupList.groupId == "Казнач";

select * from vendInvoiceJour
        index hint InvoiceAccountIdx
     join vendInvoicePurchLink
        where vendInvoiceJour.PurchId             == vendInvoicePurchLink.PurchId               &&
                 vendInvoiceJour.InvoiceId           == vendInvoicePurchLink.InvoiceId             &&
                 vendInvoiceJour.InternalInvoiceId == vendInvoicePurchLink.InternalInvoiceId  &&
                 vendInvoiceJour.InvoiceDate        == vendInvoicePurchLink.InvoiceDate        &&
                 vendInvoicePurchLink.origPurchId == purchTable.PurchId;

if (vendInvoiceJour)
    {
        ret = false;
    }
    if (vendInvoiceJour && userGroupList)
    {
        ret = true;
    }

return ret;
}

humor

Previous post Next post
Up