C# Что вы думаете об этом коде?(не чит)

[Gonzo]

https://pastebin.com/U3uWtBay

Изменено 13 июня, 2018 пользователем Gonzo

[JustHack]

[censored]

Изменено 11 апреля, 2020 пользователем JustHack

[Garik66]

  В 13.06.2018 в 12:01, JustHack сказал:

У автора, странная манера комментов))

Показать  

Соглашусь - обычно на такие вопросы хочется ответить: "Я не думаю об этом коде "

 

[mrPTyshnik]

Что лично мне бросилось в глаза:

закоменченный код(с зеродеем вспомнил, так же было в сорцах на сиес сурс): 
 //code

 //code

 //code

 

 

/*

 code

 code

 code

*/

 

for (int i = 0; i < myStringleft.Count(); i++)
    {
        dictionary.Add(myStringleft[i], myStringRight[i]);
    }

Маловероятно, но ключ(Stringleft) уже может существовать

 

 

Не понимаю зачем было писать функцию "CheckString", если можно было воспользоваться string.CompareTo("0" - равны, "-1" - не равны)

 

 

 

 

бест кратчесы:

int CheckEncoding(string[] tmpContent)
            {
                foreach (string a in tmpContent)
                {
                    if (a.Contains("о") || a.Contains("щ") || a.Contains("ы") || a.Contains("п") || a.Contains("я") || a.Contains("р"))
                    {
                        return 1251;
                    }
                    else
                    {
                        return 65001;
                    }
                }
                return 65001;
            }

 

 

 

еще одна особенность которая занимает больше кода:

int temp = null;

temp = 123;

 

можно int temp = 123; - ну это кому как
 

а что за софт?

Изменено 13 июня, 2018 пользователем mrPTyshnik
дай деду из деревни отредактировать его ошибки!

[Gonzo]

Программа для пакетной обработки файлов)

 

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

 

писал поздно ночью и только учусь.

[partoftheworlD]

  В 13.06.2018 в 16:49, Gonzo сказал:

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

Показать  

Т.е обычные вещи для современной ОС и командной строки.

И опять C#, что за поколение С# -ов?

[Gonzo]

            if (myStringleft != null && myStringRight != null)
            {
                if (myStringleft.Count() == myStringRight.Count())
                {

for (int i = 0; i < myStringleft.Count(); i++)
    {
        dictionary.Add(myStringleft[i], myStringRight[i]);
    }

не понимаю в чем тут проблема? значение-да, возможна проблема

Маловероятно, но ключ(Stringleft) уже может существовать

[mrPTyshnik]

  В 13.06.2018 в 17:07, Gonzo сказал:

   for (int i = 0; i < myStringleft.Count(); i++)
    {
        dictionary.Add(myStringleft[i], myStringRight[i]);
    }

не понимаю в чем тут проблема? значение-да, возможна проблема


 

Показать  

если ключ myStringleft уже существует, то вылетит ексепш(лучше проверять через TryGetValue)

Если же ключ никак не сможет повториться - то оставь как есть

[Gonzo]

Про ArgumentException понял, спасибо(если скажет знакомый которому пишу, то исправлю. Скорей всего одинаковые строки недопустимы ). 

NullReference - по идее не будет, там есть проверка, чуть выше.

остальные замечания конечно же не оставлю без внимания. Всем спасибо)

 

Изменено 13 июня, 2018 пользователем Gonzo

[Xipho]

Интересный метод коллективного рефакторинга.

[Gonzo]

Всегда интересно послушать более опытных товарищей.

 

мне ещё предстоит сюда многопоточность прикрутить.

 

Там в папке которую нужно обработать , 100+ файлов. 

 

[Xipho]

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