CSV, PHP и два дымящихся mySQL

[SaDiSST]

Полный путь - в смысле, в формате 'C:\---\---\---\---.csv'? Просто по аналогии с HTML я рассчитывал, что скрипт словит файл в одной с ним директории...

[Stanislav]

Сообщение от SaDiSST

Полный путь - в смысле, в формате 'C:\---\---\---\---.csv'? Просто по аналогии с HTML я рассчитывал, что скрипт словит файл в одной с ним директории...

Вы не поверите ловит:

	public function get_file()
	{
	
		$file_content = file ('products.csv');
		
		for ($i = $this->strStart; $i <= count($file_content); $i++) {
			if(isset($file_content[$i]))
			{
				$val = explode(';',$file_content[$i]);
				$this->mas_content[$val[0]] = $val;	
			}
		}
		
		echo '<pre>';
		print_r($this->mas_content);
		
	}

Array
(
    [1] => Array
        (
            [0] => 1
            [1] => Книга
            [2] => kniga
            [3] => 5.50
            [4] => Отличная книга. Содержит много интересной информации.
            [5] => Книги - один из основных источников знаний. Используется человечеством с незапамятных времен и может нести в себе абсолютно любую информацию
            [6] => 

        )

    [2] => Array
        (
            [0] => 2
            [1] => Ящик
            [2] => yashik
            [3] => 3.70
            [4] => Просто ящик. Нет, серьёзно.
            [5] => Служит для хранения и транспортировки чего угодно.
            [6] => 

        )

    [3] => Array
        (
            [0] => 3
            [1] => Жираф
            [2] => jiraf
            [3] => 18.50
            [4] => 
            [5] => Плюшевый жираф. Ничего особенного, если бы не его масштаб 1:2 от реального.
            [6] => 

        )

    [4] => Array
        (
            [0] => 4
            [1] => Снег
            [2] => sneg
            [3] => 0.99
            [4] => Привет, зима!
            [5] => Настоящий, пушистый и колючий снег, как из детства. Абсолютно бесполезен в странах Африканского континента.
            [6] => 

        )

    [5] => Array
        (
            [0] => 5
            [1] => Колбаса сырокопчёная
            [2] => kolbasa
            [3] => 1.20
            [4] => 
            [5] => Колбаса - бесспорно, одно из лучших изобретений человеческого гения. Уступает только двум вещам в своей полезности: колесу и сну.
            [6] => 

        )

)

[SaDiSST]

Сообщение от Stanislav

Вы не поверите ловит

Вы тоже мне не поверите, но... А, впрочем, посмотрите сами (скриншот в нормальном размере)

Но вот выводится почему-то с [3] позиции. И почему-то в панели управления БД не внесены добавленные из файла позиции...

Сообщение от SaDiSST

Но вот выводится почему-то с [3] позиции.

Этот вопрос решил - я убрал 1ю строку из .csv-файла и перенес содержимое на её место. Заменил

public $strStart = 2;

на

public $strStart = 0;

Но что не так с чтением кодировки файла?

Вопрос с кодировкой решен, всё оказалось довольно банально.

[Stanislav]

отлично можете дальше двигаться

[xen]

Посмотрел что в итоге получилось и вот какие замечания:
- в конструкторе класса Parser идет подключение к БД. Лучше передавать объект класса в конструктор, парсер не должен знать ничего о подключении к БД
- метод get_file. Можно конечно и так читать файл, но в php есть для этого готовые функции(fgetcsv)
- метод set_data.
1) update таблицы: Обновлять надо не только записи с текущим именем пользователя, но и с пустым этим полем(если не так - то что тогда делать с записями без пользователя?), при обновлении надо делать поле userID=<текущий пользователь>
2) insert: Вставка не всегда может завершиться успехом, т.к. после выборки всех записей(get_data) другой пользователь может вставить свою запись. Надо обрабатывать результат вставки.
После обработки всех записей(или в другое время) надо снять блокировки со всех строк БД, т.е. удалить пользователя. Это насколько я понял задачу. Потому что, если один пользователь добавил запись - то что теперь другой её не сможет изменить никогда?
В обоих методах не должно быть "or die".

[ADSoft]

fgetcsv- "криво" работает с кирилицей ....

[Stanislav]

Сообщение от xen

Посмотрел что в итоге получилось и вот какие замечания:
- в конструкторе класса Parser идет подключение к БД. Лучше передавать объект класса в конструктор, парсер не должен знать ничего о подключении к БД
- метод get_file. Можно конечно и так читать файл, но в php есть для этого готовые функции(fgetcsv)
- метод set_data.
1) update таблицы: Обновлять надо не только записи с текущим именем пользователя, но и с пустым этим полем(если не так - то что тогда делать с записями без пользователя?), при обновлении надо делать поле userID=<текущий пользователь>
2) insert: Вставка не всегда может завершиться успехом, т.к. после выборки всех записей(get_data) другой пользователь может вставить свою запись. Надо обрабатывать результат вставки.
После обработки всех записей(или в другое время) надо снять блокировки со всех строк БД, т.е. удалить пользователя. Это насколько я понял задачу. Потому что, если один пользователь добавил запись - то что теперь другой её не сможет изменить никогда?
В обоих методах не должно быть "or die".

по классу можно еще кучу всяких плюшек сделать, и еще бла-бла-бла, цель была помочь автору код писался в слепую, хотите пишите свою версию автору будет полезно.

по первому пункту как автору нужно так пусть и обновляет, цель не дать готовый вариант а дать толчок.

по второму пункту полный бред

В обоих методах не должно быть "or die".

Какая чушь. Если пишите, тогда давайте обоснование, мы просто прервали работу скрипта, автор захочет пусть обрабатывает исключения наводит марафет и т.д

вы похоже суть не улавливаете между помощью и написать готовое рабочее решение, не?

[xen]

По поводу die автор вроде ясно написал что надо потом вывести результат работы. По второму пункту хотелось бы услышать обоснование.

[Stanislav]

Сообщение от xen

По поводу die автор вроде ясно написал что надо потом вывести результат работы. По второму пункту хотелось бы услышать обоснование.

Для начала прежде чем лезть с советами или критиковать подучите мат-часть потом приходите ..

читать умеете?

$result = mysqli_query($sql) or DIE('Update was unsuccessful: ' . mysqli_error());

так для чего нужен or Die? а главное каким боком он связан с выдачей результатов (дальнейшие действие автора?)

по второму пункту я даже писать ничего не хочу, потому как такого бреда я еще не видовал.

insert: Вставка не всегда может завершиться успехом,

логично - автору нужно пусть обрабатывает, но читайте свое обоснование, кровь аж пошла

т.к. после выборки всех записей(get_data) другой пользователь может вставить свою запись

ээээ ....

После обработки всех записей(или в другое время) надо снять блокировки со всех строк БД

все блокировки снимаются автоматически, дальше хуже....

, т.е. удалить пользователя.

психопатия?

Потому что, если один пользователь добавил запись - то что теперь другой её не сможет изменить никогда?

лучше больше не пишите, идите почитайте книг умных, ок?

[xen]

Stanislav, либо я очень плохо объясняю, либо вы совершенно не понимаешь о чем речь.

так для чего нужен or Die? а главное каким боком он связан с выдачей результатов (дальнейшие действие автора?)

Просто бесит когда люди пишут die после возможно не успешного sql-запроса, т.к. ошибочный запрос это не всегда прерывание работы скрипта. Если нет соединения с БД - то можно сделать die. В данном же случае лучше собрать в массив результаты удачных и не удачных запросов и в конце работы скрипта вывести, например, 10 записей обновилось, 5 добавилось, 3 ошибки

insert: Вставка не всегда может завершиться успехом,

не только потому что была ошибка в запросе. Делать SELECT для проверки существования записи, а потом UPDATE по результатам работы SELECT неверно даже для одной записи(уже писал почему), а вы предлагаете вообще выбрать все записи.

все блокировки снимаются автоматически, дальше хуже....

А зачем по вашему в задаче нужно поле пользователь?
Вообще, сразу видно что у вас совершенно нет понимания того что доступ к БД может быть получен из нескольких PHP процессов одновременно и это вам следует изучить матчасть.

Stanislav, неужели до вас никак не дойдет что данный алгоритм будет нормально работать только если скрипт запускает один пользователь, если пользователей больше(о чем собственно автор и написал в задании) - то результат может быть не всегда правильным, т.к. нельзя исключать вероятность запуска скипта одновременно несколькими пользователями.