Си диез / Говнокод #13158 Ссылка на оригинал

0

  1. 1
  2. 2
  3. 3
  4. 4
  5. 5
  6. 6
  7. 7
  8. 8
  9. 9
  10. 10
  11. 11
  12. 12
  13. 13
  14. 14
  15. 15
  16. 16
  17. 17
private void timer1_Tick(object sender, EventArgs e)
        {
            sec++;

            if (sec > 59)
            {
                min++;
                if (min > 59)
                {
                    ch++;
                    min = 0;
                }
                sec = 0;
            }

            label1.Text = "Время игры: \r\n" + ch + ":" + min + ":" + sec;
        }

Часики -_-

Запостил: ipro ipro, (Updated )

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

  • ava sbs # Ссылка на оригинал 0

    label1.Text = DateTime.Now.ToShortTimeString()+":"+Dat eTime.Now.Second.ToString("0#");
    Ответить

    • ava tirinox # Ссылка на оригинал 0 ⇈

      // <s>fixed</s>
      StringBuilder sb = new StringBuilder("Время игры: \r\n");
      sb.Append(DateTime.Now.ToShortTimeString ());
      sb.Append(":");
      sb.Append(DateTime.Now.Second.ToString(" 0#"));
      label1.Text = sb.ToString();
      Ответить

      • ava bormand # Ссылка на оригинал 0 ⇈

        Но какой смысл юзать стрингбилдер ради трех конкатенаций очень коротких строк? Имхо ненужное переусложенение кода.

        P.S. А че в шарпе нет нормального форматирования дат аля strftime? Только конкатенировать шорттайм и секунды?
        Ответить

        • ava bormand # Ссылка на оригинал 0 ⇈

          Есть же ведь. Так что по минусу обоим знатокам шарпа:
          String.Format("Время игры: {0:H:mm:ss}", DateTime.Now);

          P.S. Да еще и дату два раза получаете не закешировав значение. Фу, понасрали больше чем автор.
          Ответить

          • ava bormand # Ссылка на оригинал 0 ⇈

            P.S. Ну и конечно нужно вычесть время начала игры. Все-таки код должен показывать игровое время, а не настоящее.
            Ответить

          • ava bormand # Ссылка на оригинал 0 ⇈

            Ну ок, ночью у меня сарказмодетектор не всегда срабатывает 🙂
            Но код sbs'а хуже, чем код автора, да еще и решает совсем не ту задачу 😉
            Ответить

          • ava neeedle # Ссылка на оригинал 0 ⇈

            Да ну?

            Наверное нужно запиливать фак, для тех кто не сразу может понять. Что бы соглашения были о тегах, так сказать, ибо я тоже не понял, что значит <s></s>.
            Ответить

            • ava tirinox # Ссылка на оригинал 0 ⇈

              Да ладно, какое это значение имеет?
              Одному было лень бибитэг вспоминать, другой этого не понял и влепил минус. Такие мелочи 🙂
              Ответить

              • ava neeedle # Ссылка на оригинал 0 ⇈

                Вот видите, а могли бы чуть ниже нажать ссылочку bbcode и посмотреть. 🙂
                Ответить

  • ava guest # Ссылка на оригинал 0 

    private static string Time(DateTime startTime, DateTime currentTime){
	TimeSpan gameTime = currentTime - startTime;
	return gameTime.ToString ();
}


    Как вариант решения, а то холивары уже пошли)
    Ответить

    • ava bormand # Ссылка на оригинал 0 ⇈

      А оно всякие AM/PM не вывалит в соответствующих локалях? Все-таки, имхо, в этом случае лучше самому Стринг.Формат дернуть.
      Ответить

      • ava ipro # Ссылка на оригинал 0 ⇈

        Да не должен, судя по MSDN там интервал от 00 до 23, хотя хз, не буду утверждать)
        Ответить

        • ava bormand # Ссылка на оригинал 0 ⇈

          А, тут TimeSpan, ну да, по логике не должен, это я тупанул.
          Ответить

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

Из-за тебя ушел bormand, guest!

    А не использовать ли нам bbcode?

    ↑
    8
    ↓