Пожалуйста помогите найти ошибку =)

[Обледеневший]

#include<iostream>

using namespace std;

char *a=NULL;
char *b[50000];

int main()
{
    a=new char[100];
    for(int i=0;i<100;i++,a[i]=*"q");
    cin.get(a,100);

    int g=0;
    while((a[g]!=*"q") && (g<100)){
        g++;
    };
    g-=2;

    int o;
    cin>>o;

    for(int l=0;l<50000;l++,b[l]=new char[50]);
    for(int i=0;i<o;i++)
        cin.get(b[i],50);

    return 0;
}

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

[mMAg]

for(int l=0;l<50000;l++,b[l]=new char[50]);

Ну кто вас так научил писать? НЕ пользуйтесь запятой, если не уверены в том, что знаете как это работает. Операции l++,b[l]=new char[50] выполняются последовательно слева направо, а значение выражения есть результат выполнения последней операции. В результате вы выходите из цикла так и не выделив памяти... Мой вам совет, пишите лучше в следующий раз вот так:

for(int l=0;l<50000;l++){b[l]=new char[50]};

Ну а сейчас для понтов можете вот так написать:

for(int l=0;l<5;b[l]=new char[50],l++);

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

У меня под l память нормально выделилась

[netrino]

Сообщение от mMAg

В результате вы выходите из цикла так и не выделив памяти...

Немного не понял, почему?..)

Сообщение от mMAg

Ну а сейчас для понтов можете вот так написать:

for(int l=0;l<5;b[l]=new char[50],l++);

Угу, ещё и так можно:
for(int l=0; l<50000;b[l++] = new char[50]);
А вообще память в цикле лучше не выделять, особенно в таком количестве и такими маленькими кусками. А раз уж выделяете, то лучше это, как сказал mMAg, пихнуть в тело цикла, дабы это было наглядней, плюс это может избавить вас от каких-нибудь неочевидных неприятных ситуаций.

[pu4koff]

*"q" - какой ужас... и это лишь бы не писать 'q'

[Обледеневший]

Огромное спасибо всем

[mMAg]

Немного не понял, почему?..)

Позволю себе ответить на этот вопрос.

for(int l=0;l<50000;l++,b[l]=new char[50]);

Ну, я думаю, тут все просто. Вот что происходит:
l = 0;
l < 50000? да, идем дальше;
l++ - в итоге l = 1;
b[1] = new char[50].
В итоге память под b[0] не выделяется, а выделяется под b[50000]. Совершенно не то, чего мы ожидали. Именно поэтому я и предложил написать:

for(int l=0;l<50000;l++,b[l]=new char[50]);

И так можно написать, сократив мою запись:

for(int l=0; l<50000;b[l++] = new char[50]);

Только вот зачем эти лишние мороки? Я никогда не использую 2 последних варианта. Ибо это сокращает написание, но повышает вероятность ошибки на усталости.

[netrino]

А, Вы про нулевой элемент?) Я неправильно понял - решил, что Вы имеете ввиду, что не происходит ни одной итерации цикла)