Question

I know there are lots of methods already given in stackoverflow but in my case all of them taking too long time. I post a method which takes less time but still it is too long to implement. Please help me so that it takes less execution time. Also take consideration that i am using .net 2.0 framework.

        try
        {
            List<string> lstEmails = new List<string>();
            string filter1 = string.Format("(anr={0})", "groupname");
            DirectorySearcher searcher = new DirectorySearcher(entry);
            searcher.Filter = filter1;
            searcher.SearchScope = SearchScope.Subtree;
            searcher.PropertiesToLoad.Add("mail");
            IEnumerable res = (IEnumerable)searcher.FindOne().GetDirectoryEntry().Invoke("members");
            //IEnumerable<string> rest = (IEnumerable<string>)res;

            if (res != null)
            {
                try
                {
                    int index = 0;
                    foreach (IEnumerable resl in res)
                    {
                        DateTime start = DateTime.Now;
                        DirectoryEntry dr = new DirectoryEntry(resl);
                        string strEmail = null;
                        if (dr.Properties["mail"].Value != null)
                        {
                            strEmail = dr.Properties["mail"].Value.ToString();
                            Console.WriteLine(strEmail);
                            DateTime stop = DateTime.Now;
                            Console.WriteLine((stop - start).TotalMinutes.ToString());
                            index++;
                            Console.WriteLine(index.ToString());
                        }
                        if (!string.IsNullOrEmpty(strEmail))
                        {
                            // groupMemebers.Add("sam",strEmail);
                        }
                    }
                }
                catch { }
            }


        }
        catch { }

This is your suggested method Daro..

    DirectoryContext context = new DirectoryContext(DirectoryContextType.Domain, domain, "domainname" + strLDAPUserName, strLDAPPassword);
        DomainController controller = DomainController.FindOne(context);
        DirectoryEntry entry = new DirectoryEntry(string.Format("LDAP://{0}",controller.Domain), strLDAPUserName, strLDAPPassword, AuthenticationTypes.Secure);
List<string> userList = new List<string>();  
        DateTime StartTime = DateTime.Now;   
        using (DirectorySearcher ds = new DirectorySearcher(entry)) 
        {
            ds.PropertiesToLoad.Add("mail");  
            ds.PageSize = 10000;
            string DistingushiedName = "CN=" + groupName + ",OU=Users,dc=CompanyName,dc=com";
            ds.Filter = "(&(objectClass=user)(memberof:1.2.840.113556.1.4.1941:="+DistingushiedName+"))";   
            ds.SearchScope = SearchScope.Subtree; 
            try 
            {
                foreach (SearchResult user in ds.FindAll())   
                {
                    try  
                    {
                        userList.Add(user.Path);//.Properties["mail"][0].ToString()); 
                    }
                    catch (Exception E)    
                    {
                        throw new Exception(E.Message);
                    }
                }
            }
            catch(Exception E)    
            {
                throw new Exception(E.Message); 
            }
            DateTime EndTime = DateTime.Now;
            TimeSpan Dif = EndTime.Subtract(StartTime);
        } 

Answer 1

Hey this is the correct way…

  try
        {
            List<string> ReturnArray = new List<string>();
            DirectoryContext context = new DirectoryContext(DirectoryContextType.Domain, domainName, domainName + "\\" + UserName, Password);
            DomainController controller = DomainController.FindOne(context);
            string LDAPAddress = string.Format("LDAP://{0}", controller.Domain);
            DirectoryEntry deDirEntry = new DirectoryEntry(LDAPAddress, UserName, Password);
            deDirEntry.AuthenticationType = AuthenticationTypes.Secure;

            DirectorySearcher mySearcher = new DirectorySearcher(deDirEntry);
            mySearcher.PropertiesToLoad.Add("distinguishedName");
            string sFilter = String.Format("(&(objectcategory=group)(cn=" + GroupName + "))");

            mySearcher.Filter = sFilter;
            mySearcher.Sort.Direction = SortDirection.Ascending;
            mySearcher.Sort.PropertyName = "cn";
            SearchResult result;
            DirectoryEntry ResultEntry;
            result = mySearcher.FindOne();
            ResultEntry = result.GetDirectoryEntry();
            GroupName = ResultEntry.Properties["distinguishedName"].Value.ToString();
            mySearcher = new DirectorySearcher(deDirEntry);
            mySearcher.PropertiesToLoad.Add("cn");
            mySearcher.PropertiesToLoad.Add("mail");
            sFilter = String.Format("(&(objectClass=person)(memberOf={0}))", GroupName);
            mySearcher.Filter = sFilter;
            mySearcher.Sort.Direction = SortDirection.Ascending;
            mySearcher.Sort.PropertyName = "cn";
            SearchResultCollection results;
            results = mySearcher.FindAll();
            foreach (SearchResult resEnt in results)
            {
                ResultPropertyCollection propcoll = resEnt.Properties;
                foreach (string key in propcoll.PropertyNames)
                {
                    if (key == "mail")
                    {
                        foreach (object values in propcoll[key])
                        {
                            if (!String.IsNullOrEmpty(values.ToString()))
                            {
                                ReturnArray.Add(values.ToString());
                                Console.WriteLine(values.ToString());
                            }
                        }
                    }
                }
            }

            return ReturnArray;
        }
        catch
        {
            return null;
        }

Thanks everyone for your valuable suggesstion