System.NullReferenceException

@Гиоргий · Регистрация: 21.02.2018

Author24 — интернет-сервис помощи студентам

Доброго времени суток.дан код регистрации пользователя.вот отрывок из кода:

C#

static class UserManager
    {
        private static User[] _users = new User[10];
 
         
        static public int Register(User user)
        {
            for(int i = 0; i < _users.Length; i++)
            {
                _users[i] = new User();
            }
 
            if (user.Username == null && user.Password == null)
            {
                return -1;
            }
 
            for (int i = 0; i < _users.Length; i++)
            {
                if (_users[i].Username == user.Username)
                {
                    return -2;
                }
            }
 
            for (int i = 0; i < _users.Length; i++)
            {
                if (_users[i].Username == null && _users[i].Password == null)
                {
                    _users[i].Username = user.Username;
                    _users[i].Password = user.Password;
                    return 0;
                }
 
            }
 
            return -3; 
        }

на массиве _users[i].Username _users[i].Password выдает ошибку-- System.NullReferenceException.
Я прочитал что при создании вот этого массива --- private static User[] _users = new User[10] выделяется память только под массив и надо и под элементы тоже. я создал цикл ---

C#

 for(int i = 0; i < _users.Length; i++)
            {
                _users[i] = new User();
            }

но что то не идет(( подскажите пожалуйста как надо правильно

@VladPVS · 09.05.2018, 17:18

Гиоргий, попробуйте следующий код:

C#

static class UserManager
{
    private static User[] _users = new User[10];
 
    static public int Register(User user)
    {
        if (user == null || user.Username == null || user.Password == null)
        {
            return -1;
        }
 
        for (int i = 0; i < _users.Length; i++)
        {
            if (_users[i] == null) break;
                        
            if (_users[i].Username == user.Username)
                return -2;
        }
 
        for (int i = 0; i < _users.Length; i++)
        {
            if (_users[i] == null)
            {
                _users[i] = user;
 
                return 0;
            }
        }
 
        return -3;
    }

Ну а вообще, следовало бы использовать список (List) вместо массива, так как он расширяемый и всегда ясно сколько в нём элементов (свойство Count), таким образом не нужно перед каждой вставкой элемента убеждаться в наличии свободного места. Кроме того, вместо числовых кодов лучше возвращать элементы перечислений либо, в случае ошибок, генерировать исключения, так как в будущем будет трудно разбираться, что такое -1, -2 и -3.

@Гиоргий · 09.05.2018, 17:28 **[ТС]**

Сообщение от VladPVS

Ну а вообще, следовало бы использовать список (List) вместо массива

Мы еще не проходили List, и не могу его использовать

@Гиоргий · 12.05.2018, 12:55 **[ТС]**

Сообщение от VladPVS

C#

for (int i = 0; i < _users.Length; i++)
 {
 if (_users[i] == null) break;
if (_users[i].Username == user.Username)
return -2;
 }

я вот тут с break не догадался сделать. спасибо за помощь

@Гиоргий 7 / 4 / 0 Регистрация: 21.02.2018 Сообщений: 153
	09.05.2018, 17:28 [ТС]	3
	Сообщение от VladPVS Ну а вообще, следовало бы использовать список (List) вместо массива Мы еще не проходили List, и не могу его использовать 0

	12.05.2018, 12:55

System.NullReferenceException

Решение