{"action":"created","issue":{"url":"https://api.github.com/repos/Biunovich/nsudotnet-1-/issues/1","labels_url":"https://api.github.com/repos/Biunovich/nsudotnet-1-/issues/1/labels{/name}","comments_url":"https://api.github.com/repos/Biunovich/nsudotnet-1-/issues/1/comments","events_url":"https://api.github.com/repos/Biunovich/nsudotnet-1-/issues/1/events","html_url":"https://github.com/Biunovich/nsudotnet-1-/issues/1","id":77061160,"number":1,"title":"Общие замечания","user":{"login":"sfactor","id":352511,"avatar_url":"https://avatars.githubusercontent.com/u/352511?v=3","gravatar_id":"","url":"https://api.github.com/users/sfactor","html_url":"https://github.com/sfactor","followers_url":"https://api.github.com/users/sfactor/followers","following_url":"https://api.github.com/users/sfactor/following{/other_user}","gists_url":"https://api.github.com/users/sfactor/gists{/gist_id}","starred_url":"https://api.github.com/users/sfactor/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/sfactor/subscriptions","organizations_url":"https://api.github.com/users/sfactor/orgs","repos_url":"https://api.github.com/users/sfactor/repos","events_url":"https://api.github.com/users/sfactor/events{/privacy}","received_events_url":"https://api.github.com/users/sfactor/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":7,"created_at":"2015-05-16T15:22:24Z","updated_at":"2015-05-20T10:51:24Z","closed_at":null,"body":"Сначала несколько вопросов:\r\n\r\n- Почему некоторые методы в криптоклассах помечены как internal, а некоторые — нет?\r\n\r\n- Почему ключи и IV сохраняются в файлы с расширением XML?\r\n\r\nПотом несколько замечаний:\r\n\r\n- Программа не консольная, но не выполняет полностью требования к WPF-версии, поэтому максимально возможное количество баллов будет 6 (но см. небольшое замечание ниже)\r\n\r\n- В случае, если в коде подразумевается поддержка нескольких криптоалгоритмов, обычно всё реализуется следующим образом: все криптоклассы наследуются от единого интерфейса и реализуют один и тот же набор методов, определенный в этом интерфейсе. Это позволяет легко подменять один криптоалгоритм другим, а также легко и красиво расширять систему\r\n\r\n- Текущие криптоклассы совершенно неприменимы в многопоточном окружении. Попытка пользоваться ими одновременно из разных потоков приведет к ошибкам. Предлагаю сделать их thread-safe, за это могу накинуть дополнительно 1 балл"},"comment":{"url":"https://api.github.com/repos/Biunovich/nsudotnet-1-/issues/comments/103841858","html_url":"https://github.com/Biunovich/nsudotnet-1-/issues/1#issuecomment-103841858","issue_url":"https://api.github.com/repos/Biunovich/nsudotnet-1-/issues/1","id":103841858,"user":{"login":"Biunovich","id":11133115,"avatar_url":"https://avatars.githubusercontent.com/u/11133115?v=3","gravatar_id":"","url":"https://api.github.com/users/Biunovich","html_url":"https://github.com/Biunovich","followers_url":"https://api.github.com/users/Biunovich/followers","following_url":"https://api.github.com/users/Biunovich/following{/other_user}","gists_url":"https://api.github.com/users/Biunovich/gists{/gist_id}","starred_url":"https://api.github.com/users/Biunovich/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/Biunovich/subscriptions","organizations_url":"https://api.github.com/users/Biunovich/orgs","repos_url":"https://api.github.com/users/Biunovich/repos","events_url":"https://api.github.com/users/Biunovich/events{/privacy}","received_events_url":"https://api.github.com/users/Biunovich/received_events","type":"User","site_admin":false},"created_at":"2015-05-20T10:51:24Z","updated_at":"2015-05-20T10:51:24Z","body":"Добавлена потоко - безопасность."}}