Osadchyi homework3 by NikOsadchyi · Pull Request #43 · MykolaSuperman/HomeWorkBase03

NikOsadchyi · 2020-11-12T21:09:22Z

No description provided.

GKuzzinshtern

В общем, там где мои мысли повторялись, я их уже не писал.
Хочу отметить, что нежелательно использовать имена переменных типа i,j,k даже если на данный момент это однозначный маленький цикл. Он со временем может наполнится еще кодом и читатели будут гадать что означает эта буква. Короткое содержательное имя - будет самое то! :)

homework3.cpp

GKuzzinshtern · 2020-11-14T19:54:04Z

homework3.cpp

+}
+
+bool is_dgt(char var) {
+  int dec_v = static_cast<int>(var);


можно пользоваться операторами сравнения для char. Не стоит их кастить в данном случае к int

сравнивать что символ является числом? 10 сравнений?)

if (var >= '0' && var <= '9') - начить это цифра

homework3.cpp

GKuzzinshtern · 2020-11-14T19:58:20Z

homework3.cpp

+}
+
+void reverse_str() {
+  constexpr int arr_len{100};


для размеров можно использовать size_t вместо int

это наверное дело вкуса?)

сам не могу привыкнуть :(

homework3.cpp

GKuzzinshtern · 2020-11-14T20:11:03Z

homework3.cpp

+  int arr[arr_len]{};
+  for (int i = 0; i < arr_len; ++i) {
+    int val{};
+    std::cout << "Enter a number: ";


побольше тексту!

GKuzzinshtern · 2020-11-14T20:30:10Z

homework3.cpp

+    std::cin >> val;
+    for (int j = 0; j < arr_len; ++j) {
+      std::cout << val;
+      if (arr[j] == 0) {


если пользователь введет "0", похоже будет "ой" :( Практика использовать потенциально возможное значение как критерий поведения к хорошему не приводит. Т.е. 0 - не должен означать, что ячейка не заполнена.

согласен, пока что не придумал решения. Заводить флаг, если первый элемент, то ставить его первым, а дальше от него шагать?

или смещать элементы на место удаленного...

GKuzzinshtern · 2020-11-14T20:36:32Z

homework3.cpp

+    for (int j = 0; j < arr_len; j++) {
+      if (num == arr[j]) {
+        std::cout << "Element " << num << " is removed from array" << std::endl;
+        arr[j] = '-';


не-не-не. Так нельзя делать. У Вас arr - это интовый массив. То что Вы присвоили, компилятор неявно преобразовал const char в int. И значение в данный элемент массива попало 45. Если бы Вы вывели итоговый массив, то увидели бы это.
Здесь вообще, по-моему, надо поступать аналогично с заданием на добавление числа. Только при удалении смещать элементы массива, чтоб заполнить освободившееся место.

придумал обходной костыль.)

Зачем? Смещайте элементы. std::string здесь вообще не к месту (хоть я его и люблю :) )

MykolaSuperman · 2020-12-04T21:54:31Z

homework3.cpp

+#include <locale>
+
+int get_str_len(char* str) {
+    int count{};


такое как размер последовательности лучше обьявлять как безнаковое целое, а не знаковое
лучше для этого подойдет тип size_t

MykolaSuperman · 2020-12-04T21:56:22Z

homework3.cpp

+    int str_len = get_str_len(_str);
+
+    int strt = 0;
+    int end = str_len - 1;


Suggested change

int str_len = get_str_len(_str);

int strt = 0;

int end = str_len - 1;

int str_len {get_str_len(_str)};

int strt {0};

int end {str_len - 1};

MykolaSuperman · 2020-12-04T21:58:34Z

homework3.cpp

+bool is_dgt(char var) {
+    int dec_v = static_cast<int>(var);
+    constexpr int lwr_ascii_dec_lim = 48;
+    constexpr int uppr_ascii_dec_lim = 57;


как вы думаете тот кто читает этот код он знает на изусть таблицу ascii?)
лучше писать символьные литералы 0

MykolaSuperman · 2020-12-04T21:59:40Z

homework3.cpp

+            arr[i] = arr[i] & '_';
+        }
+        else {
+            arr[i] = arr[i];


а зачем это? просто не надо else ветки

MykolaSuperman · 2020-12-04T22:01:05Z

homework3.cpp

+    std::cin >> arr;
+    for (int i = 0; arr[i] != '\0'; ++i) {
+        if (std::isalpha(arr[i], loc)) {
+            arr[i] = arr[i] & '_';


капец, зачем так сложно) ну представте себя в роли человека который будет читать этот код, вот как понять мы побитово складываем один символ с символом подчеркивания??????)))))

MykolaSuperman · 2020-12-04T22:03:29Z

homework3.cpp

+
+void remove_elmtn() {
+    constexpr int arr_len{ 5 };
+    std::string arr[arr_len];


ничего не понял а зачем вам массив строк?

MykolaSuperman · 2020-12-04T22:04:06Z

homework3.cpp

+
+    while (elmnts_deleted != arr_len) {
+        bool present = false;
+        std::string num{};


зачем тут string в качестве строки?

MykolaSuperman · 2020-12-04T22:05:29Z

homework3.cpp

+        std::string num{};
+        std::cout << "Enter a number which you want to delete from array: ";
+        std::cin >> num;
+        std::string rmvd_ch = "-";


непонял вообще что это и зачем эта магия?
просто проверяйте что строка не пустая и все я думаю что этого вполне достаточно

MykolaSuperman · 2020-12-04T22:07:31Z

homework3.cpp

+        std::cin >> num;
+        std::string rmvd_ch = "-";
+        for (int j = 0; j < arr_len; j++) {
+            if (num == arr[j] && arr[j] != rmvd_ch) {


вот подумайте над этим условием. что куда лучше поставить, т.е. поменять местами

MykolaSuperman · 2020-12-04T22:09:24Z

homework3.cpp

+                arr[j] = val;
+                val = tmp;
+                j = 0;
+            }


я думаю вам стоит подумать, и реализацию лучше написать

NikOsadchyi added 2 commits November 12, 2020 23:04

Implemented tasks

bf717c7

removed left cpp

5634faf

NikOsadchyi requested review from GKuzzinshtern, MykolaSuperman and sserohin November 12, 2020 21:09

NikOsadchyi added 2 commits November 12, 2020 23:10

removed comments

39fe93f

added newline

31281a9

GKuzzinshtern reviewed Nov 14, 2020

View reviewed changes

NikOsadchyi added 2 commits December 3, 2020 22:52

added changes

6dd4013

hot fixes

ad93876

Repository owner deleted a comment Dec 3, 2020

MykolaSuperman reviewed Dec 4, 2020

View reviewed changes

Conversation

NikOsadchyi commented Nov 12, 2020

Uh oh!

GKuzzinshtern left a comment

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

3 participants

Comments