alloya/oop

Repository for OOP labs

Hey! Nice to see you.

---

👩‍💻 About Me :

I am Yulia from Yoshkar-Ola, Russia and currently living in Belgrade, Serbia.

💻 Junior frontend developer with experience in TypeScript, Angular, React, Sass, and Bootstrap.

✨ Driven to create visually appealing and functional web experiences for users.

💡 Committed to continuous learning and improvement of my skills.

🚂🚋🚋 Collaborative team player with strong communication and problem-solving skills.

---

🛠️ Languages and Tools :

               

---

🔥 My Stats :

oop's Issues

Замечания по списку

• Не реализованы конструктор копирования и перемещения, а также операторы присваивания (копирующий и перемещающий)

• Обращение к nullptr при попытке взять end у пустого списка

• Из операций над итератором есть только префиксный инкремент. Нет декремента и постфиксной формы инкремента

• При текущей реализации итератор на end() невозможно уменьшить, т.к. он указывает на nullptr и не знает, как переместиться назад

• Констнантный итератор - это не итератор, который нельзя инкрементить и декрементить, а итератор, который можно перемещать вперед и назад, но доступ к данным возвращает константную ссылку (*) либо указатель на константные данные (->).

• Итератор реверсивный на самом деле у вас не является таковым

• У итератора не реализован оператор ->

Замечания по CountWords

• Обновить структуру тестов:

• Функция CountWords
  • подсчитывает частоту слов разделяющихся whitespace-ами
  • игнорирует регистр символов
  • возвращает пустой map, если во входной строке нет печатаемых символов

Замечания по CMyStack

	CMyStack(CMyStack<T> & stack)
	{
		*this = stack;
	}

• Это не конструктор копирования. КК должен принимать аргумент по константной ссылке

Замечания по Flipbyte

bool IsItNumber(string number)
{
    for (unsigned int i = 0; i < number.length(); i++)
    {
        if (!isdigit(number[i]))
        {
            return false;
        }
    }
    return true;
}

• объекты следует передавать по константной ссылке, если функция не модифицирует их состояние
  примитивные объекты передавать по константной ссылке выгоды нет, лучше по значению

Замечания по FindText

• Не соблюдены соглашения об именовании (функции)

Замечания по Rational

• Добавить в тесты всех бинарных операторов проверки, что все работает, когда один из аргументов (слева и справа) является целым числом

Замечания по Find and Replace

1>c:\teaching\2016\ips\lushinskaya\oop\lab2\findandreplace\utils.cpp(34): warning C4267: '=': conversion from 'size_t' to 'int', possible loss of data
1>c:\teaching\2016\ips\lushinskaya\oop\lab2\findandreplace\utils.cpp(39): warning C4267: '=': conversion from 'size_t' to 'int', possible loss of data

• программа должна компилироваться без предупреждения в 64 и 32 битных конфигурацих

Замечания по mult matrix

            if (file.eof())
            {
                return false;
            }
            file >> matrix[m][n];

лучше так:

if (!(file >> matrix[m][n]))
{
    return false;
}

• m, n лучше заменить на row и col

bool GetMatrix(ifstream & file, Matrix & matrix)
{
	for (size_t row, col = 0; col < MATRIX_DIM; col++)
	{
		for (row = 0; row < MATRIX_DIM; row++)
		{
			if (!(file >> matrix[col][row]))
			{
				return false;
			}
		}
	}
	return true;
}

void PrintMatrix(ostream & output, const Matrix & matrix)
{
	output << fixed;
	for (size_t col = 0; col < MATRIX_DIM; col++)
	{
		for (size_t row = 0; row < MATRIX_DIM; row++)
		{
			output << setprecision(3) << matrix[col][row] << '\t';
		}
		output << endl;
	}
}

• перепутаны строки и столбцы в именах переменных

Замечания по программе Bodies

• Выяснить и уверенно объяснить, зачем в CBody деструктор объявлен виртуальным.

• Почему в CParallelepiped объявлен деструктор, но почему-то без ключевого слова virtual. Объяснить

• Почему в CParallelepiped метод GetVolume() объявлен без ключевого слова virtual? Объяснить.

Замечания по Solve4

• Одинаковые корни должны схлапываться в 1
  Тут ожидается 3 корня, а не 4
  https://www.wolframalpha.com/input/?i=x%5E4-x%5E2%3D0

Замечания по телевизору

• Выделить группу тестов, проверяющих поведение включенного телевизора, во вложенный fixture test suite.

• Не хватает теста, что телевизор не может включать предыдущий канал в выключенном состоянии

• Нужен тест переключения телевизора на предыдущий канал сразу после включения

Замечания по vector

• Данные должны считываться из стандартного потока ввода, а не из файла