1. C# / Говнокод #16490

    +135

    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
    public JsonResult MusicPlayer(DateTime date)
        {
            var order = OrdersRepository.FindAll().ToList();
            int play = 0;
            
            foreach (var item in order)
            {
                if (item.Date<=date){
                    var date2 = date.ToString();
                    return Json(new { date2});
                }
                return Json(new { play });
            }

            return Json(new { play });
        }

    Отправление данных через Json

    Запостил: Emir, 06 Августа 2014

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

    • ava tirinox 06.08.2014 17:20 # +1

      О! Прикольно. Это типа анонимный класс создается? И полю play присваиватся локальное значение play ? Или как?
      Ответить

      • ava guest 08.08.2014 14:40 # 0

        tirinox
        О! Прикольно. Это типа анонимный класс создается? И полю play присваиватся локальное значение play ? Или как?

        Скорее Анонимные типы создается
        Ответить

    • ava TauSigma 06.08.2014 17:31 # 0

      Многовато написано...
      public JsonResult MusicPlayer(DateTime date) {
	var item=OrdersRepository.FindAll().FirstOrDefault();

	return item!=null && item.Date<=date
		? Json(new { date.ToString() })
		: Json(new { 0 });
}

      А вообще, что ЭТО должно делать?
      Ответить

      • ava kegdan 06.08.2014 17:45 # +2

        Посылать Json в ASP.NET MVC в асинхронном запросе
        Ответить

        • ava TauSigma 06.08.2014 17:54 # +2

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

          • ava kegdan 06.08.2014 18:05 # 0

            честно - вообще не понимаю что тут происходит. находим любой заказ, дата которого меньше указанной в запросе и возвращаем дату. Если хочется статистику - можно было получше придумать. Зачем дату отправлять строкой - хз. почему любой заказ - тоже хз
            Ответить

            • ava TauSigma 06.08.2014 18:06 # 0

              В том-то и проблема, что не любой, а только первый заказ.
              Ответить

              • ava kegdan 06.08.2014 18:09 # 0

                По логике они добавляются по порядку. Поэтому будет либо первый либо пусто (ну да, ты так и показал). Получается так информируют о том, что в заказах что то есть. Тогда зачем передавать дату?
                Ответить

                • ava TauSigma 06.08.2014 18:53 # 0

                  У меня только одно предположение:
                  Чтобы клиенту сложнее было подставить ответ, пока он не сделает первый заказ.
                  Ответить

            • ava guest 08.08.2014 11:33 # 0

              Происходит вот что, если подробно: Я беру во вьюшке из инпута текущую дату через Json отправляю в OrdersController в акшн MusicPlayer, а там уже эту дату(date) (date делаем строкой чтобы можно было отправить через Json) сравниваю с датой в заказа (item.Date) которое в базе и если там была так (item.Date<=date) должно быть (item.Date>=date чтобы определить что это новый заказ). Короче если item.Date>=date то отправляем его, если нет тогда отправляем 0. Остальное JS ом обрабатываю если пришло date то перезаписываю инпут и проигрываю звук, если ноль останавливаю музыку.
              Ответить

              • ava TauSigma 08.08.2014 13:10 # 0

                >Короче если item.Date>=date то отправляем его, если нет тогда отправляем 0.
                А почему-бы не отправлять дату последнего заказа, а не дату пришедшую с клиента?

                Либо можно всегда отправлять дату последнего заказа. В таком варианте, можно задействовать eTag.
                Ответить

          • ava guest 08.08.2014 11:18 # 0

            Извини что не отвечал, я недавно зарегистрировался, по коду немножко не попал, оповещает админа если клиент отправил заказ. При приходе нового заказа проигрывается звук ну типа как в mail.ru
            Ответить

      • ava Lennis 06.08.2014 21:39 # 0 

        new { 0 }
        не будет работать

        нужно имя филда указать в котором будет этот 0 хранится

        int zero = 0;
new { zero }
        Ответить

        • ava TauSigma 06.08.2014 21:41 # 0

          Да я прям здесь написал, скопипастив первоисточник. Потом только заметил, что забыл типы массивов проставить...
          Конечно-же:
          new Int32[] { 0, };
new String[] { date.ToString(), };
          Ответить

          • ava kegdan 06.08.2014 21:52 # 0

            Тип можно не указывать

            new [] { 0 };
new [] { date.ToString()};
            Ответить

            • ava TauSigma 06.08.2014 21:58 # +1

              Для ынтерпрайза - лучше указать. И вары не использовать.
              А то потом коллеги полкараулят в тёмном уголке, и встреча с тёмным властелином будет казаться не такой уж страшной карой...
              Ответить

              • ava kegdan 06.08.2014 22:00 # 0

                Тут даже коню (и даже оранжевой пони) понятны типы.

                Кстати в коде то не массивы, а анонимные типы (не люблю их).
                Ответить

        • ava bormand 06.08.2014 21:47 # +1

          О_о. Это что за уличная магия, имеющая доступ к именам переменных?
          Ответить

          • ava TauSigma 06.08.2014 22:06 # 0

            Это из области варов, когда можно не указывать тип переменной передаваемой в массив.
            static void Main(String[] args)
{
	var result = ProcessCode();
	var array = new { result };
	Console.WriteLine(array.Any(p => p.ToString() == "quick") ? "fox" : "dog");
}
static String ProcessCode()
{
	return "quick";
}


            И легонечко отрефакторив код - получаем логическую ошибку:
            static void Main(String[] args)
{
	var result = ProcessCode();
	var array = new { result };
	Console.WriteLine(array.Any(p => p.ToString() == "quick") ? "fox" : "dog");
}
static Int32 ProcessCode()
{
	return 1;
}
            Ответить

            • ava TauSigma 06.08.2014 22:07 # 0

              >И легонечко отрефакторив код
              Не код, а всего лишь метод "ProcessCode"
              Ответить

            • ava kegdan 06.08.2014 22:22 # +1

              не скомпилится же

              только так
              https://ideone.com/WeW2mb
              Ответить

              • ava TauSigma 06.08.2014 22:39 # 0

                Да блин...
                var array = new []{ result };


                Ща ещё пару часиков на работе посижу и домой поеду...
                А то уже косячить начинаю...
                Ответить

                • ava 1024-- 06.08.2014 22:48 # 0

                  > Ща ещё пару часиков на работе посижу и домой поеду...
                  Как-то поздно для любой точки России.
                  Ответить

                  • ava kegdan 06.08.2014 22:49 # 0

                    у меня вообще 3 ночи. Время веселых монад!
                    Ответить

                  • ava TauSigma 06.08.2014 22:53 # 0

                    У меня халява, жену с ребёнком отправил на дачу, а дома - только с WinRT ковыряться, но мне сегодня влом...
                    Так что можно поработать в своё удовольствие и никто домой не гонит ^_^
                    GMT +4
                    Ответить

                    • ava 1024-- 06.08.2014 23:00 # +2

                      > поработать в своё удовольствие
                      А мы тут отвлекаем.
                      Надо будет говорить тише. Особенно - про весёлые монады.
                      Ответить

        • ava kegdan 06.08.2014 21:54 # 0

          Тут уже не массив, тут анонимный тип
          Ответить

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