new pumps #84

AndreySolodovnikov · 2020-04-11T20:59:01Z

переписал помпы, теперь аналогичны майнерам

AndreySolodovnikov · 2020-04-11T21:00:24Z

сюда случайно попал коммит на новый баланс статов нефтекачек, но это тоже нужно, так что не думаю что проблема

SaloEater

Приношу свои извинения, но очень сильно захотелось так сделать.

SaloEater · 2020-04-14T07:09:07Z

...ain/java/gregtech/common/tileentities/machines/basic/GT_MetaTileEntity_BasicDrillerBase.java

+    @Override
+    public void onPostTick(IGregTechTileEntity aBaseMetaTileEntity, long aTick) {
+        super.onPostTick(aBaseMetaTileEntity, aTick);
+        if (aBaseMetaTileEntity.isServerSide()) {


Сделать return, если тикает у клиента.

Suggested change

if (aBaseMetaTileEntity.isServerSide()) {

if (this.isClientSide(aBaseMetaTileEntity)) {

mMaxProgresstime = 0;

return;

}

...

private bool isClientSide(IGregTechTileEntity aBaseMetaTileEntity)

{

return !aBaseMetaTileEntity.isServerSide();

}

SaloEater · 2020-04-14T07:15:46Z

...ain/java/gregtech/common/tileentities/machines/basic/GT_MetaTileEntity_BasicDrillerBase.java

+    public void onPostTick(IGregTechTileEntity aBaseMetaTileEntity, long aTick) {
+        super.onPostTick(aBaseMetaTileEntity, aTick);
+        if (aBaseMetaTileEntity.isServerSide()) {
+            if (aBaseMetaTileEntity.isAllowedToWork() && aBaseMetaTileEntity.isUniversalEnergyStored(ENERGY[mTier] * (SPEED[mTier] - mProgresstime)) && hasFreeSpace()) {


Было бы круто вынести в отдельный метод проверки связанные с TE

Suggested change

if (aBaseMetaTileEntity.isAllowedToWork() && aBaseMetaTileEntity.isUniversalEnergyStored(ENERGY[mTier] * (SPEED[mTier] - mProgresstime)) && hasFreeSpace()) {

if (

isTEAllowedToWorkAndStoredEnoughEnergy(aBaseMetaTileEntity)

&& hasFreeSpace()

) {

...

private bool isTEAllowedToWorkAndStoredEnoughEnergy(IGregTechTileEntity aBaseMetaTileEntity)

{

return aBaseMetaTileEntity.isAllowedToWork() && aBaseMetaTileEntity.isUniversalEnergyStored(ENERGY[mTier] * (SPEED[mTier] - mProgresstime));

}

нет ни малейшего смысла создавать функции которые будут использованы ровно 1 раз, функции нужны для упрощения понимания или для уменьшения количества кода, тут имхо ни того, ни другого

К сожалению, с этой стороны есть только мое имхо по которому и я предложил данное изменение.

SaloEater · 2020-04-14T07:38:24Z

...ain/java/gregtech/common/tileentities/machines/basic/GT_MetaTileEntity_BasicDrillerBase.java

+        if (aBaseMetaTileEntity.isServerSide()) {
+            if (aBaseMetaTileEntity.isAllowedToWork() && aBaseMetaTileEntity.isUniversalEnergyStored(ENERGY[mTier] * (SPEED[mTier] - mProgresstime)) && hasFreeSpace()) {
+                miningPipe:
+                if (waitMiningPipe) {


Можно вынести весь цикл в метод и, возможно, избавиться от флага waitMiningPipe.

Suggested change

if (waitMiningPipe) {

if (waitMiningPipe && noMiningPipesInInput()) {

mMaxProgresstime = 0;

return;

}

...

private bool noMiningPipesInInput()

{

for (int i = 0; i < mInputSlotCount; i++) {

ItemStack s = getInputAt(i);

if (s != null && s.getItem() == MINING_PIPE.getItem() && s.stackSize > 0) {

waitMiningPipe = false;

return false;

}

}

return true;

}

SaloEater · 2020-04-14T08:12:28Z

src/main/java/gregtech/common/tileentities/machines/multi/GT_MetaTileEntity_OilDrillBase.java

@@ -87,7 +87,7 @@ protected void setElectricityStats() {
        this.mEfficiencyIncrease = 10000;
        int tier = Math.max(1, GT_Utility.getTier(getMaxInputVoltage()));
        this.mEUt = -3 * (1 << (tier << 1));
-        this.mMaxProgresstime = (workState == STATE_AT_BOTTOM ? (1280 * getRangeInChunks() * getRangeInChunks() / (1 << getMinTier())) : 80) / (1 << tier);
+        this.mMaxProgresstime = (workState == STATE_AT_BOTTOM ? 320 : 80) / (1 << tier);


Не может ли быть такого, что эти значения уже вынесены в константы?
В любом случае их лучше вынести в константы.

SaloEater · 2020-04-14T08:36:35Z

...ain/java/gregtech/common/tileentities/machines/basic/GT_MetaTileEntity_BasicDrillerBase.java

+                mMaxProgresstime = 0;
+                return;
+            }
+            if (mProgresstime == SPEED[mTier] - 1) {


Выражение SPEED[mTier] - 1 стоит тоже вынести, но его смысл для меня непонятен, поэтому на ваше усмотрение.

Suggested change

if (mProgresstime == SPEED[mTier] - 1) {

if (mProgresstime != SPEED[mTier] - 1) {

return;

}

это самый банальный счетчик тиков, если поставить =! то будет работать каждый тик за исключением заданного значения => неверная работа

Обращаю внимание на 68 строку в ченже.

SaloEater · 2020-04-14T08:44:49Z

...ain/java/gregtech/common/tileentities/machines/basic/GT_MetaTileEntity_BasicDrillerBase.java

+                return;
+            }
+            if (mProgresstime == SPEED[mTier] - 1) {
+                if (isPickingPipes) {


После предыдущего ченжа будет логичнее избавиться от return'a и превратить конструкцию в

Suggested change

if (isPickingPipes) {

if (isPickingPipes) {

} else {

}

SaloEater · 2020-04-14T08:56:05Z

...ain/java/gregtech/common/tileentities/machines/basic/GT_MetaTileEntity_BasicDrillerBase.java

+                    }
+                    return;
+                }
+                if (drillY == 0) {


Выражение drillY == 0 || drillZ > RADIUS[mTier] тоже стоит вынести в отдельную функцию, но название для меня вообще не очевидно, поэтому на ваше усмотрение.

Suggested change

if (drillY == 0) {

if (drillY == 0 || drillZ > RADIUS[mTier]) {

moveOneDown(aBaseMetaTileEntity);

} else {

while (drillZ <= RADIUS[mTier]) {

while (drillX <= RADIUS[mTier]) {

if(workBlock(aBaseMetaTileEntity))

return;

drillX++;

}

drillX = -RADIUS[mTier];

drillZ++;

}

}

эта проверка совершенно лишняя, помпа не может добывать жидкости на том же уровне на котором находится контроллер

Объединил оригинальные 80 и 84 строки, чтобы избавиться от return'ов.

SaloEater · 2020-04-14T09:02:50Z

...ain/java/gregtech/common/tileentities/machines/basic/GT_MetaTileEntity_BasicDrillerBase.java

+                    if (drillY == 0) {
+                        aBaseMetaTileEntity.disableWorking();
+                        isPickingPipes = false;
+                    } else if (aBaseMetaTileEntity.getBlockOffset(0, drillY, 0) == MINING_PIPE_TIP_BLOCK || aBaseMetaTileEntity.getBlockOffset(0, drillY, 0) == MINING_PIPE_BLOCK) {


Suggested change

} else if (aBaseMetaTileEntity.getBlockOffset(0, drillY, 0) == MINING_PIPE_TIP_BLOCK || aBaseMetaTileEntity.getBlockOffset(0, drillY, 0) == MINING_PIPE_BLOCK) {

} else if (isPipeBelowTEOnOffset(drillY) {

...

private bool isPipeBelowTEOnOffset(int offset)

{

return aBaseMetaTileEntity.getBlockOffset(0, offset, 0) == MINING_PIPE_TIP_BLOCK

|| aBaseMetaTileEntity.getBlockOffset(0, offset, 0) == MINING_PIPE_BLOCK;

}

SaloEater · 2020-04-14T09:07:46Z

src/main/java/gregtech/common/tileentities/machines/basic/GT_MetaTileEntity_Miner.java

-                }
+        } else {
+            ItemData association = GT_OreDictUnificator.getAssociation(new ItemStack(block, 1, blockMeta));
+            if (association != null && association.mPrefix.toString().startsWith("ore")) {


Немного странно, что "ore" за столько лет не вынесли в константу )

Suggested change

if (association != null && association.mPrefix.toString().startsWith("ore")) {

if (doesBlockHaveOreAssociation(block, blockMeta)) {

...

private bool doesBlockHaveOreAssociation(Block block, int blockMeta)

{

ItemData association = GT_OreDictUnificator.getAssociation(new ItemStack(block, 1, blockMeta));

return association != null && association.mPrefix.toString().startsWith("ore");

}

а смысл констранты из 3х букв? думаешь это может изменится в будущем ?

Это скорее nit, просто выразил удивление, что нет стандарта для наименований.

SaloEater · 2020-04-14T09:15:18Z

src/main/java/gregtech/common/tileentities/machines/basic/GT_MetaTileEntity_Miner.java

+        int blockMeta = aBaseMetaTileEntity.getMetaIDOffset(drillX, drillY, drillZ);
+        if (block instanceof GT_Block_Ores_Abstract) {
+            TileEntity tTileEntity = getBaseMetaTileEntity().getTileEntityOffset(drillX, drillY, drillZ);
+            if (tTileEntity != null && tTileEntity instanceof GT_TileEntity_Ores && ((GT_TileEntity_Ores) tTileEntity).mNatural) {


Suggested change

if (tTileEntity != null && tTileEntity instanceof GT_TileEntity_Ores && ((GT_TileEntity_Ores) tTileEntity).mNatural) {

if (tTileEntity != null && isNaturalOre(tTileEntity )) {

...

private bool isNaturalOre(TileEntity tTileEntity)

{

return tTileEntity instanceof GT_TileEntity_Ores

&& ((GT_TileEntity_Ores) tTileEntity).mNatural;

}

new pumps

5e7984b

add gui

7c638d1

SaloEater suggested changes Apr 14, 2020

View reviewed changes

AndreySolodovnikov merged commit 03c65a3 into IIA-dev May 6, 2020

AndreySolodovnikov deleted the new-pumps branch October 25, 2020 21:56

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

new pumps #84

new pumps #84

AndreySolodovnikov commented Apr 11, 2020

AndreySolodovnikov commented Apr 11, 2020

SaloEater left a comment

SaloEater Apr 14, 2020 •

edited

Loading

SaloEater Apr 14, 2020

AndreySolodovnikov Apr 14, 2020

SaloEater Apr 14, 2020

SaloEater Apr 14, 2020

SaloEater Apr 14, 2020

SaloEater Apr 14, 2020

AndreySolodovnikov Apr 14, 2020

SaloEater Apr 14, 2020

SaloEater Apr 14, 2020

SaloEater Apr 14, 2020

AndreySolodovnikov Apr 14, 2020

SaloEater Apr 14, 2020

SaloEater Apr 14, 2020

SaloEater Apr 14, 2020

AndreySolodovnikov Apr 14, 2020

SaloEater Apr 14, 2020

SaloEater Apr 14, 2020

-        if (aBaseMetaTileEntity.isServerSide()) {
+        if (this.isClientSide(aBaseMetaTileEntity)) {
+             mMaxProgresstime = 0;
+             return;
+        }
+        ...
+        private bool isClientSide(IGregTechTileEntity aBaseMetaTileEntity)
+        {
+            return !aBaseMetaTileEntity.isServerSide();
+        }

-            if (aBaseMetaTileEntity.isAllowedToWork() && aBaseMetaTileEntity.isUniversalEnergyStored(ENERGY[mTier] * (SPEED[mTier] - mProgresstime)) && hasFreeSpace()) {
+            if (
+                isTEAllowedToWorkAndStoredEnoughEnergy(aBaseMetaTileEntity)
+                && hasFreeSpace()
+            ) {
+            ...
+            private bool isTEAllowedToWorkAndStoredEnoughEnergy(IGregTechTileEntity aBaseMetaTileEntity)
+            {
+                return aBaseMetaTileEntity.isAllowedToWork() && aBaseMetaTileEntity.isUniversalEnergyStored(ENERGY[mTier] * (SPEED[mTier] - mProgresstime));
+            }

-                if (waitMiningPipe) {
+                    if (waitMiningPipe && noMiningPipesInInput()) {
+                        mMaxProgresstime = 0;
+                        return;
+                    }
+                    ...
+                    private bool noMiningPipesInInput()
+                    {
+                        for (int i = 0; i < mInputSlotCount; i++) {
+                            ItemStack s = getInputAt(i);
+                            if (s != null && s.getItem() == MINING_PIPE.getItem() && s.stackSize > 0) {
+                                waitMiningPipe = false;
+                                return false;
+                            }
+                        }
+                        return true;
+                    }

-                if (drillY == 0) {
+                if (drillY == 0 || drillZ > RADIUS[mTier]) {
+                    moveOneDown(aBaseMetaTileEntity);
+                } else {
+                    while (drillZ <= RADIUS[mTier]) {
+                        while (drillX <= RADIUS[mTier]) {
+                            if(workBlock(aBaseMetaTileEntity))
+                              return;
+                           drillX++;
+                        }
+                        drillX = -RADIUS[mTier];
+                        drillZ++;
+                    }
+                }

-                    } else if (aBaseMetaTileEntity.getBlockOffset(0, drillY, 0) == MINING_PIPE_TIP_BLOCK || aBaseMetaTileEntity.getBlockOffset(0, drillY, 0) == MINING_PIPE_BLOCK) {
+                    } else if (isPipeBelowTEOnOffset(drillY) {
+                    ...
+                    private bool isPipeBelowTEOnOffset(int offset)
+                    {
+                       return aBaseMetaTileEntity.getBlockOffset(0, offset, 0) == MINING_PIPE_TIP_BLOCK
+                       || aBaseMetaTileEntity.getBlockOffset(0, offset, 0) == MINING_PIPE_BLOCK;
+                    }

-            if (association != null && association.mPrefix.toString().startsWith("ore")) {
+            if (doesBlockHaveOreAssociation(block, blockMeta)) {
+            ...
+            private bool doesBlockHaveOreAssociation(Block block, int blockMeta)
+            {
+                ItemData association = GT_OreDictUnificator.getAssociation(new ItemStack(block, 1, blockMeta));
+                return association != null && association.mPrefix.toString().startsWith("ore");
+            }

-            if (tTileEntity != null && tTileEntity instanceof GT_TileEntity_Ores && ((GT_TileEntity_Ores) tTileEntity).mNatural) {
+            if (tTileEntity != null && isNaturalOre(tTileEntity )) {
+            ...
+            private bool isNaturalOre(TileEntity tTileEntity)
+            {
+                return tTileEntity instanceof GT_TileEntity_Ores
+                    && ((GT_TileEntity_Ores) tTileEntity).mNatural;
+            }

new pumps #84

new pumps #84

Conversation

AndreySolodovnikov commented Apr 11, 2020

AndreySolodovnikov commented Apr 11, 2020

SaloEater left a comment

Choose a reason for hiding this comment

SaloEater Apr 14, 2020 • edited Loading

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

SaloEater Apr 14, 2020 •

edited

Loading