C
ВО получаю. учу все что в программе курса есть.
Size: a a a
2021 April 17
C
еще программирование для ардунио и С++ осталось доучить
c
Minoru
классика на тему «C++ осталось доучить» https://abstrusegoose.com/strips/ars_longa_vita_brevis.png
классика на тему «C++ осталось доучить» https://abstrusegoose.com/strips/ars_longa_vita_brevis.png
c
Minoru
неплохо было бы добавить в README пару предложений о том, что именно делает эта программа
неплохо было бы добавить в README пару предложений о том, что именно делает эта программа
c
Minoru
а в код добавить док-комментарии к методам, чтобы было понятно, что они должны делать
а в код добавить док-комментарии к методам, чтобы было понятно, что они должны делать
c
Minoru
https://gitlab.com/getnumers/getnumbers/-/blob/f479c911ad3a91ed034fdaa47ac943a2484ee1d6/FileEngine.py#L10-11 Во-первых, тут может быть баг: если `len(self.__files)` вернёт 0, то `p` будет равно `-1`, и `if` выполнит then-ветку, где попытается обратиться к `self.__files[0]`. Во-вторых, `p` тут используется только в `if`, и при этом эксплуатируется тот факт, что 0 это False, а не ноль — True; лучше написать в явном виде: `if len(self.__files) == 1:`, без `p`
https://gitlab.com/getnumers/getnumbers/-/blob/f479c911ad3a91ed034fdaa47ac943a2484ee1d6/FileEngine.py#L10-11 Во-первых, тут может быть баг: если `len(self.__files)` вернёт 0, то `p` будет равно `-1`, и `if` выполнит then-ветку, где попытается обратиться к `self.__files[0]`. Во-вторых, `p` тут используется только в `if`, и при этом эксплуатируется тот факт, что 0 это False, а не ноль — True; лучше написать в явном виде: `if len(self.__files) == 1:`, без `p`
c
Minoru
…и, конечно же, добавить в этот `if` ветку для случая, когда `len() == 0`
…и, конечно же, добавить в этот `if` ветку для случая, когда `len() == 0`
c
Minoru
https://gitlab.com/getnumers/getnumbers/-/blob/f479c911ad3a91ed034fdaa47ac943a2484ee1d6/FileEngine.py#L30 Написано «от 0», хотя enumerate выше уже поправлен и нумерует файлы с единицы
https://gitlab.com/getnumers/getnumbers/-/blob/f479c911ad3a91ed034fdaa47ac943a2484ee1d6/FileEngine.py#L30 Написано «от 0», хотя enumerate выше уже поправлен и нумерует файлы с единицы
c
Minoru
https://gitlab.com/getnumers/getnumbers/-/blob/f479c911ad3a91ed034fdaa47ac943a2484ee1d6/FileEngine.py#L31 `user` хранит целое число, а инициализируется строкой. Не надо так делать, это сбивает читающего с толку. Инициализируй нулём, например
https://gitlab.com/getnumers/getnumbers/-/blob/f479c911ad3a91ed034fdaa47ac943a2484ee1d6/FileEngine.py#L31 `user` хранит целое число, а инициализируется строкой. Не надо так делать, это сбивает читающего с толку. Инициализируй нулём, например
c
Minoru
думаю, для этого раунда «ревью» комментариев достаточно :)
думаю, для этого раунда «ревью» комментариев достаточно :)
c
Minoru
> + Enables Rust support in the kernel.
> …
> + If unsure, say N.
хехе (цитата из https://lkml.kernel.org/lkml/20210414184604.23473-5-ojeda@kernel.org/)
> + Enables Rust support in the kernel.
> …
> + If unsure, say N.
хехе (цитата из https://lkml.kernel.org/lkml/20210414184604.23473-5-ojeda@kernel.org/)
AO
А шо не так
AO
Всё что не оттестировано по умолчанию выключено. И не рекомендуется к включению тоже.
AO
Требования вроде даже такие если что-то контрибутится
c
Minoru
меня позабавило, что если прикрыть один глаз и на минутку забыть, что это ядро, от предлагается в любой непонятной ситуации отказывается от Раста
меня позабавило, что если прикрыть один глаз и на минутку забыть, что это ядро, от предлагается в любой непонятной ситуации отказывается от Раста
AO
<fat>Не такой уж и плохой совет</fat>
c
Minoru
/me пошёл за новой тряпочкой для протирки монитора от жира
/me пошёл за новой тряпочкой для протирки монитора от жира