Наткнулся только что вот на такой код одного из программистов наших клиентов...
Интересно, кто больше кривых мест в коде найдет? Сыграем в такую игру?
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;
}