- 1
- 2
- 3
- 4
public void OnObjectException(EventArgs e, Exception ex)
{
throw ex;
}
Нашли или выдавили из себя код, который нельзя назвать нормальным, на который без улыбки не взглянешь? Не торопитесь его удалять или рефакторить, — запостите его на говнокод.ру, посмеёмся вместе!
+134.2
public void OnObjectException(EventArgs e, Exception ex)
{
throw ex;
}
Индусятина!
+138.9
[DataContract]
public class MyDataContract {
string lASTNAME = "";
string nAMESUFFIX = "";
string mIDDLENAME = "";
string fIRSTNAME = "";
[DataMember]
public string LASTNAME {
get { return lASTNAME; }
set { lASTNAME = value; }
}
[DataMember]
public string FIRSTNAME {
get { return fIRSTNAME; }
set { fIRSTNAME = value; }
}
[DataMember]
public string MIDDLENAME {
get { return mIDDLENAME; }
set { mIDDLENAME = value; }
}
[DataMember]
public string NAMESUFFIX {
get { return nAMESUFFIX; }
set { nAMESUFFIX = value; }
}
}
Акуенная конвенция именования переменных
+140
void f(bool b)
{
if(b.ToString().Length() == 4)
; // типа true
else if(b.ToString().Length() == 5)
; // типа false
}
Сравнение
+3
try
{
GetRecentOrdersList();
}
catch (XmlFileHasWrongFormatException)
{
wrongClientSettingsXmlFormat = true;
}
if (wrongClientSettingsXmlFormat == true)
{
IList<RecentDocumentInfo> fakeList = new List<RecentDocumentInfo>();
SetRecentOrdersList(fakeList);
wrongClientSettingsXmlFormat = false;
}
Как правильно обрабатывать исключения
+133.9
void FF_MouseEnter(object sender, MouseEventArgs e)
{
itForSelected.IsSelected = !new bool();
}
ппц)
+145.9
try
{
if (SQL_SUCCESS == SQLAllocHandle(SQL_HANDLE_ENV, m_environmentHandle, out m_environmentHandle))
{
if (SQL_SUCCESS == SQLSetEnvAttr(m_environmentHandle,SQL_ATTR_ODBC_VERSION,(IntPtr)SQL_OV_ODBC3,0))
{
if (SQL_SUCCESS == SQLAllocHandle(SQL_HANDLE_DBC, m_environmentHandle, out m_connectionHandle))
{
if (SQL_NEED_DATA == SQLBrowseConnect(m_connectionHandle, inConnection, stringLength, outConnection, DEFAULT_RESULT_SIZE, out stringLength2Ptr))
{
if (SQL_NEED_DATA != SQLBrowseConnect(m_connectionHandle, inConnection, stringLength, outConnection, DEFAULT_RESULT_SIZE, out stringLength2Ptr))
{
throw new ApplicationException("No Data Returned.");
}
}
}
}
}
}
catch (Exception ex)
{
throw new ApplicationException("Cannot Locate SQL Server.");
}
Увидел на codeguru. Аффтар импортирует функции WinAPI для работы с SQL в C#. Работа с исключениями также доставляет.
+145.9
public void newMessageTrySafe(Message message)
{
if (SeparateThread.Wait(100, 5, new ExitWaitDelegate(this.)))
{
try
{
this.newMessage(message);
}
catch (Exception exception)
{
Utils.log("DataCache.newMessage() error: " + exception.Message, new object[0]);
}
try
{
Monitor.Exit(this.FRecalcSynchronizer);
return;
}
catch (SynchronizationLockException)
{
return;
}
}
Utils.log("newMessageTrySafe(): Deadlock! Unable to call newMessage bacause it's locked by this thread", new object[0]);
}
Решение проблемы дедлоков в многопоточной программе.
+134.6
private bool _isProcessingThreadExited = false;
public bool IsShutDownComplete()
{
if (!_isProcessingThreadExited)
{
// a thread hasnt exited yet
return false;
}
return true;
}
... осталось еще bool на true проверить через ToString().Length > 4 - и это будет верх говнокодерства...
+128.2
public static string ValidateEmail(string Email)
{
try
{
if (Email.Trim() == string.Empty)
{
return "Empty Parameter - Email";
}
}
catch
{
return "Empty Parameter - Email";
}
return null;
}
+143.7
/*Hide deleted button*/
if (((UF.FileTypeID == FP.FileType1) && (FP.FileType1Edit == 1))
|| ((UF.FileTypeID == FP.FileType2) && (FP.FileType2Edit == 1))
|| ((UF.FileTypeID == FP.FileType3) && (FP.FileType3Edit == 1))
|| ((UF.FileTypeID == FP.FileType4) && (FP.FileType4Edit == 1))
|| ((UF.FileTypeID == FP.FileType5) && (FP.FileType5Edit == 1))
|| ((UF.FileTypeID == FP.FileType6) && (FP.FileType6Edit == 1))
|| ((UF.FileTypeID == FP.FileType7) && (FP.FileType7Edit == 1))
|| ((UF.FileTypeID == FP.FileType8) && (FP.FileType8Edit == 1))
|| ((UF.FileTypeID == FP.FileType9) && (FP.FileType9Edit == 1))
|| ((UF.FileTypeID == FP.FileType10) && (FP.FileType10Edit == 1))
|| ((UF.FileTypeID == FP.FileType11) && (FP.FileType11Edit == 1)))
{
if (CBL.GetListOfButtons(CPF.ID, 1) == true)
{
bool ButtonVisible = true;
String BText = "";
foreach (ConfirmButtons CB in CBL.Items)
{
BText = "";
if ((CB.ActionID > 0)&&(aAction.GetActionInfo(CB.ActionID) == true))
{
if (DTS.isStepAllowed(CB.ActionID, sysUser.GetID(), UserRoleID, aRequest.ID) == true)
{
ButtonVisible = true;
#region Exeptions
if (CB.TypeName == "Confirm")
{
/*----------Check parallel process status--------------*/
if (aRequest.IsParent == 1)
{
if (CheckBP.CheckParallelBP(aRequest.ID, aRequest.aReqStatus.ID) == true)
{
if ((CheckBP.IsNecessary == 1) || ((CheckBP.ChildID > 0) && (CheckBP.ChildCurState > 0)))
{
if (((CheckBP.IsNecessary == 1) && ((CheckBP.ChildID == 0))) || ((CheckBP.PositiveEndState != CheckBP.ChildCurState) && (CheckBP.NegativeEndState != CheckBP.ChildCurState)))
{
ButtonVisible = false;
CFTitleText.Text = "В данный момент вы не можете cогласовать заявку. Незавершен параллельный процесс: '" + CheckBP.Name.ToString() + "'!";
}
}
}
}
Кусок, начиная со строки 881(из 1307) метода Page_Load. Мастурбация мозга..