Conversation
| #include "BinaryRep.h" | ||
|
|
||
|
|
||
| void Complement(signed char x, int out[]) { |
There was a problem hiding this comment.
Стайлгайд вообще не выдержан. Так же, как и мысли о том, что переменным и аргументам нужно давать не однобуквенные имена
There was a problem hiding this comment.
И это какое-то очень странное название для функции
| } | ||
| } | ||
|
|
||
| void PrintBin(int A[]) { |
There was a problem hiding this comment.
Я сюда могу передать массив и 2 и из 122 элементов. Стоит явно обернуть массив нужного размера в структуру.
There was a problem hiding this comment.
А ещё массив из bool будет логичнее
| int sum = A[i] + B[i] + carry; | ||
| S[i] = sum & 1; | ||
| carry = (sum >> 1); |
There was a problem hiding this comment.
Здесь получается не очень честно, потому что у Вас может получиться двойка. А если перейти на язык
электроники, то без этого можно обойтись :) https://en.wikipedia.org/wiki/Adder_(electronics)#Full_adder
| } | ||
| } | ||
|
|
||
| int BinToInt(int A[]) { |
There was a problem hiding this comment.
А зачем, если Вы везде оперируете с signed char. Кстати, вместо него можно было взять int8_t.
|
|
||
|
|
||
| int main() { | ||
| int x, y; |
There was a problem hiding this comment.
Инициализировать значения, всё такое?..
| void Complement(signed char x, int out[]); | ||
| void PrintBin(int A[]); | ||
| void SumBin(int A[], int B[], int S[]) ; | ||
| int BinToInt(int A[]); |
There was a problem hiding this comment.
А кто будет писать комментарии к тому, что это вообще за функции?!
2 participants
Шалахина Анна дз-12