Russian Qt Forum

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

Как минимум неправильно работаете со списками

C++ (Qt)
void MyServer::slotUpdateUsers  (){
    count=0;
    QTcpSocket *pClient = (QTcpSocket *)sender();
    QList<UserInfo*>::iterator o=user_list->begin();
    for (; o != user_list->end(); ++o){//ищем пользователя
        count++;
        if(user_list->at(count)->uSocket() == pClient){
            user_list->removeAt(count);
        }
    }
    pClient->deleteLater();
}
 

Какой смысл заводить итератор o и брать содержимое списка через user_list->at(count)? Причём всегда на последнем цикле выходите за границы размера списка и пропускаете первый элемент.
Внимательно читайте описание QList в ассистенте:

Сменил at на UserInfo* oa= *o, oa->...
и т.д. Не помогло. Как мне кажется проблема не в них.
тем более я нопонимаю, как я могу не брать первый итем, если a++ пост инкремент. Т.е. цикл, начинается с 0 и кончается последним итемом в листе. Единственное, что end возвращает: imaginary item after the last item in the list. А т.к. в условии стоит != то он отсекается. Как я выхожу за рамки и не получаю первый итем?

нашел от чего падает:
как только клиент говорит:
m_pTcpSocket->close();
сервер крашится. %)

disconnectFromHost () тоже самое.

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

Так же если разрывать соединение на сервере. А так все нормально работает, нехватает некоторых опций, но это из-за того, что я мучаюсь с закрытием сокета, который не хочет закрываться. Бен, ай нид хелп. ^^

небольшой совет: старайся освобождать память выделенную через оператор new! лучше конечно используй std::auto_ptr или еще что лучше boost::shared_ptr

edit:
+ не обязательно все хранить в куче, можно и на стеке

ps. на самом деле здесь столько "вкусностей" даже не знаю есть ли смысл продолжать. странно что программа у тебя раньше не упала.

Попробуйте так

C++ (Qt)
void MyServer::slotUpdateUsers( void )
{
    size_t count = 0;
    QTcpSocket *pClient = (QTcpSocket *)sender();
    while (count < user_list.size()) {
        if(user_list->at(count)->uSocket() == pClient)
            user_list->removeAt(count);
        else
            ++count;
    }
}
 

На профессионализм не претендую. :) О каких вкусностях вы говорите?

хмм, похоже сервер крашится при попытки удалить сокет:
в slotNewConnection - если оставлять, то он падает. В чем может быть загвоздка? Или там в чем угодно? :)

Границы цикла у вас определяются правильно, а обращаетесь к элементам списка вы не через итератор, а через индекс count, который всегда начинается с 1.

Как он может начинаться с одного, если перед списком идет присваивание 0, а в цикле стоит постинкремент? В случае с преинкрементом, я согласен. Или же в куте как-то меняют пост на пре?

При таком использовании инкремента нет разницы пост он или пре.
Qt тут не причём. Учите C.

Имеется в виду, что инкремент поучаствовал в выражении a++; а потом вернул увеличенное значение? Ясно. Я думал это распространяется на весь цкл. :) бу знать.

Решил проблему след образом:
Так не выдает ошибки. Почему?

Хм, я бы на вашем месте имея такие серьёзные пробелы в знаниях о программировании отложил бы в сторону написание сложных программ, тем более client-server-ных и взался за чтение книг.

А я бы на вашем месте, не давал советов, которые не просят. :) По одним книгам ничего не узнаешь.

Согласен, но учить язык при написании client-server-a мне кажется это слишком круто)

Это уже позвольте мне решать. :) Спасибо. Лучше объясните, почему переопределение deleteLater(); убрало ошибку. :)
Сам разобрался. Не на то указатель стоял в соединении сигнала и слота... ;D Он патался удалить объект логин, а не сокет.  ;D