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

    +111

    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
    var serializer = new XmlSerializer(CurcedFiles.GetType());
                    try
                    {
                        CurcedFiles = (List<string>)serializer.Deserialize(stream);
                        status = true;
                    }
                    catch (Exception)
                    {
                        stream.Close();
                        Random rnd = new Random();
                        int a = rnd.Next(1, 1000);
                        File.Move("CurceFile.xml", "CurceFile_error " + a.ToString() + ".xml");
                        status = false;
                    }

    Попытка прочитать из xml список файлов для удаления.

    Запостил: Flyperformances, 04 Сентября 2012

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

    • ava bormand 04.09.2012 13:26 # +1

      > CurcedFiles
      ПроклятыеФайлы?
      Ответить

    • ava guest 04.09.2012 14:21 # +1

      Понравилось.
      1) Регистрация ошибки содержит больше кода, чем код с потенциальной ошибкой.
      2) Есть подозрение, что поток stream в девичестве был CurceFile.xml.
      3) Random лучше на GUID заменить.
      Ответить

      • ava bormand 04.09.2012 14:54 # +1

        > Random лучше на GUID заменить.
        Есть такое.. В текущей реализации примерно после 32 ошибок вероятность совпадения имен превысит 1/2, что совсем не айс.
        Ответить

      • ava Steve_Brown 05.09.2012 14:42 # 0

        4) явное Close() вместо using или finally как-то подозрительно (хотя не факт, что ошибка)
        5) булевая переменная status

        А вместо random в идеале - лучше дату-время и/или порядковый номер.
        Ответить

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