10.6 컨테이너 클래스 숙제 피드백

안녕하세요 강의를 듣고있는 학생입니다.

10.6에서 컨테이너 클래스의 개념을 설명해주시고

IntArray를 직접 구현해보는 숙제를 내주셔서 직접 머리를 짜내서 코드를 구현해봤습니다.

이 글을 보신 숙련자 분들께 제가 구현한 intArray를 피드백 받고싶어서 글을 작성합니다.

1. #include <iostream>

2. #include <vector>

3. using namespace std;

4. class IntArray

5. {

6. private:

7. int m_length = 0;

8. int* m_data = nullptr;

10. public:

11. IntArray(const int length)

12. :m_length(length)

13. {

14. m_data = new int[m_length];

15. }

16. IntArray(const std::initializer_list<int>& list)

17. :IntArray(list.size())

18. {

19. int count = 0;

20. for (auto& ele : list)

21. {

22. //cout << "list" << endl;

23. m_data[count] = ele;

24. count++;

25. }

26. }

27. ~IntArray()

28. {

29. delete[] m_data;

30. }

31. void initialize(const std::initializer_list<int>& list)

32. {

33. delete[] m_data;

34. m_length = list.size();

35. m_data = new int[m_length];

36. if (m_data != nullptr)

37. {

38. int count = 0;

39. for (auto& ele : list)

40. {

41. m_data[count] = ele;

42. count++;

43. }

44. }

45. }

46. void reset()

47. {

48. delete[] m_data;

49. m_data = nullptr;

50. }

51. void resize(const int size)

52. {

53. m_length = size;

54. }

55. void insertBefore(const int & value, const int & ix)

56. {

57. m_length += 1;

58. int *n_data = new int[m_length];

59. int i = ix;

60. for (int n = 0; n < ix; n++)

61. n_data[n] = m_data[n];

62. n_data[ix] = value;

63. for (i; i < m_length; i++)

64. {

65. n_data[i + 1] = m_data[i];

66. }

67. delete[] m_data;

68. m_data = n_data;

69. }

70. void remove(const int& ix)

71. {

72. m_length -= 1;

73. int* n_data = new int[m_length];

74. int count = ix;

75. for (int i = 0; i < ix; i++)

76. {

77. n_data[i] = m_data[i];

78. }

79. for (count; count < m_length; count ++)

80. {

81. n_data[count] = m_data[count+1];

82. }

83. delete[] m_data;

84. m_data = n_data;

85. }

86. void push_back(const int& value)

87. {

88. this->m_length += 1;

89. int* n_data = new int[m_length];

90. for (int i = 0; i < m_length; i++)

91. n_data[i] = m_data[i];

92. n_data[m_length-1] = value;

93. delete[] m_data;

94. m_data = n_data;

95. }

96. friend ostream& operator << (ostream& out, IntArray& ia)

97. {

98. for (int i = 0; i < ia.m_length; i++)

99. out << ia.m_data[i] << " ";

100. out << endl;

101. return out;

102. }

103. };

104. int main()

105. {

106. IntArray my_arr{ 1, 3, 5, 7, 9 };

107. cout << my_arr << endl;

108. my_arr.insertBefore(10, 1);

109. cout << my_arr << endl;

110. my_arr.remove(2);

111. cout << my_arr << endl;

112. my_arr.push_back(13);

113. cout << my_arr << endl;

114. my_arr.initialize({ 1, 2, 3 });

115. cout << my_arr << endl;

116. return 0;

}

피드백 받고싶은 이유는 함수를 구현할 때 Array에 int를 추가, 삭제, 수정을 할때

새로 동적할당을 받은 뒤에 그 공간에 기존 데이터를 복사해주며 추가,삭제,수정 해줄 부분을 해주고

기존 동적할당 공간은 지우고 기존 객체가 가르키는 포인터를 새로 할당한 공간으로 설정 해주었습니다.

선생님 께서 예시로 작성하는 코드 짜임새와 코드라인수에 비해

제 머리속에서 나온 그대로를 코드로 담은거라 너무 조잡 난해하다는 생각이 들었습니다.

그래서 코드를 더 간단하게 짜려면 이 내용을 어떤식으로 수정하면 좋을지 피드백 받고싶습니다.

혹은 코드에 아쉬운점들을 고수분들의 관점으로 보고 말씀해주시면 배워보고싶습니다 감사합니다.