Whole Tomato Software Forums
Whole Tomato Software Forums
Main Site | Profile | Register | Active Topics | Members | Search | FAQ
User name:
Password:
Save Password
Forgot your password?

 All Forums
 Visual Assist
 Feature Requests
 Limit navigation bar dropdownlist extents
 New Topic  Reply to Topic
 Printer Friendly
Author Previous Topic Topic Next Topic  

Uniwares
Tomato Guru

Portugal
2199 Posts

Posted - Dec 09 2021 :  10:27:29 AM  Show Profile  Reply with Quote
Right now I am working through a crazy .cs file, part of a (decades old) ASP.NET project, almost 20k lines of code in one file. No need to mention that a few methods here are crazy long.
Now, the drop down lists here are quite crazy too and extend over pretty much everything on screen. And still, methods and parameters are cut off somewhere in the middle without indication.

Example:
private static void FillScenario_InterestRatePromo_PaymentPromo(ProductPackageSimulationOverview Sim, LanguageSettings languageSettings, int textMaxLength, bool isActive_PR_2014_01314_3_v1_0_CheckCreditTypeId, HtmlTableRow tr_scen_interestRatePromo, UserControls.FlowItem fiMarginInterestRatePromo, Label lblMarginInterestRatePromo, HtmlTableRow tr_scen_paymentPromo, UserControls.FlowItem fiPaymentPromo, Label lblPaymentPromo) 
{ };

protected ProductPackageSimulationOverviewRequest PrepareAdjustedRequests(ServiceInput serviceInput, ucSimulator.Core.ExternalServices.wsSimulation.Scenario Scn, ProductPackageSimulationOverviewRequest SimReq, ProductPackageSimulationOverviewResponse SimResp, int UpdateModule, bool resetPlans)
{ };

Ok, so where I go with that: would be nice to have some reasonable limit on the extend of the drop down list and the way extra long lines are cut off. Currently long lines are simply truncated, so add a truncation symbol, add ellipsis, replace long var names with abbreviations, whatever, but make it "handy".

Also, the definition bar and its drop down becomes useless, since its also truncating.


feline
Whole Tomato Software

United Kingdom
17771 Posts

Posted - Dec 14 2021 :  10:05:43 AM  Show Profile  Reply with Quote
It turns out this is a known limitation, just not one that people often run into:

case=32305

I have added your request to be able to set a larger limit.

zen is the art of being at one with the two'ness
Go to Top of Page

Uniwares
Tomato Guru

Portugal
2199 Posts

Posted - Dec 14 2021 :  10:16:36 AM  Show Profile  Reply with Quote
One should never run into such thing. But there is worse lurking in such code... like String.Format() with 58 parameters. Yikes. Can we have a insta-kill feature for when someone writes code like this it will crash VS forever? Please?
Go to Top of Page

feline
Whole Tomato Software

United Kingdom
17771 Posts

Posted - Dec 14 2021 :  1:31:28 PM  Show Profile  Reply with Quote
Please tell me I didn't just read 58 parameters... my head now hurts!

zen is the art of being at one with the two'ness
Go to Top of Page

Uniwares
Tomato Guru

Portugal
2199 Posts

Posted - Dec 22 2021 :  11:21:12 AM  Show Profile  Reply with Quote
this.ddl_dis_offer.Attributes.Add("OnChange",
    "__it_setFeeDiscountOfferRepeaterValues();"
    + String.Format("__it_upDiscountOffers('{0}','{1}','{2}','{3}','{4}','{5}','{6}','{7}','{8}','{9}','{10}','{11}','{12}','{13}','{14}','{15}','{16}','{17}','{18}','{19}','{20}','{21}','{22}','{23}','{24}','{25}','{26}','{27}','{28}','{29}','{30}','{31}','{32}','{33}','{34}','{35}','{36}','{37}','{38}','{39}','{40}','{41}','{42}','{43}','{44}','{45}','{46}','{47}','{48}','{49}','{50}','{51}','{52}','{53}','{54}','{55}','{56}', '{57}', '{58}')", // remainder omitted for "brevity" 


One can not make this up.
Go to Top of Page

feline
Whole Tomato Software

United Kingdom
17771 Posts

Posted - Dec 23 2021 :  05:41:38 AM  Show Profile  Reply with Quote
I am tempted to ask why, but I doubt I would get anything approaching a sensible answer *tired sigh*

zen is the art of being at one with the two'ness
Go to Top of Page
  Previous Topic Topic Next Topic  
 New Topic  Reply to Topic
 Printer Friendly
Jump To:
© 2021 Whole Tomato Software, LLC Go To Top Of Page
Snitz Forums 2000