topic Re: Endless loop with WHILE ? in Application Development and Automation Discussions

Endless loop with WHILE ?

Former Member — Fri, 12 Sep 2008 12:15:41 GMT

Hi Fox,

The bellow coding should terminate the function after a certain time in seconds (as input parameter) and displays the variable count. But it terminates at the end with the short dump since the count becomes an overflow. Thanks for you help.

Function xxxx

import: time_out type i.

DATA: c_syuzeit TYPE sy-uzeit,

count TYPE i.

c_syuzeit = sy-uzeit.

c_syuzeit = c_syuzeit + time_out.

WHILE c_syuzeit > sy-uzeit.

ADD 1 TO count.

ENDWHILE.

ENDFUNCTION xxx.

Re: Endless loop with WHILE ?

Former Member — Fri, 12 Sep 2008 12:20:51 GMT

use EXIT with certain condition in while endwhile.

Re: Endless loop with WHILE ?

JozsefSzikszai — Fri, 12 Sep 2008 12:22:00 GMT

what is the problem?

here: WHILE c_syuzeit > sy-uzeit.

ADD 1 TO count.

ENDWHILE.

the value of syuzeit is not changed at all, which means the condition (c_syuzeit > sy-uzeit) will remain true for ever (or until count is overflown)

Re: Endless loop with WHILE ?

Former Member — Fri, 12 Sep 2008 12:24:06 GMT

Hi,

in this:

WHILE c_syuzeit > sy-uzeit.

ADD 1 TO count.

ENDWHILE.

You don't change any value of c_syuzeit and sy-uzeit, therefore you get an endless loop.

Change any value of them an insert an exit-contition.

Regards, Dieter

Re: Endless loop with WHILE ?

Former Member — Fri, 12 Sep 2008 12:28:45 GMT

well I thought the sy-uzeit is refreshed by the system automatically, isn't it like this ?

Re: Endless loop with WHILE ?

Former Member — Fri, 12 Sep 2008 12:33:16 GMT


> well I thought the sy-uzeit is refreshed by the system automatically, isn't it like this ?

But your

c_syuzeit = sy-uzeit.
c_syuzeit = c_syuzeit + time_out.

condition would also be refreshed again if sy-uzeit is refreshed by the system automatically.

means again ENDLESS loop found

Amit.

Re: Endless loop with WHILE ?

Former Member — Fri, 12 Sep 2008 12:37:45 GMT

Hello

1. Declare count TYPE p.

2. Quite unnecessary increase counter. Try simple count = 0.

Re: Endless loop with WHILE ?

JozsefSzikszai — Fri, 12 Sep 2008 12:41:18 GMT


> well I thought the sy-uzeit is refreshed by the system automatically, isn't it like this ?

yeah, you must be right. Than probably the counter was too big, could you please try with smaller, like 1?

Re: Endless loop with WHILE ?

Former Member — Fri, 12 Sep 2008 12:49:20 GMT

Yes but sy-uzeit will be chenged

Re: Endless loop with WHILE ?

suresh_datti — Fri, 12 Sep 2008 12:50:29 GMT


> Hi Fox,

I guess the endless loop will continue until Fox responds..

~Suresh

Re: Endless loop with WHILE ?

Former Member — Fri, 12 Sep 2008 12:50:35 GMT

Use ABAP command WAIT seconds

Re: Endless loop with WHILE ?

Former Member — Fri, 12 Sep 2008 12:52:53 GMT

this is not correct since

c_syuzeit = sy-uzeit.

c_syuzeit = c_syuzeit + time_out.

are not part of the loop

Re: Endless loop with WHILE ?

kostas_tsioubris — Fri, 12 Sep 2008 12:57:22 GMT

Hi,

sy-uzeit will not refresh automatically. You should add the get time statement.

e.g

DATA: c_syuzeit TYPE sy-uzeit,
count TYPE i.

c_syuzeit = sy-uzeit.
c_syuzeit = c_syuzeit + time_out.

WHILE c_syuzeit > sy-uzeit.
get time.
ADD 1 TO count.
ENDWHILE.

Kostas

Re: Endless loop with WHILE ?

Former Member — Fri, 12 Sep 2008 12:57:38 GMT

> >


> > well I thought the sy-uzeit is refreshed by the system automatically, isn't it like this ?

> yeah, you must be right. Than probably the counter was too big, could you please try with smaller, like 1?

I Guess Sy-uziet would not be refreshed as the while loops goes on.

I found Sy-uziet remain same in below piece of code:

DATA: c_syuzeit TYPE sy-uzeit,
count TYPE i.
data: time_out type i value '1'.
c_syuzeit = sy-uzeit.
c_syuzeit = c_syuzeit + time_out.

(Break-point)WHILE c_syuzeit > sy-uzeit.
ADD 1 TO count.
ENDWHILE.

PS:since Function would not be called in Recursive way.

Amit.

Re: Endless loop with WHILE ?

JozsefSzikszai — Fri, 12 Sep 2008 13:00:59 GMT

> I Guess Sy-uziet would not be refreshed as the while loops goes on.quote}

I have also read it somewhere, I just could not find it now.

I tried the following piese of code and sy-uzeit changes:

WRITE : / sy-uzeit.

WAIT UP TO 2 SECONDS.

WRITE : / sy-uzeit.

So, probably in some cases it changes, in some not. Would be great to know exactly when

Re: Endless loop with WHILE ?

kostas_tsioubris — Fri, 12 Sep 2008 13:03:12 GMT

> >


> > I Guess Sy-uziet would not be refreshed as the while loops goes on.quote}
> 
> I have also read it somewhere, I just could not find it now.
> 
> I tried the following piese of code and sy-uzeit changes:
> WRITE : / sy-uzeit.
> WAIT UP TO 2 SECONDS.
> WRITE : / sy-uzeit.
> 
> So, probably in some cases it changes, in some not. Would be great to know exactly when

My quess is that wait up to n seconds is just a sap macro (such as break username) which actually uses the get time statement in order to calculate the seconds.

Kostas

Re: Endless loop with WHILE ?

johndeconinck — Fri, 12 Sep 2008 13:11:05 GMT

Hi,

Sy-uzeit will always contain the time the report has been launched (it will not change during runtime). To update this variable you can use the ABAP command "GET TIME".

Kind regards,

John.

Re: Endless loop with WHILE ?

Former Member — Fri, 12 Sep 2008 13:12:31 GMT

that would process synchronous processing and this is not expected.

Re: Endless loop with WHILE ?

Former Member — Fri, 12 Sep 2008 13:14:55 GMT

Thanks John

Re: Endless loop with WHILE ?

vinod_vemuru2 — Fri, 12 Sep 2008 13:19:46 GMT

Hi Kami,

Well Well Well!!!!!!!!

I got the mistake u r doing.

Check below code.


DATA: time TYPE sy-uzeit,
      counter TYPE i.
time = sy-uzeit + 3.

WHILE time > sy-uzeit.
ADD 1 TO counter.
ENDWHILE.
WRITE:/1 'counter = ', counter.

Just debug this code. It works fine without any issues.

But if u directly execute the same code it gives timeout.

Here is the explanation.

When u debug for each statement COMMIT WORK will be executed and system fields will be updated like date, time.

(U can see the status message COMMIT WORK executed by the system.) But when u execute directly COMMIT WORK happend for the whole code once.

So sy-uzeit will ramain same in the loop.

So this condition will never fails.

Hope u understand.

Thanks,

Vinod.