помогите найти ошибку: 


.....
перегрузка  такая:

ошибка-то в чём? опять телепатов вызывать?

и он выдает такое сообщение: Unhandled Exception 
что неправильно не могу понять. Вроде должно работать?

Напишите текст main() по одному оператору в строчке и запустите — должно стать понятно, в какой момент вылетает. Скорей всего дело в вводе-выводе — пиведите код vvod, vivod.

дело в том, что он не выделяет какую-либо ошибку, просто вылетает окно - General Protection Exception 

Maryann, предоставленный Вами код выглядит нормально. А если убрать все строчки из main, то будет GPE?

Если убрать все строчки из main() , никаких ошибок он не выдает
ошибка возникает, только после строчки m3=m1+m2

Это сообщение отредактировал(а) Maryann - 20.1.2008, 15:58

Maryann, а не уничтожается ли объект temp до того, как его значение будет передано в объект m3?

Под MSVC 9.0 все прекрасно работает, исключений нет. Вы от нас ничего не скрываете? ;) Какой компилятор?

Добавлено через 3 минуты и 8 секунд

С какой стати? Все чисто: все данные хранятся по значению (длина, массив int), вызывается по умолчанию сгенерённый копиктор, который их благополучно побайтово копирует.

ama_kid, возможно, и что же теперь делать? 

Maryann, ну для начала упомянуть компилятор\среду разработки, потому как у меня на MSVC++ 2005 тоже все отлично проглатывается без GPE

ama_kid, Borland C++, version 4.5 copyright 1991, 1994

Добавлено через 13 минут и 43 секунды
Только что написала то же самое для векторов - заработало!!!   Странно, что не работает для матриц

Невозможно.


Сменить компилятор(если это позволяют обстоятельства). Советую MinGW или MSVC Express.

Это сообщение отредактировал(а) archimed7592 - 20.1.2008, 17:30

Самое странное, что в Borland C++ 3.1 работает... А если вам предложить выбросить этот компилятор в топку и скачать нормальный, вы, конечно же, откажетесь?

не факт. более чем возможно что где-нибудь в не превиденном коде аффтора рушиться память.


Отладчик мой друг
Отладчик мой друг
Отладчик мой друг
Отладчик мой друг
Отладчик мой друг
Отладчик мой друг
Отладчик мой друг
Отладчик мой друг
Отладчик мой друг
Отладчик мой друг
Отладчик мой друг
Отладчик мой друг
Отладчик мой друг

У меня есть 4 или 5 разных компиляторов )), но среди них нет такого, который работал бы для всех тем. Вот например с файловой системой - ни один нормально не работает, со списками - не все, приходится уродовать алгоритмы , рекурсию в деревьях они вообще не признают, а в остальном так, средненько. Вот так и живу(. Я уже вообще не верю, что нормальные компиляторы существуют(((

Чушь какая-то... Если эти компиляторы начала 90-х(или старше), то совет сменить на современный уже был дан(ты ж не удивляешься, что запорожец не всегда ездит ). Если речь о современных, то, скорее проблемы в твоих навыках, а не в компиляторах.

Ужас какой. Как же только люди во всём мире программируют, не задумывалась? 

mingw, intel'овский, борландовский старше 98-го года, visual-овский старше 2003-го. По-моему, есть из чего выбрать 

Добавлено через 38 секунд
archimed7592, мысли опять сходятся 

Может быть, но часто бывает так, что на одном компиляторе программа работает, а на другом вылетает вообще. Вот сдам сессию - буду искать нормальный компилятор

 .

Ок, притча о программистах, у которых работает только на одном компиляторе, а на другом программа сыплется.


У меня есть ф-ция, которая меняет местами элементы в массиве. Она принимает указатель на массив и два индекса(тех элементов что нужно местами поменять).
Теперь, некоторый алгоритм насчитал мне число N, заведомо меньшее размера массива минус 2(N < arraySize - 2), и мне, по логике вещей нужно поменять местами элементы с индексами N и (N + 1). Очевидно, что для этого нужно вызвать ф-цию так: mySwap(myArray, N, N + 1); , но, так уж сложилось, что после этого вызова мне нужно работать с некоторым числом, равным N + 1, причём достаточно интенсивно(т.е. постоянно вычислять N + 1 - неэффективно). Дабы не заводить лишних переменных я решил просто увеличить N на единицу:
Но, т.к. я больно умный, я решил записать это в одну строку:
Логично? Логично! Читаемо? Безусловно! Лаконично? А то! (подумал я).
Я завершил свой проект, но полгода спустя заказчик потребовал перейти на компилятор X(по каким-то, ведомым одному ему причинам).
Что ж, подумал я, сейчас настроим скрипт сборки на компилятор X и делов то! А не тут-то было... Две недели под отладчиком не принесли результата... Заказчик был недоволен. И т.д. и т.п.

Теперь, внимание, вопрос: Maryann, как ты считаешь, кто, в данной ситуации виноват, компилятор или программист?
Подумай, может не компилятор надо искать, а книгу какую-нибудь толковую, да время найти, чтобы более детально изучить язык?  

Конечно... заказчик!   

Первое правило менеджера: клиент всегда прав .

Каким компилятором пользоваться зависит от того - каким компилятором это будут компилировать в универе, одновременно и под старые и под новые заточить код не получится.

здесь - программист

Книга - книгой, изучать язык - это все хорошо. Я, собственно, этим и занимаюсь. Но ведь в том, что моя перегрузка работает неправильно, виноват компилятор? Так что компилятор искать тоже надо.

Добавлено через 4 минуты и 13 секунд
Lazin, у нас в универе такие компиляторы, на которых мало что работает, там очень старые версии) . Поэтому под них никто и не пытается писать)))

Вина компилятора не доказана. Полный код в студию. И выполнеяемые действия, приводящие к краху.

Добавлено через 51 секунду
вспомнил пословицу: select isn't broken © кто-то

Уже было сказано, но я повторю: если компилятор допотопный, то виновата ты, посколько выбрала такой компилятор. Если компилятор современный, то скорее всего виновата ты, посколько современные компиляторы и работают одинакого(по Стандарту) и багов в них мало(по крайней мере, наткнутся на них сложно). В конце концов, попробуй как-то более конструктивно обосновать, что виноват именно компилятор и ты увидишь, что он вовсе не виноват .

У-у-у-у демагогию развели. На всякий случай напомню какой был код:



Maryann, если вам ещё актуально для сесии, попробуйте перегрузить оператор =. Может в нем компилятор тупит. Ещё по-моему тут неявно вызывается конструктор копирования, но за второе отвечать не буду.

Компилятор редко тупит... Чаще тупят программисты. К примеру, в данном случае, ни ОП, ни КК явно определять не нужно, ибо компилятор обязан определить их и, поверьте, он их определяет.

Я бы добавил: и в данном случае сгенерированные копиктор и оператор копирующего присваивания корректны. Все-таки меня терзают смутные сомнения, что от нас что-то скрывают... Maryann, вы бы выложили целиком файлы, которые не работают...

Это сообщение отредактировал(а) Ulysses4j - 22.1.2008, 01:01

ну вот:


Добавлено через 46 секунд
перед m3=m1+m2 - ошибка

Добавлено через 4 минуты и 35 секунд

я его перегрузила так:


и он тоже не работает(

а как же

?
что вводим,

Добавлено через 10 секунд
????

целые числа . Например 1 2 3

В чём точно заключается ошибка?


Не надо его перегружать!


По моим расчётам нужно вводить 8 чисел...

Maryann,  проверил этот код. работает.
вывод:


и должно работать.

Это сообщение отредактировал(а) warmonger_ - 23.1.2008, 16:09

вылетает сообщение gpe, ничего конкретного не говорится

да, 8: 1 2 3 4 1 2 3 4 например.

Добавлено через 1 минуту и 58 секунд

это хорошо. Хорошо еще, что на экзамене компилировать не надо будет...эх

Maryann, Вы можете назвать компилятор? Уже не в первый раз просим...

archimed7592, 

уже называли...

неужели автору не кажется, что он "малёхо" староват?

Вот тоже проблемы, подобного типа, при перегрузки операции + возвращает не готовый элемент, а элемент после удаления. Вот полный код:

Ошибок не выдаёт, но поля элемента M3 представляет не окончательный вариант, а вот что: 

MX: 0x00000000 -> CXX0030: Error: expression cannot be evaluated
n: -858993460
m: -858993460

Компилятор: Visual C++ 6 1994 - 1998 г. 

А если добавить деструктор, то вообще вылетает с ошибкой. 

конструктор копирования определи. А вообще лучше определить в класе операции тип *=, где * это тип операции, и определить свободные ф-ии где эти операторы и будут вызваться. Но конструктор копирования в данном случае все равно необходим.

Чё-то я не догнал, а где использовать конструктор копирования? Вот я создаю temp, в последствии его поля заполняются как надо, но когда возвращается значение из функции, результат почему-то исчезает, т.е. он не переходит в M3, почему? Я же объявил конструктор, где выделяется память под необходимое поле (MX). И не очень мне ясно, что вы упомянули о свободных функциях и операциях тип *=, если можно по-подробнее, то очень бы хотелось. Заведомо премного благодарен!

Добавлено через 9 минут и 59 секунд
Ну вот я написал конструктор копирования:

но тогда operator= не вызывается, а  в выражении M3 = M1 + M2 вызывается сразу этот самый конструктор, не дав поработать operator+ 'у. 

1. когда temp возвращается из ф-ии он копируется, и удаляется. А так как у тебя не переопределен конструктор копирования вызывается конструктор копирования по умолчанию, вследствии чего у тебя копируется указатель, который удаляется при удалении Temp. В результате в объекте который получен из temp содержится кривой указатель, который никуда не указывает.
2. 

вот тебе пример, того, что я имел в виду

Добавлено @ 09:53

вот такой у тнебя должен быть контсруктор копирования.

Это сообщение отредактировал(а) vinter - 20.4.2008, 09:59

Создал я такой конструктор копирования, написал operator+=(Matrix&), переписал соответствующим образом operator+(Matrix& rM), кстати, последний не может иметь более 1 аргументов, комп. ругается на operator+(Matrix& lM, Matrix& rM). Вот что в конечном образом получилось
Конструктор копирования

Перегруженный +=

И наконец-то operator+

И вот что получается при вызове конструктора копирования в строке  Matrix temp(*this); передается *this уже пустой, т.е.

MX 0xcccccccc -> CXX0030: Error: expression cannot be evaluated
n  -858993460
m -858993460

Вот последнего я что-то не пойму: почему именно так происходит?! 

Я немного ошибся, в конструктор в строчке Matrix temp(*this); копируется всё нормально, но вот когда возвращается результат trmp+=lM, то перед этим вызывается деструктор, который удаляет temp, а в конечный результат уже копируется, судя по всему, удалённый temp, что влечёт вызов конструктора без параметров и получаем M3: MX(NULL), n(0), m(0). Хотя без перегруженного деструктора результат тот же.




Это сообщение отредактировал(а) student0511 - 20.4.2008, 14:15

операцию покажи которую ты выполняешь.

Не не пашет, переделал, а результат остался прежним.
Вот весь текст программы

Выражение сложение двух объектов: M3 = (M1 + M2); , соответственно.

Это сообщение отредактировал(а) student0511 - 20.4.2008, 14:48

Будет работать, если здесь  
задашь размерность M3:

Ну да, работает, а почему не пашет, если не задавать размер?

для этог случая у тебя будет всегда

т.к в дефолтовом конструкторе ты обнуляешь n, m

Добавлено через 2 минуты и 5 секунд
вообще у тебя неправильно написан класс, сам допускаешь возможность NULL указателя при  присваивании, а никак это не проверяешь. Либо проверяй это, либо уберай дефолтовый конструктор в private.

Всё. Исправил. Работает. 
Вот код 

    Вот только некоторые вопросы остались: а обязательно при создании operator ('+' или '-', '+=' или '-=') результат возвращать по ссылке и передавать, и что это даёт? Хотя убрал ссылки с operator'ов + и - : всё работает вроде как надо, а убрал передачу аргумента по ссылке в += и -=, то уже никак. В чём проблема? И ещё один вопрос: как применить конструктор к указателю this и возможно ли это? А то в operator= лишний код дублируется, хотелось бы так: (*(this))(M.n, M.m), - компиллятор выдает ошибку "error C2064: term does not evaluate to a function". :-( 

Это сообщение отредактировал(а) student0511 - 21.4.2008, 00:30

student0511, читай книги.