Re: [Rails-devel] 1880 "almost" playable for review and testing- Bughunting now possible would be the better state :)

[Erik V. <eri...@xs...>]

Found the error: tile 455 was missing in the repository.  My mistake, probably.

 

There is one more change I would like to apply: the blue harbour tiles should exist as pictures only, not as separate tiles in the XML.

That’s because Tiles.xml originates from TileDesigner, where I cannot create blue tiles.

 

So the definition of Haikou would become:

      <Hex name="Q13" tile="-902" pic="-9021" orientation="4" value="20,30,40,50" score="town" city="Haikou" />

and -902/-903 should replace -9021/-9031 in all XML.

 

If score=”town” doesn’t work, it’s getting time for Stefan and/or me to start debugging. But let’s first apply these changes.

 

Erik.

 

From: Erik Vos [mailto:eri...@xs...] 
Sent: Saturday, March 10, 2012 5:52 PM
To: Dr....@t-...; 'Development list for Rails: an 18xx game'
Subject: Re: [Rails-devel] 1880 "almost" playable for review and testing- Bughunting now possible would be the better state :)

 

Hi Martin,

 

Thanks for your patch.  As usual, I have mainly looked at the impact of your changes on the generic Rails code.

 

With or without your patch, I can’t currently run 1880 because of a fatal exception:

Exception in thread "Thread-3" java.lang.NullPointerException

      at javax.swing.ImageIcon.<init>(ImageIcon.java:190)

      at rails.ui.swing.RemainingTilesWindow.init(RemainingTilesWindow.java:92)

It may be a problem on my side, I’ll try to debug that later.

 

Meanwhile, two comments on your patch:

 

1.  You have split the tile type OFF_MAP_AREA into separate OFF_MAP_CITY and OFF_MAP_TOWN types. 

I understand why you are doing that (the blue “off-map harbours” score as towns), but I don’t think this rather special 1880 feature warrants the addition of a new generic type.

There should be a much easier way to accomplish what you want: in Map.xml, replace

      <Hex name="Q13" tile="-9021" orientation="4" value="20,30,40,50" city="Haikou" />

by

      <Hex name="Q13" tile="-9021" orientation="4" value="20,30,40,50" score="town" city="Haikou" />

and similar for hex N16.

I’m saying “there should be” because this way to configure scoring is as yet hardly tested.  This is the right time to do so…

 

BTW I suppose in NetworkVertex line 251 you have

       if ((station.getType().equals(Station.OFF_MAP_CITY)) || (station.getType().equals(Station.OFF_MAP_CITY)))  {

… I suppose one of these OFF_MAP_CITY’s is a typo for OFF_MAP_TOWN.  But if you follow my suggestion, this change isn’t necessary at all.

 

 

2. For your new blue preprinted harbour tiles, you define a new tile colour name Tile.BLUE_COLOUR_NAME.

I don’t think that there is any need to define such a new name.  This and the other “colour names” in this class have only superficial relation with the actual tile colours – their function is to define the tile type with respect to upgradeability and related characteristics.  Here, I used the word “colour” for lack of a better name – the word “type” would much better describe the purpose of this item, but “type” is already heavily overused. Perhaps I should have used “level”.

 

In this context, your off-map harbour hexes behave exactly like all other off-map hexes (except for scoring, see above).  So, strange as it may seem, the “colour” of your new tiles 9021 and 9031 can best be described as “red”.

(In hindsight, “offmap” would perhaps have been a better choice for the “colour” name of all off-map tiles than “red”).

 

 

I’ll come back to your questions separately, if time permits.


Erik.

 

From: Dr....@t-... [mailto:Dr....@t-...] 
Sent: Friday, March 09, 2012 8:27 AM
To: Rails Development
Subject: [Rails-devel] 1880 "almost" playable for review and testing- Bughunting now possible would be the better state :)

 

Hi Stefan, Erik, Frederick, Brent, et al.

 

please check the code enclosed. I hope its complete and can be added to the latest master.

 

If not my git setup is definetely FUBARed.

commit c3bf5cd3efef8640b149d9f89744a6415864a1d3
Author: Martin Brumm <Dr....@t-...> 2012-03-08 22:33:22
Committer: Martin Brumm <Dr....@t-...> 2012-03-08 22:33:22
Parent: 3f95ed86c7a463c64341871e73028af7edd88911 (added the 1880 specific ShareSellingRound to the configuration.)
Branches: master

    Implementing a SpecialBuyRight for a Number of Trains, not yet finished
    need to find the trigger and a valid way to proceed for the following
    mechanismn:
    
    If a train has been bought and we have a phase change the second train
    might be acquired once by the owner of the private Rocket of China
    (2+2,3,3+3,4 are possible trains/phases) So we need to ask the player if
    he wants to exercise that special buy action... Afterwards if he does
    the paper is closed. Different approaches not yet successfull.
    
    Second special power, after the last train of 2+2 or 3 or 3+3
    provenience the owner of the P0 might once declare his company closed
    and gets 40, 70 or 100 Yuan respectivly, private company is then closed.
    Note that money goes to the player not any company.
    
    Otherwise most things should be in and working now.
    
    So we need here a big check from Erik, Stefan, Brent and Frederick if
    everything is ok :)
    
    And let the testing start... (i am damn sure something will go wrong :))
    
    Regards,
    Martin

Does anybody know a command to extract the commit history and put it in one file without the code ? :)

Known Bugs: 

StatusWindow isnt reflecting Player Order Correctly after Startround has ended: Workaround: Save and reload Game

Revenue Calculation for Minors is whacky... Havent figured out yet where the huge amount of trains is coming from :)

Advice needed on implementing the specialpowers of the privates, sofar my tries have been unsuccessfull and thus deleted from the code.

And as usual please point out or refactor the code if needed by your experience.

Probably some others testing needs now to commence :)