На каком я сейчас уровне?

Question

[Дядя Богдан]

Здраствуйте! Мне интересно узнать на каком я сейчас уровне на шарпе. Я написал вот такую простую консольную программу. Буду благодарен, если оцените код!

Репозиторий

Comments

[edward_freedom]

если у тебя будет сотня енамов, будет сотня условий? В чем преимущество в выборе свитча, вместо ифа?

switch(positionInput)
                                {
                                    case "trainee":

                                        position = Position.Trainee;

                                        break;

                                    case "junior":

                                        position = Position.Junior;

                                        break;

                                    case "middle":

                                        position = Position.Middle;

                                        break;

                                    case "senior":

                                        position = Position.Senior;

                                        break;

                                    default:

                                        Console.WriteLine("Position not found");

                                        positionApply = false;

                                        break;
                                }

[Дядя Богдан]

edward_freedom, А как это можно исправить?

[Василий Банников]

Дядя Богдан, например у тебя есть офигенный класс Enum в котором достаточно методов, чтобы убрать эту портянку

[edward_freedom]

Дядя Богдан, я бы в данном случае не использовал enum вообще, а использовал обычный класс с константами. Или можно как тут, создать словарь.
Можно с помощью рефлексии. Коды отсюда и отсюда

enum Position
        {
            [Description("trainee")] Trainee,
            [Description("junior")] Junior,
        }

var enumTestGet = Position.Trainee;
            Debug.WriteLine(enumTestGet.ToDescriptionString());

            var enumTestSet = "junior".GetValueFromDescription<Position>();
            Debug.WriteLine(enumTestSet.ToString());

public static class MyEnumExtensions
    {
        public static string ToDescriptionString(this Enum val)
        {
            var attributes = (DescriptionAttribute[])val
                .GetType()
                .GetField(val.ToString())
                .GetCustomAttributes(typeof(DescriptionAttribute), false);
            return attributes.Length > 0 ? attributes[0].Description : string.Empty;
        }


        public static T GetValueFromDescription<T>(this string description)
        {
            foreach (var field in typeof(T).GetFields())
            {
                if (Attribute.GetCustomAttribute(field,
                    typeof(DescriptionAttribute)) is DescriptionAttribute attribute)
                {
                    if (attribute.Description == description)
                        return (T)field.GetValue(null);
                }
                else
                {
                    if (field.Name == description)
                        return (T)field.GetValue(null);
                }
            }

            return default(T);
        }
    }

Я бы еще создал отдельный контроллер для работы с консолью, а сейчас у тебя не удобно сделано, везде идет обращение к консоли

Answer 1

[Василий Банников]

1. Не умеешь пользоваться гитом, тк загрузил файлы через Upload
2. Код невозможно проверить, тк ты закинул только cs файлы, но не приложил не менее важный csproj - не разбираешься, как собирается проект.
3. Про свич кейс тебе уже сказали в комменте.
Про остальное мало что можно сказать - код самый обычный, на три с минусом, не очень хороший, но и не слишком ужасный.
С архитектурой плохо, тк детали реализации смешаны с бизнес-правилами.
По алгоритмам нельзя оценить, тк никаких сложных алгоритмов в твоём проекте нет.

Уровень, имхо: стажёр/младший разработчик(обязательно под менторством и наблюдением более опытного)

Comments

[edward_freedom]

1. ты когда привозишь машину на мойку. кроме мойки еще и перекрашивать просишь и пороги поварить?

[Дядя Богдан]

А что не так с Upload? Это быстро и удобно.

Не думаю что там сложно разобратся просто в .cs, но возьму на заметку что нужно загружать весь проект

[Василий Банников]

edward_freedom, умение пользоваться гитом является одним из обязательных для разработчика.
Так что это я решил тоже оценить.
С тем же успехом он бы мог и на гуглодиск свой код загрузить - тогда бы да, умение пользоваться гитом я бы не стал оценивать.

[Василий Банников]

Дядя Богдан,

Это быстро и удобно.

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

но возьму на заметку что нужно загружать весь проект

Только не забудь bin/obj и прочие ненужные папки в .gitignore добавить.

Не думаю что там сложно разобратся просто в .cs

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

Без нормального README - это просто куча файлов, которые даже как портфолио стыдно показывать.

[profesor08]

edward_freedom, чтоб воспользоваться мойкой, не достаточно туда только приехать.

Answer 2

[lutokris]

Я к сожалению только начинаю изучать Python, Django, но почему мне кажется что вы написали что-то типа HelloWorld))

Comments

[edward_freedom]

в чем ответ?

Answer 3

[Saboteur]

Уровни есть в тетрисе. Или в думе.
А в разработке есть грейды, которые от одной компании к другой меняются как небо и земля.
Сходи на собеседование, узнай там.

По написанной программе ничего нельзя понять. Обычное использование стандартных функций.
Никто не знает кто придумывал техзадание и требования к нему, и насколько точно они были выполнены.
Нет адекватного описания программы, чтобы понять зачем ее нужно использовать.

То есть и сам код очень простенький, и подход к написанию продукта невнятный.

Answer 4

[Voland69]

Открыл первый попавшийся файл в репе:
1. Какова цель GetCompanyById, если к этой же коллекции есть доступ на чтение снаружи?
2. Что произойдет, если в этот метод я подам -1, 0, 100500?

К остальным замечаниям присоединяюсь (нет файла проекта, нет ридми и т.п.).
То есть сейчас это даже не программа - а набор исходников, под какой фреймворк/сдк, под какую среду?
Я же могу это закинуть в проект под .NET Framework 4.0 и сказать что у вас тут десяток ошибок.

Comments

[edward_freedom]

что у вас тут десяток ошибок

компилировать программу и не просили, просили оценить код

[edward_freedom]

1. создание прослойки для обращения к свойству нормально, не нормально что само это свойство публичное

[Voland69]

edward_freedom, компилировать и не надо, было бы неплохо открыть проект в IDE дабы удобнее было структуру и ссылки посмотреть.
1. создание прослойки совершенно нормально, но именно к публичному свойству удивило. И то что эта прослойка работает с листом, а обращается к индексатору и ничего не проверяет - т.е. упадет дай ей только повод.