質問をすることでしか得られない、回答やアドバイスがある。

15分調べてもわからないことは、質問しよう!

新規登録して質問してみよう
ただいま回答率
85.35%
PHP

PHPは、Webサイト構築に特化して開発されたプログラミング言語です。大きな特徴のひとつは、HTMLに直接プログラムを埋め込むことができるという点です。PHPを用いることで、HTMLを動的コンテンツとして出力できます。HTMLがそのままブラウザに表示されるのに対し、PHPプログラムはサーバ側で実行された結果がブラウザに表示されるため、PHPスクリプトは「サーバサイドスクリプト」と呼ばれています。

配列

配列は、各データの要素(値または変数)が連続的に並べられたデータ構造です。各配列は添え字(INDEX)で識別されています。

Q&A

解決済

2回答

1381閲覧

配列問題のTLEを解決したい

KK0618

総合スコア11

PHP

PHPは、Webサイト構築に特化して開発されたプログラミング言語です。大きな特徴のひとつは、HTMLに直接プログラムを埋め込むことができるという点です。PHPを用いることで、HTMLを動的コンテンツとして出力できます。HTMLがそのままブラウザに表示されるのに対し、PHPプログラムはサーバ側で実行された結果がブラウザに表示されるため、PHPスクリプトは「サーバサイドスクリプト」と呼ばれています。

配列

配列は、各データの要素(値または変数)が連続的に並べられたデータ構造です。各配列は添え字(INDEX)で識別されています。

0グッド

0クリップ

投稿2020/08/04 01:46

編集2020/08/04 02:08

前提・実現したいこと

競技プログラミングの以下の問題を解決したいです。
問題のURL:https://www.codewars.com/kata/5ce399e0047a45001c853c2b/train/php

ls = [0, 1, 3, 6, 10] // 入力 ls = [0, 1, 3, 6, 10] ls = [1, 3, 6, 10] ls = [3, 6, 10] ls = [6, 10] ls = [10] ls = [] // 対応する和をリストにまとめて表示 [20, 20, 19, 16, 10, 0] // 出力

発生している問題・エラーメッセージ

 ある程度のサイズの配列なら大丈夫なのですが、大きすぎる配列だと

Execution Timed Out (12000 ms)

とエラーが出るので、処理の冗長を無くさなくてはいけません。
(このエラーが発生している時の入力配列は、競プロのシステム上見ることができないです。。。)

該当のソースコード

php

1function partsSums($ls) { 2 $result_array = []; 3 $length = count($ls); 4 $tmp = 0; 5 for($i = 0; $i < $length; $i++) { 6 $tmp += array_pop($ls); 7 array_unshift($result_array, $tmp); 8 } 9 $result_array[] = 0; 10 return $result_array; 11}

試したこと

繰り返し処理がおかしいと思い

while(!empty($ls)) { $result_array[] = array_sum($ls); array_shift($ls); }

と書いてみたり

for($i = $length - 1; $i >= 0; $i--) { $tmp += $ls[$i]; array_unshift($result_array, $tmp); }

と書いてみましたが、結果はやはりタイムアウトエラーでした

原因と実装方法について、アドバイスを頂いてもよろしいでしょうか。

気になる質問をクリップする

クリップした質問は、後からいつでもMYページで確認できます。

またクリップした質問に回答があった際、通知やメールを受け取ることができます。

バッドをするには、ログインかつ

こちらの条件を満たす必要があります。

anozon

2020/08/04 02:00

問題のリンクはありますか?
guest

回答2

0

問題のURLがないので最悪ケースのテストができないのですが
array_shift, array_unshift は言語によっては O(1)でなく O(n) の場合があります。

ループ内でpush、最後に array_reverse する実装に変更してもLTEになりますか?

投稿2020/08/04 02:03

anozon

総合スコア662

バッドをするには、ログインかつ

こちらの条件を満たす必要があります。

KK0618

2020/08/04 02:21

anozonさま ありがとうございます! アドバイス通りに実装しましたら、TLEにならず、実装することができました! タッチの差で別のアンサーを頂きましたので、高評価とさせてください。。。 arra_shiftやarray_unshiftがO(n)になってしまうとは予想もしていませんでした。 勉強になります!
guest

0

ベストアンサー

array_unshiftを行うと、配列全部について順序を振り直すため、配列の長さに比例したコストが発生します。

$result_arrayは逆順で([]=を使って末尾を伸ばす形で)作成しておいて、最後にarray_reverseで順番を逆転する、とするほうがいいかと思います。

投稿2020/08/04 02:02

maisumakun

総合スコア146018

バッドをするには、ログインかつ

こちらの条件を満たす必要があります。

KK0618

2020/08/04 02:19

ありがとうございます! TLEにならずに実装できました! 関数の中身の処理まで知ると、どこでコストがかかっているのか?が分かりますね! 勉強になりました!
guest

あなたの回答

tips

太字

斜体

打ち消し線

見出し

引用テキストの挿入

コードの挿入

リンクの挿入

リストの挿入

番号リストの挿入

表の挿入

水平線の挿入

プレビュー

15分調べてもわからないことは
teratailで質問しよう!

ただいまの回答率
85.35%

質問をまとめることで
思考を整理して素早く解決

テンプレート機能で
簡単に質問をまとめる

質問する

関連した質問