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

    +133

    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
    var status = true;
                    var name = aspnet_UsersRepository.FindAll().FirstOrDefault(u => u.aspnet_Membership.Email.Equals(model.Email, StringComparison.InvariantCultureIgnoreCase));
                    if (name != null) { return View("Error_user"); }
    
                    /*13.08.2014 EmirMamashovCode{} нахождение сущ. ли такой агент*/
                    var usersBalanses = UserBalancesRepository.FindAll().ToList();
                   /* если заполнено код агента и имя агента */
                    if (model.KodAgent != null && model.NameAgent != null)
                    {
                        foreach (var usersBalanse in usersBalanses)
                        {
                            if (model.KodAgent == usersBalanse.KodforPay)
                            {
                                status = true;
                            }
                            else{status = false;}
                        }
                        if (status == false)
                        {
                            return View("Error_notFoundAgent");
                        }
                       
                    }

    нахождение сущ. ли такой агент

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

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

    • и ч0, работает?
      Ответить
      • нет естественно.
        foreach (var usersBalanse in usersBalanses)
                            {
                                if (model.KodAgent == usersBalanse.KodforPay)
                                {
                                    status = true;
                                }
                                else{status = false;}
                            }


        Тут явно хочется
        foreach (var usersBalanse in usersBalanses)
                    status |= model.KodAgent == usersBalanse.KodforPay;


        а иначе агент найден если только он последний

        да и
        if (model.KodAgent == null || model.NameAgent == null) return;

        в середине метода выглядит как - мать моя женщина, а может нам вообще ниче не стоило делать? Данных то нужных нет!
        Ответить
        • тут явно надо не это var usersBalanses = UserBalancesRepository.FindAll().ToList( );
          А что нибудь типа, (не знаю как это в шарпе написать )
          var usersBalanses = UserBalancesRepository.FindbyAttributes( 'Условие для отбора').ToList();
          Ответить
          • Where это называется
            UserBalancesRepository.FindAll().Where(x=>x.KodforPay == model.KodAgent ).FirstOrDefault();


            или даже
            UserBalancesRepository.FindAll().FirstOrDefault(x=>x.KodforPay == model.KodAgent);

            Ответить
            • хотя FindAll - это тоже что то из linq, а не метод класса, который выдает колекцию. Так что FindAll нинужен
              Ответить
        • if(!UserBalancesRepository.FindAll().Any(p=>p.KodforPay==model.KodAgent))
              return View("Error_notFoundAgent");
          Ответить
          • Ну или так. Скажи умный человек, зачем findall?
            Ответить
            • Хз...
              Если-бы он кушал предикат, то можно было сказать что это:
              http://msdn.microsoft.com/en-us/library/1kkxfxdd%28v=vs.80%29.aspx
              Старый, добрый, {Array,List<T>}.FindAll.
              А в этом контексте - хз что это может обозначать...
              Ответить
              • Может он через findAll из какого нибудь DataBaseSet конвертит в IEnumarable?
                Ответить
                • Не, для этого каст есть.

                  Типизированные датасеты с линку так выглядят:
                  public SettingsDataSet.TreeRow GetTreeNode(Int32 nodeId)
                  {
                  	return this.DataSet.Tree.FirstOrDefault(p => p.NodeID == nodeId);
                  }
                  public String[] GetServerList()
                  {
                  	return this.DataSet.Client.Select(p => p.Server).Distinct().ToArray();
                  }
                  Ответить
                  • ну так автор может не знать каста. Какая то ересь короче
                    Ответить
            • kegdan 14 часов назад # 0
              Ну или так. Скажи умный человек, зачем findall?

              Потому что гавно код, главная работала :))
              Ответить
      • а почему не должно ?
        Ответить
      • Lokich 17 часов назад # 0
        и ч0, работает?

        Да работает, но неправильно конечно. status будет все время false потому что оно в цикле находится, и все время будет return View("Error_notFoundAgent").
        Ответить
    • Опять любители 2Листочек... 2Словарик совести не хватает заюзать? :(

      Надо-же, кто-то натянул на стандартный SqlMembershipProvider ещё и ORM...
      Там уже и так всё хранимками обёрнуто (это отдельное говно, но по крайней мере, уже проверенное веками), нафига было деревянное колесо к машине прибивать...
      Ответить
      • зелено - видимо на этом сайте чаще регаются, чем логинятся

        Хотя даже в этом случае словарь не на много хуже
        Ответить
        • > видимо на этом сайте чаще регаются, чем логинятся
          Увидев в стоке, подумал, что это про ГК.
          Когда я только пришёл сюда, (либо была какая-то аномалия, либо я никого не знал) тут писали чаще олдфаги, чем ньюфаги. А сейчас почти все посты создают пользователи с ID over 9000, но писать комментарии ещё не могут.
          Ответить
          • а как узнать свой ID?
            Ответить
            • Навести мышь на имя пользователя около комментария/поста или, если комментариев/постов нет, в кабинке навести на "мои говнокоды".
              Ответить
              • Олдфаг ITT!
                Ответить
              • Помню тут как-то мерялись ID я даже почему то был дальним родственником борманда
                Ответить
                • Три тыщи постов прошло. А было как вчера. http://govnokod.ru/13821#comment197037
                  Интересно, что чувствуют олдфаги, потратив 5 лет на 10-15К постов про багры...
                  Ответить
                  • год прошел. Эх... каким же дураком был год назад.
                    Ответить
                  • мне кажется с датами что то не то. На сколько я помню, я на говнокоде не сидел в прошлом сентябре. Только под новый год
                    Ответить
                  • wvxvw - 988. Как мы видим он свихнулся.
                    Ответить
                    • более древние настолько обезумели, что могут делать только багрование. Скоро и с бормандом это начнется. Сначала будет читать заумные лекции, потом абыр! абыр! начнет писать про отсосы у кавказских членодевок, спрашивать про С++ и кидать зигу багор.
                      Ответить
                      • Я здесь раньше борманда.
                        Ответить
                        • Хм. А я думал Борманд тоже из отцов основателей. Тогда стоит опасаться тебя
                          Ответить
                          • Я вообще ньюфаг.
                            Ответить
                          • Надо почаще наводить мышь на ники.

                            А если хочется понять, когда примерно пользователь зарегистрировался, надо разделить ID на 1900 - столько лет прошло с июля 2009го до его регистрации.
                            Ответить
                            • 1900 новых юзеров в год? Фигассе.
                              Ответить
                            • Довольно сомнительная методика. Жаль, я сейчас не найду, когда я первый раз комментировал, да и лениво, если честно.
                              Ответить
                              • > Довольно сомнительная методика.
                                Помогает понять, кто отец-основатель, а кто - сын-продолжатель...
                                > Жаль, я сейчас не найду, когда я первый раз комментировал, да и лениво, если честно.
                                Лениво было ещё со второго комментария.
                                Вот кандидаты на пост первых комментариев:

                                Comment #46454 by eth0 (2010-09-07T19:38:18+04:00, post #4183)
                                Идиот, промахнулся темой.

                                Comment #46455 by eth0 (2010-09-07T19:41:23+04:00, post #4181)
                                Нет таких слов для выражения моей охреневатости в сторону автора сего чуда, знатного норкомана.
                                Через полторы минуты после изучения нашёл "дырку" (видимо, не первая и не последняя).
                                http://wiki.wup.ru/hista.php?d=/../../config.ini&id=1
                                Создаёт статью №1 с конфигом, пароль "adgjmptw" - охрененно сложный, да.
                                Можно было бы пошариться по файловой структуре, но что-то так лениво.

                                Автору - вон из профессии.

                                Comment #46456 by eth0 (2010-09-07T19:42:40+04:00, post #4183)
                                (Идиот, разумеется, я. Хорошо, что раз можно сообщение редактировать.)

                                Comment #46756 by eth0 (2010-09-09T19:28:14+04:00, post #4183)
                                Именно.

                                Comment #47138 by eth0 (2010-09-11T12:18:28+04:00, post #4224)
                                Ответ на главный вопрос жизни, вселенной и всего такого.

                                Comment #47596 by eth0 (2010-09-14T18:26:11+04:00, post #4250)

                                > Koeff
                                На этом месте граммар-наци должен доставать StG-44 и топать ножками. Особенно раздражает, когда "коеффы" чередуются с "бордерами".
                                Ответить
                                • Хорошо, когда у кого-то есть боевые треножникискрипты, или навык гугления.
                                  Но тогда. Тогда формула работает с точностью до месяца. Тысяча чертей!
                                  Ответить
                                  • А, Вы про формулу... Я про белый текст уже забыл.
                                    Ответить
                                    • самое время на d3 запилить симпатишные отчеты с графиками, пайчартами и няшными анимациями
                                      Ответить
                                  • Да, судя по всему, очень похоже на то. Чуть раньше этого поста идут страницы, на всех полно непрочитанных комментариев. Поскольку я читал их от гостя, интереса перечитывать заново не было. Н-да. Уже четыре года подряд не анонимно говнокодчевал капчу.
                                    Ответить
                                  • > Тогда формула работает с точностью до месяца.
                                    Кто-то специально дорегивал ботов, чтобы формула не портилась?
                                    Ответить
                    • >988
                      Ньюфаг.
                      Док, док, док, доконвог!
                      Ответить
    • показать все, что скрытоПрочитайте говоскоп на сегодня, чтобы узнать, что я въебал этому посту...
      Ответить
    • Авт. люб. исп. сокр.?
      Ответить
    • - Конечно, милый. Наш дом - это и твой дом тоже.
      Ответить

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