1. C++ / Говнокод #2429

    +62.8

    1. 01
    2. 02
    3. 03
    4. 04
    5. 05
    6. 06
    7. 07
    8. 08
    9. 09
    10. 10
    11. 11
    12. 12
    13. 13
    14. 14
    15. 15
    16. 16
    17. 17
    18. 18
    19. 19
    20. 20
    21. 21
    22. 22
    23. 23
    24. 24
    25. 25
    26. 26
    27. 27
    28. 28
    29. 29
    30. 30
    31. 31
    32. 32
    33. 33
    34. 34
    35. 35
    36. 36
    37. 37
    38. 38
    39. 39
    40. 40
    41. 41
    42. 42
    43. 43
    44. 44
    45. 45
    46. 46
    47. 47
    bool failed = true;
	int count = 0;
	do
	{
		{
			ADO cnn( service().connectionString() );
			if (count == 0)
			{
				_RecordsetPtr rs = cnn.Command(L"SELECT MAX(MessNo) as MAXNO FROM RobotLog", ADODB::adCmdText)
					.Execute();
				if (rs->EndOfFile == VARIANT_FALSE)
				{
					vt = rs->Fields->Item[L"MAXNO"]->Value;
					if (vt.vt != VT_NULL)
						messNo = vt.lVal + 1;
				}
				rs->Close();
			}
			else
				messNo++;
			try
			{
				cnn.Command(L"INSERT INTO RobotLog(MessNo,MessType,MessDate,MessText,Empl_Code,CompName,Robot_Code, "
					L"Robot_Name,Doc_Name,Doc_No,Doc_CardDate) VALUES(?,?,?,?,?,?,?,?,?,?,?)", ADODB::adCmdText)
					.CreateParameter(messNo)
					.CreateParameter((long) messType)
					.CreateParameter((DATE) COleDateTime::GetCurrentTime())
					.CreateParameter(bstr_t(messText.AllocSysString()))
					.CreateParameter((long)m_EmplCode)
					.CreateParameter(service().computerName())
					.CreateParameter((long)m_ID)
					.CreateParameter(sRobotName)
					.CreateParameter(sDocName, false)
					.CreateParameter(sDoc_No, false)
					.CreateParameter(dCardDate)
					.Execute();
				failed = false;
			}
			catch(_com_error& )
			{
//				DebugMsgE("CRobotThread::LogRobotWork[%s] error: 0x%08x::%s - %s. Attempt #:%d", (const char*)m_RobotName, err.Error(), (const char*) err.Description(), 
//					(const char*) err.ErrorMessage(), count);
			}
		}
		count++;
		Sleep(100);
	}while (failed && count < 10000);

    Вычисляемое значение 'messNo' прописывается в PK. Код выполняется мультипоточночно, примерно из 5 - 15 одновременно работающих потоков.
    Вот такая вот попытка 10000 раз прописать неуникальное значение первичного ключа :)
    Но, самое смешное, что на моё предложение перенести заботу об уникальности и инкрементации в SQL, мне было велено "не умничать, ЭТО правильный алгоритм !...".

    Запостил: UncleVova, 18 Января 2010

    Комментарии (9) RSS

    • ava shapigor 18.01.2010 09:36 # +2

      Ну иииипаааать...
      Эт уж слишком...
      Ответить

    • ava guest 18.01.2010 09:49 # 0

      Вот и не умничайте. Грешно смеяться над больными людьми.
      Ответить

    • ava guest 18.01.2010 13:39 # +1

      А теперь за твои доклады, порочащие партию, UncleVova я тебя уволю!
      Началник. (с)
      Ответить

      • ava striker 18.01.2010 15:35 # 0

        Пожалейте его, начальник!
        Ответить

      • ava UncleVova 18.01.2010 16:09 # +2

        А, я сам уже оттуда уволился. Кабы этот перл был единственным....
        Ответить

        • ava guest 18.01.2010 17:58 # +1

          Это единственный перл и ты лишь из-за него уволился? Ты класный чувак!
          Ответить

          • ava UncleVova 19.01.2010 08:02 # 0

            Не, ты не понял. Перл далеко не единственный.
            Ответить

    • ava TarasB 18.01.2010 20:44 # −1

      А я вчера написал код с отступами в 16 символов. Я победил. Где мой приз?
      Ответить

      • ava UncleVova 19.01.2010 09:44 # 0

        Ну, с отступами погорячился. Первый раз коммичу. Просто при копипасте таб развернулся не в 4 а в 8 символов. Простите засранца. Но, на отступы можно забить. Там вся фишка в последней строчке кода :).
        Ответить

    Добавить комментарий