[Gambas-user] static const?

Jussi Lahtinen jussi.lahtinen at ...626...
Tue Apr 13 21:29:30 CEST 2010


Generally bad code...
What that supposed to do?
Either it will return TabStrip3.Count or zero.

This would do exactly same thing:

PUBLIC FUNCTION ScanTab(IdCaption AS Integer) AS Integer

  FOR a = 0 TO TabStrip3.Count - 1
      IF TabStrip3[a].Caption = IdCaption THEN
         RETURN TabStrip3.Count
      ENDIF
  NEXT

END

Jussi


2010/4/13 Fabián Flores Vadell <fabianfloresvadell at ...626...>:
>> No STATIC with CONST. Or maybe the compiler is more tolerant than that?
>
> Maybe the compiler is a too much good guy :)
>
> I think that the compiler shouldn't allow to do something like that:
>
>
> PUBLIC FUNCTION ScanTab(IdCaption AS Integer) AS Integer
>
>   DIM a AS Integer = 0
>   DIM b AS Integer = 0
>
>   FOR a = 1 TO TabStrip3.Count STEP 1
>       IF TabStrip3[(a - 1)].Caption = IdCaption THEN
>          b = a
>          a = TabStrip3.Count
>          RETURN a
>       ENDIF
>   NEXT
>
> END
>
>
> Someone thought that this code is good one.
>
> Leaving aside the obvious conceptual confusion of the author, and that
> the routine doesn't do what it claims, it is important to note that it
> is possible to change the variable that controls the FOR structure,
> and included the return statement inside the loop.
>
> I think the compiler should not allow these things, at least for mercy
> through the eyes of those who are finally forced to read code like
> this.
>
>
> --
> Fabián Flores Vadell
> www.speedbooksargentina.blogspot.com
>
> ------------------------------------------------------------------------------
> Download Intel® Parallel Studio Eval
> Try the new software tools for yourself. Speed compiling, find bugs
> proactively, and fine-tune applications for parallel performance.
> See why Intel Parallel Studio got high marks during beta.
> http://p.sf.net/sfu/intel-sw-dev
> _______________________________________________
> Gambas-user mailing list
> Gambas-user at lists.sourceforge.net
> https://lists.sourceforge.net/lists/listinfo/gambas-user
>




More information about the User mailing list