170 похожих чатов

Здравствуйте, вопрос по чистоте кода. Достаточно чистый код или можно

сделать лучше? Не бейте тапками, только учусь. Краткое описание:
Класс, выдающий уникальный строковый идентификатор пользователя, состоящий из знаков в поле allCharacters и длиной в idLength
Проверяющий идентификаторы на уникальность(были ли уже такие)
Основной метод - GetIdentifier(). Метод AddToEnd расширяет массив, содержащий все идентификаторы и вставляет в него новый идентификатор. Метод isUnique проверяет сгенерированный идентификатор на уникальность.

15 ответов

11 просмотров

Guid надо юзать

Бессмысленные аллокации строк через +=

Ну и проверка будет иметь алгоритмическую сложность O(n), что не очень хорошо

А почему весь класс статик?

Если это часть какой то программы, то лучше заменить на Guid

Intergetn- Автор вопроса

Ну, там особо нечему быть нестатическим

Intergetn
Ну, там особо нечему быть нестатическим

А если я захочу несколько экземпляров? Чтобы я генерил уникальные айди для одного кейса, потом уникальные айди для второго независимого кейса. При этом айди между собой могут повторяться, мне все равно, главные чтобы в первой группе они были между собой уникальными и во второй группе они были между собой уникальными

Intergetn- Автор вопроса

Здесь утечка памяти, массив будет увеличиваться, без возможности его очистить.

Intergetn- Автор вопроса
Олег
Здесь утечка памяти, массив будет увеличиваться, б...

Сделать метод, с помощью которого можно будет удалить идентификатор из массива или что?

_

Intergetn- Автор вопроса
Олег
_

О, мой код стал примером как не надо делать

Похожие вопросы

Обсуждают сегодня

Всем привет, написал код ниже, но он выдает сегфолт, в чем причина? #include <stdio.h> #include <stdlib.h> #include <string.h> struct product { char *name; float price; };...
buzz базз
70
Здравствуйте. Задача состоит в том, чтобы сделать real-time чат в мобильном приложении. После передачи сообщения пользователем через веб-сокеты, для основного и долговременног...
🐾
5
Всем доброго дня, ребят подскажите пожалуйста, если в курсе по ассемблеру используется MASM32, могу ли я использовать FASM? В чем явная разница и будет ли у меня все работать?
Botsman
17
Хотел бы спросить у знающих, правильную ли я выбрал книгу для начала изучения ассемблера Юрова В.И ? Или есть более лучшие книги для начала обучения?
Botsman
25
Книга Юрова В.И пойдёт для обучения?
Botsman
24
$params = [ 'formid' => 'feedbackForm', 'formTpl' => '@CODE: <form class="form-validate" data-id="ajax_form"> <fieldset class="margin-bottom-md"> ...
Pathologic
1
Люди добрые, помогите с идеями, потому что свои закончились. У клиента падает софтина в момент инициализации модуля OtlEventMonitor на RegisterWindowMessage('Gp/OtlTaskEvents/...
Михаил Усков
7
Тут просто дело в том, что я не могу сейчас дать такие подробности из за того что рассчитать это всё нереально. Этого проекта который я хочу сделать ещё даже не существует) И ...
🐾
8
> Примечательно, что новый владелец удаляет из GitHub любые жалобы, указывающие на подозрительную активность или смену владельца, и, видимо, рассчитывает на то, что пользовате...
Alex Sherbakov
2
GridView fully ignored first parent(SizedBox), and take width from second parent(Container). How can I constrain GridView by first parent? Widget build(BuildContext context) {...
Hamster
1
Карта сайта