Ви не увійшли. Будь ласка, увійдіть або зареєструйтесь.
Ласкаво просимо вас на україномовний форум з програмування, веб-дизайну, SEO та всього пов'язаного з інтернетом та комп'ютерами.
Будемо вдячні, якщо ви поділитись посиланням на Replace.org.ua на інших ресурсах.
Для того щоб створювати теми та надсилати повідомлення вам потрібно Зареєструватись.
Український форум програмістів → C++ → Передача та отримання масива
Сторінки 1
Для відправлення відповіді ви повинні увійти або зареєструватися
Добрий день.
Виникло питання про передачу та отримання масива функціями.
В мене є метод, який приймає текст і розбиває його на деяку послідовність слів записуючи в масив.
Далі для перевірки масиву мені потрібно його передати в іншу функцію, яка знаходиться в іншому класі.
Приклад:
Проблема в тому, що функція, яка повинання виконувати дії над масивом, приймає лише його перший елемент.
void FirstClass :: setArray() {
array[5] = {0, 1, 2, 3, 4};
}Так масиви в С не задаються. Виключення - коли ви декларуєте (оголошуєте) масив.
А так - от варіант:
void FirstClass :: setArray() {
//somewhere was const int lenOfArray
for(int i = 0; i < lenOfArray; array[i] = i, i++);
}
Як оголошена змінна "array" FirstClass?
Якщо у CheckClass не відомо роміру масиву, то треба додатково його у функцію передавати, типу
void CheckClass :: checkArray(int* array, int size) {
/* Будь-які дії */
}А взагалі, краще використовуй std::vector, набагато менше мороки буде
void FirstClass :: setArray() { array[5] = {0, 1, 2, 3, 4}; }
Це створює (проголошує і ініціалізує) локальну в функції setArray змінну array типу int[5] - якщо, звісно, array не було проголошено раніше, але ви про це ані слова не кажете. Ця змінна припиняє своє існування разом з останньою фігурною дужкою функції (точніше, разом із тією фігурною дужкою, в якій вона була проголошена).
Якщо ж array було проголошено раніше (ви про це не казали), то це буде записувати щось (адресу константного масиву) в елемент array[5].
Bartash вибачаюсь за таку помилку, я навів це як приклад, в мене масив заповнюється по іншому.
0x9111A в мою функцію (не ту яку я тут написав) передається 2 масива і їхні розміри, але проблема в тому, що передається лише перший елемент масиву.
koala приношу свої вибачення за неповні дані.
Ось архів з кодами класів:
Переглянув файл - там нема таких класів і функцій. Ви певні, що то саме воно?
Переглянув файл - там нема таких класів і функцій. Ви певні, що то саме воно?
Те, що я показував раніше - загальний приклад, я гадав, що цього вистачить.
Те, що я представив в архіві - мої оригінальні класи і функції, які я використовую.
В ідеалі вони повинні працювати таким чином:
1. В класі ShingleText, в метод createShingles передається текст і кількість слів тексту;
2. Текст розбивається на послідовність слів (шингли) і записується в масив;
3. Масив сортується;
4. Масив і довжина масиву передається в клас CheckText в функцію setArrayResult.
І як я вже казав, в теорії передається лише перший елемент масиву.
1. Потенційна проблема: кількість слів випливає з тексту, а якщо випадково передати неправильну кількість - що буде?
2. Код жахливий, але то біс з ним.
3. Нащо передавати параметром методу класу елемент класу? Метод і так знає, які в класі елементи.
4. Де саме вони передаються? Не бачу. Який файл, який рядок?
1. Потенційна проблема: кількість слів випливає з тексту, а якщо випадково передати неправильну кількість - що буде?
2. Код жахливий, але то біс з ним.
3. Нащо передавати параметром методу класу елемент класу? Метод і так знає, які в класі елементи.
4. Де саме вони передаються? Не бачу. Який файл, який рядок?
1. Кількість слів рахується та передається правильно.
2. Чи це в оформленні, чи і логіці коду? Скажіть будь-ласка, як можна зробити його не таким жахливим?
3. Виправив, дякую.
4. Це відбувається в головній програмі а саме:
Боюся, що всього цього мені не досить, щоб вказати на помилку. Зазвичай просять показати мінімальний шматок коду, що демонструє проблему. Код, який не можна скомпілювати, вочевидь, нічого не демонструє.
Стосовно якості коду:
1. Користуйтеся конструкторами, створення і ініціалізація мають відбуватися одночасно (вкрай бажано).
Не
ShingleText* originalShingles = new ShingleText;
...
originalShingles -> createShingles(originalText -> getInputText(),
originalText -> getWordCount());а
ShingleText* originalShingles = new ShingleText(originalText -> getInputText(),
originalText -> getWordCount());2. Ви передаєте в функції кілька параметрів об'єкту, хоча краще передати сам об'єкт (чи посилання на нього) зі всіма параметрами.
Не
originalShingles -> createShingles(originalText -> getInputText(),
originalText -> getWordCount());а
originalShingles -> createShingles( originalText );Не
Check -> setArrayResult(originalShingles -> getShingle(),
suspectShingles -> getShingle(),
originalShingles -> getShingleSize(),
suspectShingles -> getShingleSize());а
Check -> setArrayResult( originalShingles, suspectShingles );3. Не користуйтеся "магічними числами":
Не
shingleArray = new string[_wordCount - 2];а
const int shingleSize = 3;
...
shingleArray = new string[_wordCount - shingleSize + 1];4. В createShingles ви робите вкрай неефективно. Алгоритм має перебирати всі слова в рядку і знаходити їхні позиції за один прохід, наприклад, у відповідний масив (пар позиція - довжина, скажімо), а потім можете збирати їх в свої шингли. Ще краще замість масиву використовувати чергу, їх для того і придумали. Ви ж замість цього кілька разів шукаєте один і той самий пробіл різними функціями.
koala дуже вам дякую. Проблему з масивами вирішив, використавши пунк №2 який ви описали вище.
Сторінки 1
Для відправлення відповіді ви повинні увійти або зареєструватися